Move to 24.04 for docker integration testing

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

.github/workflows/test.yml

@@ -267,7 +267,7 @@ jobs:
         run: GO_VERSION="$(echo ${{ matrix.go-version }} | sed -e s/.x//)" make binaries
 
   test-integration-docker-compatibility:
-    runs-on: ubuntu-22.04  # TODO: ubuntu-24.04
+    runs-on: ubuntu-24.04
     timeout-minutes: 45
     steps:
       - uses: actions/[email protected]

cmd/nerdctl/container_run_cgroup_linux_test.go

@@ -296,14 +296,12 @@ func TestRunCgroupParent(t *testing.T) {
 	t.Parallel()
 	base := testutil.NewBase(t)
 	info := base.Info()
-	containerName := testutil.Identifier(t)
-	defer base.Cmd("rm", "-f", containerName).Run()
-
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
 	}
 
+	containerName := testutil.Identifier(t)
 	t.Logf("Using %q cgroup driver", info.CgroupDriver)
 
 	parent := "/foobarbaz"
@@ -314,6 +312,13 @@ func TestRunCgroupParent(t *testing.T) {
 		parent = "foobarbaz.slice"
 	}
 
+	tearDown := func() {
+		base.Cmd("rm", "-f", containerName).Run()
+	}
+
+	tearDown()
+	t.Cleanup(tearDown)
+
 	// cgroup2 without host cgroup ns will just output 0::/ which doesn't help much to verify
 	// we got our expected path. This approach should work for both cgroup1 and 2, there will
 	// just be many more entries for cgroup1 as there'll be an entry per controller.
@@ -333,6 +338,9 @@ func TestRunCgroupParent(t *testing.T) {
 	expected := filepath.Join(parent, id)
 	if info.CgroupDriver == "systemd" {
 		expected = filepath.Join(parent, fmt.Sprintf("nerdctl-%s", id))
+		if base.Target == testutil.Docker {
+			expected = filepath.Join(parent, fmt.Sprintf("docker-%s", id))
+		}
 	}
 	base.Cmd("exec", containerName, "cat", "/proc/self/cgroup").AssertOutContains(expected)
 }