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

fix: try other images when blob not found #20

Merged
merged 1 commit into from
Jul 30, 2024
Merged

fix: try other images when blob not found #20

merged 1 commit into from
Jul 30, 2024

Conversation

knqyf263
Copy link
Collaborator

@knqyf263 knqyf263 commented Jul 29, 2024

For example, if debian:10 and debian:11 are set as test images, the API endpoint prefix for retrieving the layer will be the same. That is, /v2/debian/blobs/sha256:xxx. If a client tries to retrieve the layer in debian:11, an error will be returned as the layer does not exist in debian:10. Other images have to be tried. This PR fixes the bug.

Copy link
Contributor

@DmitriyLewen DmitriyLewen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@knqyf263 knqyf263 merged commit 7abfde1 into main Jul 30, 2024
5 checks passed
@knqyf263 knqyf263 deleted the multiple_imgs branch July 30, 2024 03:53
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.

2 participants