[checkout/product-catalog]: replace alpine base image with distroless image#2161
Merged
julianocosta89 merged 3 commits intoopen-telemetry:mainfrom May 12, 2025
Merged
Conversation
Red-GV
force-pushed
the
checkout-tool-pin
branch
2 times, most recently
from
f564569 to
273137d
Compare
rogercoll
reviewed
Apr 30, 2025
Contributor
rogercoll
left a comment
rogercoll
left a comment
There was a problem hiding this comment.
Thanks for adding this, changes lgtm, but I think changing images would need a changelog entry. Would you mind adding it?
Don't have a strong opinion in using distroless in favor of alpine, but anything that reduces our images size is welcomed :)
Red-GV
force-pushed
the
checkout-tool-pin
branch
from
d4ef9e0 to
3b087fe
Compare
Contributor
Author
|
Changelog added. And here is the image sizes of the resulting image when I build them. 
|
Red-GV
force-pushed
the
checkout-tool-pin
branch
3 times, most recently
from
6fdf58d to
1b0c41f
Compare
Red-GV
changed the title
…ove to distroless images Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
rogercoll
reviewed
May 5, 2025
Contributor
rogercoll
left a comment
rogercoll
left a comment
There was a problem hiding this comment.
lgtm, just added a small Dockerfile nit
Co-authored-by: Roger Coll <roger.coll@elastic.co>
julianocosta89
approved these changes
May 12, 2025
Hailong-am
pushed a commit
to Hailong-am/opentelemetry-demo
that referenced
this pull request
Jun 9, 2025
… image (open-telemetry#2161) * [checkout/product-catalog]: bump go to v1.24, pin tools via go.mod, move to distroless images Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com> * Remove unneeded GO111MODULE Co-authored-by: Roger Coll <roger.coll@elastic.co> --------- Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com> Co-authored-by: Roger Coll <roger.coll@elastic.co>
cloud-hb
pushed a commit
to cloud-hb/opentelemetry-demo
that referenced
this pull request
Nov 17, 2025
… image (open-telemetry#2161) * [checkout/product-catalog]: bump go to v1.24, pin tools via go.mod, move to distroless images Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com> * Remove unneeded GO111MODULE Co-authored-by: Roger Coll <roger.coll@elastic.co> --------- Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com> Co-authored-by: Roger Coll <roger.coll@elastic.co>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
This PR updates the
checkoutandproduct-catalogservices togo1.24. In v1.24, there is tool version pinning within thego.moditself.This also updates the final image to a distroless image instead of the alpine one. The distroless one is smaller than alpine and won't carry anything more than the minimal environment needed to run.
I can undo the
alpine->distrolesschange moving to this image is not wanted.Merge Requirements
For new features contributions, please make sure you have completed the following
essential items:
CHANGELOG.mdupdated to document new feature additionsMaintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.