test: register docker worker with containerd snapshotter

[crazy-max]

follow-up #1886 (comment)

Adds docker+containerd worker to our matrix to be able to test against dockerd with containerd storage enabled.

[crazy-max]

@tonistiigi Needs to disable provenance for this test with containerd storage as it would export an OCI image:

{
  "containerimage.buildinfo": {
    "frontend": "dockerfile.v0",
    "attrs": {
      "filename": "Dockerfile"
    },
    "sources": [
      {
        "type": "docker-image",
        "ref": "docker.io/library/busybox:latest",
        "pin": "sha256:0d5a701f0ca53f38723108687add000e1922f812d4187dea7feaee85d2f5a6c5"
      }
    ]
  },
  "containerimage.descriptor": {
    "mediaType": "application/vnd.oci.image.index.v1+json",
    "digest": "sha256:ce51e4ac359feabf7a34388bb599e84873ba9299ce8b863ead9a093a16ffc875",
    "size": 855
  },
  "containerimage.digest": "sha256:ce51e4ac359feabf7a34388bb599e84873ba9299ce8b863ead9a093a16ffc875",
  "image.name": "moby-dangling@sha256:ce51e4ac359feabf7a34388bb599e84873ba9299ce8b863ead9a093a16ffc875"
}

This is a test case we can add in follow-up. I guess we need to read "containerimage.descriptor".annotations."config.digest".

[crazy-max]

Hum no tests run with docker+containerd. Might be related to the + char. I'm looking at it:

=== RUN   TestIntegration
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/amd64/busybox:latest@sha256:0d5a701f0ca53f38723108687add000e1922f812d4187dea7feaee85d2f5a6c5 to local mirror localhost:32933/library/busybox:latest
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/amd64/alpine:latest@sha256:c0d488a800e4127c334ad20d61d7bc21b4097540327217dfab52262adc02380c to local mirror localhost:32933/library/alpine:latest
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/moby/buildkit:buildx-stable-1 to local mirror localhost:32933/moby/buildkit:buildx-stable-1
--- PASS: TestIntegration (3.83s)
testing: warning: no tests to run

[crazy-max]

Hum no tests run with docker+containerd. Might be related to the + char. I'm looking at it:

=== RUN   TestIntegration
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/amd64/busybox:latest@sha256:0d5a701f0ca53f38723108687add000e1922f812d4187dea7feaee85d2f5a6c5 to local mirror localhost:32933/library/busybox:latest
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/amd64/alpine:latest@sha256:c0d488a800e4127c334ad20d61d7bc21b4097540327217dfab52262adc02380c to local mirror localhost:32933/library/alpine:latest
time="2023-06-20T13:49:51Z" level=info msg="trying next host - response was http.StatusNotFound" host="localhost:32933"
    run.go:266: copied docker.io/moby/buildkit:buildx-stable-1 to local mirror localhost:32933/moby/buildkit:buildx-stable-1
--- PASS: TestIntegration (3.83s)
testing: warning: no tests to run

So it's fine with go test but not with gotestsum so needs to escape this char. Will take a closer look as it might be a bug with gotestsum imo.

[tonistiigi]

Can't you do this with xx-info march ?

[crazy-max]

🤦‍♂️