Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Update the intermediate image that gets emitted to no longer rely on alpine due to glibc issues #272

Merged
merged 2 commits into from
Oct 2, 2023

Conversation

nfuden
Copy link
Collaborator

@nfuden nfuden commented Sep 28, 2023

Move to ubuntu for now.
This can be migrated to distroless in solo-projects if need be in the future but would need to have ubuntu for symbols anyways if we want to follow upstream's behavior.

https://gitlab.alpinelinux.org/alpine/tsc/-/issues/43#note_306270
sgerrand/alpine-pkg-glibc#207 (comment)
sgerrand/alpine-pkg-glibc#176

@solo-changelog-bot
Copy link

Issues linked to changelog:
https://github.com/solo-io/solo-projects/issues/5344

ci/Dockerfile Outdated Show resolved Hide resolved
@ben-taussig-solo
Copy link
Contributor

ben-taussig-solo commented Sep 29, 2023

Looks good, though it would be nice to see confirmation of this being consumed + passing CI in Gloo OSS before merging

Also, have we been able to scan this to confirm that it is no longer vulnerable to the Glibc issues?

@nfuden
Copy link
Collaborator Author

nfuden commented Oct 2, 2023

yep scanned a built version of edge and the only issues that were of critical were from the non-in docker pieces that were due to my go version (which I then updated). Scan the images from solo-io/gloo#8730 to see this

@soloio-bulldozer soloio-bulldozer bot merged commit 5974c66 into main Oct 2, 2023
2 checks passed
@soloio-bulldozer soloio-bulldozer bot deleted the fix/manual-zlib+gcc branch October 2, 2023 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants