Skip to content

test: image volume from an already-local image - #14026

Closed
ndeloof wants to merge 1 commit into
mainfrom
e2e-image-volume-local
Closed

test: image volume from an already-local image#14026
ndeloof wants to merge 1 commit into
mainfrom
e2e-image-volume-local

Conversation

@ndeloof

@ndeloof ndeloof commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

What I did
E2E regression test for #14005, pulled-image scenario: with the source image of a type: image volume already present in the local store, compose used to rewrite the mount source to a digest the daemon can't resolve as a mount source under the containerd image store (No such image). TestImageVolume only exercises this path when a previous test happens to have left the image in the local store — which is how openQA caught it while CI stayed green. Pre-pulling the image makes it deterministic, and a second unchanged up is asserted not to recreate the service.

Verified locally against a containerd image store daemon: fails on main, passes with #14011#14011 is the fix for #14005 (the built-image scenario is already covered by TestImageVolumeRecreateOnRebuild, which its containerd CI job will exercise). Unlike the built-image tests, this one only bites under the containerd image store, so it stays green on the current graphdriver CI jobs and can merge independently.

Related issue
Reproduces #14005, fixed by #14011.

Regression test for #14005, pulled-image scenario: with the source image
of a type=image volume already present in the local store, compose used
to rewrite the mount source to a digest the daemon can't resolve as a
mount source under the containerd image store (No such image).
TestImageVolume only exercises this path when a previous test happens to
have left the image locally; pre-pulling makes it deterministic. Also
asserts a second unchanged up doesn't recreate the service.

The test fails on main under the containerd image store and passes with
the fix from #14011.

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
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.

1 participant