Browse code

integration-cli: Skip TestRmiParentImageFail when using c8d snapshotters

With containerd image store the images don't depend on each other even
if they share the same content and it's totally fine to delete the
"parent" image.

The skip is necessary because deleting the "parent" image does not
produce an error with the c8d image store and deleting the `busybox`
image breaks other tests.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>

Paweł Gronowski authored on 2023/09/08 19:02:01
Showing 1 changed files
... ...
@@ -12,6 +12,7 @@ import (
12 12
 	"github.com/docker/docker/pkg/stringid"
13 13
 	"gotest.tools/v3/assert"
14 14
 	"gotest.tools/v3/icmd"
15
+	"gotest.tools/v3/skip"
15 16
 )
16 17
 
17 18
 type DockerCLIRmiSuite struct {
... ...
@@ -303,6 +304,8 @@ RUN echo 2 #layer2
303 303
 }
304 304
 
305 305
 func (*DockerCLIRmiSuite) TestRmiParentImageFail(c *testing.T) {
306
+	skip.If(c, testEnv.UsingSnapshotter(), "image are independent when using the containerd image store")
307
+
306 308
 	buildImageSuccessfully(c, "test", build.WithDockerfile(`
307 309
 	FROM busybox
308 310
 	RUN echo hello`))