-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
DO NOT MERGE: Testing https://github.com/containers/image/pull/2321 #23348
Conversation
mtrmac
commented
Jul 19, 2024
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mtrmac The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@Luap99 @edsantiago Here I see tests failing on
where an image is pushed encrypted, removed locally, and then we test that a pull without an encryption key fails. It is the kind of thing that might well be broken by the work I am testing, but I don’t see this failing when testing locally, and I can’t currently see a code path that could cause an unexpected reuse. Is it possible that this is newly being triggered by a race in tests? I’m afraid I’m only vaguely aware of the test changes in Podman. |
Trying now with more unique image names, which might explain this. |
Like you, I'm unable to reproduce locally. Even with For reference, this is the (correct) output from a recent run on another PR:
(that is also what I see on my f40 laptop). And this is what we see in CI for this PR:
I'm confused too. |
785ce3e
to
5a6372d
Compare
I’m sorry, this is actually directly related to the c/image changes I’m testing. It does not reproduce in isolation, the failure happens because the BlobInfoCache has a record of a previous, unrelated, push of the same image. |
Thank you for the followup! |
9171607
to
07d8d8b
Compare
07d8d8b
to
4cd4dcc
Compare
eceb708
to
f2640c1
Compare
Signed-off-by: Miloslav Trmač <[email protected]>
Signed-off-by: Miloslav Trmač <[email protected]>
Signed-off-by: Miloslav Trmač <[email protected]>
f2640c1
to
b18be9e
Compare
containers/image#2321 was merged. |