-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix glibc not found problem #3
Merged
martinclaus
merged 10 commits into
martinclaus:main
from
kathoef:fix-glibc-not-found-problem
Apr 19, 2024
Merged
Fix glibc not found problem #3
martinclaus
merged 10 commits into
martinclaus:main
from
kathoef:fix-glibc-not-found-problem
Apr 19, 2024
Conversation
This file contains 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
Hi @kathoef! Thank you for all your effort. Much appreciated! The server returns a 404 if the subprocess with the |
Changes from #6 suggest that
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Currently, building the
image-registry-checker
image succeeds, but the "service" fails to start with these error messages,Approaching the problem was a bit "hard", especially since I do not know anything about Rust and because all the specified unit tests were succeeding (see "development environment" added to the source code directory). Steps to follow,
To approach the root cause, I then started to write a few curl-based "integration tests" and played around with starting from other base container images and also updating Rust package versions. Turns out, the problem is rooted in using the
gcr.io/distroless/cc
image, which is a pointer togcr.io/distroless/cc-debian11
.By using the newer
gcr.io/distroless/cc-debian12
(see here for "valid" image tags) theimage_registry_checker
service at least starts successfully again. Steps to demonstrate,Please note, the "integration test environment" added via
tests.sh
is a bit of a "fun with Bash" dev project... I decided to add it, because it might help to ensure that the service does indeed what it should do (especially for people without any Rust knowledge), i.e. by testing it in a more language-agnostic way. 😉This is also motivated by the following observation: Interestingly, the
image_registry_checker
service build fromgcr.io/distroless/cc-debian12
passes all specified "integration tests" intests.sh
. See here,But it fails to do so if build from e.g.
debian:12
, where theimage_registry_checker
service for some reason always returns "404 Not Found", even if a container image is known to exist in a registry. See here,Any clues about that, @martinclaus?