Remove arm/v7 from testing

Signed-off-by: apostasie <[email protected]>

Author: apostasie

.github/workflows/test-canary.yml

@@ -40,7 +40,8 @@ jobs:
           # To avoid that, we explicitly list the architectures we do want emulation for.
           docker run --privileged --rm tonistiigi/binfmt --install linux/amd64
           docker run --privileged --rm tonistiigi/binfmt --install linux/arm64
-          docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
+      - name: "Run unit tests"
+        run: go test -v ./pkg/...
       - name: "Run integration tests"
         run: docker run -t --rm --privileged test-integration ./hack/test-integration.sh -test.only-flaky=false
       - name: "Run integration tests (flaky)"

.github/workflows/workflow-test.yml

@@ -127,7 +127,6 @@ jobs:
           # To avoid that, we explicitly list the architectures we do want emulation for.
           docker run --privileged --rm tonistiigi/binfmt --install linux/amd64
           docker run --privileged --rm tonistiigi/binfmt --install linux/arm64
-          docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
       - name: "Run integration tests"
         run: docker run -t --rm --privileged test-integration ./hack/test-integration.sh -test.only-flaky=false
       - name: "Run integration tests (flaky)"
@@ -185,7 +184,6 @@ jobs:
           # To avoid that, we explicitly list the architectures we do want emulation for.
           docker run --privileged --rm tonistiigi/binfmt --install linux/amd64
           docker run --privileged --rm tonistiigi/binfmt --install linux/arm64
-          docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
       - name: "Run integration tests"
         # The nested IPv6 network inside docker and qemu is complex and needs a bunch of sysctl config.
         # Therefore, it's hard to debug why the IPv6 tests fail in such an isolation layer.
@@ -269,7 +267,6 @@ jobs:
           # To avoid that, we explicitly list the architectures we do want emulation for.
           docker run --privileged --rm tonistiigi/binfmt --install linux/amd64
           docker run --privileged --rm tonistiigi/binfmt --install linux/arm64
-          docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
       - name: "Expose GitHub Runtime variables for gha"
         uses: crazy-max/ghaction-github-runtime@3cb05d89e1f492524af3d41a1c98c83bc3025124  # v3.1.0
       - name: "Prepare (network driver=slirp4netns, port driver=builtin)"
@@ -314,7 +311,6 @@ jobs:
           # To avoid that, we explicitly list the architectures we do want emulation for.
           docker run --privileged --rm tonistiigi/binfmt --install linux/amd64
           docker run --privileged --rm tonistiigi/binfmt --install linux/arm64
-          docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
       - name: "Prepare integration test environment"
         run: |
           # FIXME: remove expect when we are done removing unbuffer from tests

cmd/nerdctl/container/multi_platform_linux_test.go

@@ -32,13 +32,12 @@ import (
 
 func testMultiPlatformRun(base *testutil.Base, alpineImage string) {
 	t := base.T
-	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64", "linux/arm/v7")
+	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64")
 	testCases := map[string]string{
-		"amd64":        "x86_64",
-		"arm64":        "aarch64",
-		"arm":          "armv7l",
-		"linux/arm":    "armv7l",
-		"linux/arm/v7": "armv7l",
+		"amd64":          "x86_64",
+		"arm64":          "aarch64",
+		"linux/arm64":    "aarch64",
+		"linux/arm64/v8": "aarch64",
 	}
 	for plat, expectedUnameM := range testCases {
 		t.Logf("Testing %q (%q)", plat, expectedUnameM)
@@ -56,7 +55,7 @@ func TestMultiPlatformBuildPush(t *testing.T) {
 	testutil.DockerIncompatible(t) // non-buildx version of `docker build` lacks multi-platform. Also, `docker push` lacks --platform.
 	testutil.RequiresBuild(t)
 	testutil.RegisterBuildCacheCleanup(t)
-	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64", "linux/arm/v7")
+	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64")
 	base := testutil.NewBase(t)
 	tID := testutil.Identifier(t)
 	reg := testregistry.NewWithNoAuth(base, 0, false)
@@ -71,9 +70,9 @@ RUN echo dummy
 
 	buildCtx := helpers.CreateBuildContext(t, dockerfile)
 
-	base.Cmd("build", "-t", imageName, "--platform=amd64,arm64,linux/arm/v7", buildCtx).AssertOK()
+	base.Cmd("build", "-t", imageName, "--platform=amd64,linux/arm64/v8", buildCtx).AssertOK()
 	testMultiPlatformRun(base, imageName)
-	base.Cmd("push", "--platform=amd64,arm64,linux/arm/v7", imageName).AssertOK()
+	base.Cmd("push", "--platform=amd64,linux/arm64/v8", imageName).AssertOK()
 }
 
 // TestMultiPlatformBuildPushNoRun tests if the push succeeds in a situation where nerdctl builds
@@ -83,7 +82,7 @@ func TestMultiPlatformBuildPushNoRun(t *testing.T) {
 	testutil.DockerIncompatible(t) // non-buildx version of `docker build` lacks multi-platform. Also, `docker push` lacks --platform.
 	testutil.RequiresBuild(t)
 	testutil.RegisterBuildCacheCleanup(t)
-	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64", "linux/arm/v7")
+	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64")
 	base := testutil.NewBase(t)
 	tID := testutil.Identifier(t)
 	reg := testregistry.NewWithNoAuth(base, 0, false)
@@ -98,9 +97,9 @@ CMD echo dummy
 
 	buildCtx := helpers.CreateBuildContext(t, dockerfile)
 
-	base.Cmd("build", "-t", imageName, "--platform=amd64,arm64,linux/arm/v7", buildCtx).AssertOK()
+	base.Cmd("build", "-t", imageName, "--platform=amd64,linux/arm64/v8", buildCtx).AssertOK()
 	testMultiPlatformRun(base, imageName)
-	base.Cmd("push", "--platform=amd64,arm64,linux/arm/v7", imageName).AssertOK()
+	base.Cmd("push", "--platform=amd64,linux/arm64/v8", imageName).AssertOK()
 }
 
 func TestMultiPlatformPullPushAllPlatforms(t *testing.T) {
@@ -123,7 +122,7 @@ func TestMultiPlatformComposeUpBuild(t *testing.T) {
 	testutil.DockerIncompatible(t)
 	testutil.RequiresBuild(t)
 	testutil.RegisterBuildCacheCleanup(t)
-	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64", "linux/arm/v7")
+	testutil.RequireExecPlatform(t, "linux/amd64", "linux/arm64")
 	base := testutil.NewBase(t)
 
 	const dockerComposeYAML = `
@@ -135,14 +134,9 @@ services:
     - 8080:80
   svc1:
     build: .
-    platform: arm64
+    platform: linux/arm64/v8
     ports:
     - 8081:80
-  svc2:
-    build: .
-    platform: linux/arm/v7
-    ports:
-    - 8082:80
 `
 	dockerfile := fmt.Sprintf(`FROM %s
 RUN uname -m > /usr/share/nginx/html/index.html
@@ -159,7 +153,6 @@ RUN uname -m > /usr/share/nginx/html/index.html
 	testCases := map[string]string{
 		"http://127.0.0.1:8080": "x86_64",
 		"http://127.0.0.1:8081": "aarch64",
-		"http://127.0.0.1:8082": "armv7l",
 	}
 
 	for testURL, expectedIndexHTML := range testCases {