Skip to content

Commit 5a942fa

Browse files
krissettovvoland
authored andcommitted
Update gha runners and engines used in e2e tests
- gha runners updated to ubuntu 22.04 - e2e now runs against moby 23.0, 24.0 and 25.0 - temporarily skip broken test for moby < 25 Signed-off-by: Christopher Petito <chrisjpetito@gmail.com> (cherry picked from commit 6b67b95) Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
1 parent 592c146 commit 5a942fa

6 files changed

Lines changed: 20 additions & 17 deletions

File tree

.github/workflows/build.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ on:
1919

2020
jobs:
2121
prepare:
22-
runs-on: ubuntu-20.04
22+
runs-on: ubuntu-22.04
2323
outputs:
2424
matrix: ${{ steps.platforms.outputs.matrix }}
2525
steps:
@@ -37,7 +37,7 @@ jobs:
3737
echo ${{ steps.platforms.outputs.matrix }}
3838
3939
build:
40-
runs-on: ubuntu-20.04
40+
runs-on: ubuntu-22.04
4141
needs:
4242
- prepare
4343
strategy:
@@ -90,7 +90,7 @@ jobs:
9090
if-no-files-found: error
9191

9292
bin-image:
93-
runs-on: ubuntu-20.04
93+
runs-on: ubuntu-22.04
9494
if: ${{ github.event_name != 'pull_request' && github.repository == 'docker/cli' }}
9595
steps:
9696
-
@@ -134,7 +134,7 @@ jobs:
134134
*.cache-to=type=gha,scope=bin-image,mode=max
135135
136136
prepare-plugins:
137-
runs-on: ubuntu-20.04
137+
runs-on: ubuntu-22.04
138138
outputs:
139139
matrix: ${{ steps.platforms.outputs.matrix }}
140140
steps:
@@ -152,7 +152,7 @@ jobs:
152152
echo ${{ steps.platforms.outputs.matrix }}
153153
154154
plugins:
155-
runs-on: ubuntu-20.04
155+
runs-on: ubuntu-22.04
156156
needs:
157157
- prepare-plugins
158158
strategy:

.github/workflows/e2e.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ on:
1616

1717
jobs:
1818
e2e:
19-
runs-on: ubuntu-20.04
19+
runs-on: ubuntu-22.04
2020
strategy:
2121
fail-fast: false
2222
matrix:
@@ -28,11 +28,11 @@ jobs:
2828
- alpine
2929
- debian
3030
engine-version:
31-
# - 20.10-dind # FIXME: Fails on 20.10
32-
- stable-dind # TODO: Use 20.10-dind, stable-dind is deprecated
33-
include:
34-
- target: non-experimental
35-
engine-version: 19.03-dind
31+
- 25.0 # latest
32+
- 24.0 # latest - 1
33+
- 23.0 # mirantis lts
34+
# TODO(krissetto) 19.03 needs a look, doesn't work ubuntu 22.04 (cgroup errors).
35+
# we could have a separate job that tests it against ubuntu 20.04
3636
steps:
3737
-
3838
name: Checkout

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ on:
1616

1717
jobs:
1818
ctn:
19-
runs-on: ubuntu-20.04
19+
runs-on: ubuntu-22.04
2020
steps:
2121
-
2222
name: Checkout

.github/workflows/validate.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ on:
1616

1717
jobs:
1818
validate:
19-
runs-on: ubuntu-20.04
19+
runs-on: ubuntu-22.04
2020
strategy:
2121
fail-fast: false
2222
matrix:
@@ -37,7 +37,7 @@ jobs:
3737

3838
# check that the generated Markdown and the checked-in files match
3939
validate-md:
40-
runs-on: ubuntu-20.04
40+
runs-on: ubuntu-22.04
4141
steps:
4242
-
4343
name: Checkout
@@ -57,7 +57,7 @@ jobs:
5757
fi
5858
5959
validate-make:
60-
runs-on: ubuntu-20.04
60+
runs-on: ubuntu-22.04
6161
strategy:
6262
fail-fast: false
6363
matrix:
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
Pull (1 of 1): registry:5000/trust-pull:latest@sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501
2-
sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501: Pulling from trust-pull
2+
registry:5000/trust-pull@sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501: Pulling from trust-pull
33
Digest: sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501
44
Status: Downloaded newer image for registry:5000/trust-pull@sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501
55
registry:5000/trust-pull:latest

e2e/plugin/trust_test.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import (
1111
"github.com/docker/cli/e2e/internal/fixtures"
1212
"github.com/docker/cli/internal/test/environment"
1313
"github.com/docker/docker/api/types"
14+
"github.com/docker/docker/api/types/versions"
1415
"github.com/pkg/errors"
1516
"gotest.tools/v3/assert"
1617
"gotest.tools/v3/fs"
@@ -21,6 +22,8 @@ import (
2122
const registryPrefix = "registry:5000"
2223

2324
func TestInstallWithContentTrust(t *testing.T) {
25+
// TODO(krissetto): remove this skip once the fix (see https://github.com/moby/moby/pull/47299) is deployed to moby versions < 25
26+
skip.If(t, versions.LessThan(environment.DaemonAPIVersion(t), "1.44"))
2427
skip.If(t, environment.SkipPluginTests())
2528

2629
pluginName := fmt.Sprintf("%s/plugin-content-trust", registryPrefix)
@@ -50,7 +53,7 @@ func TestInstallWithContentTrust(t *testing.T) {
5053
fixtures.WithNotary,
5154
)
5255
result.Assert(t, icmd.Expected{
53-
Out: fmt.Sprintf("Status: Downloaded newer image for %s@sha", pluginName),
56+
Out: fmt.Sprintf("Installed plugin %s", pluginName),
5457
})
5558
}
5659

0 commit comments

Comments
 (0)