Merge pull request #1141 from AkihiroSuda/fix-TestRunAddHost

AkihiroSuda · web-flow · commit 9ddf5226eabc · 2022-06-18T21:36:12.000+09:00
CI: unparallelize TestRunAddHost
diff --git a/cmd/nerdctl/run_linux_test.go b/cmd/nerdctl/run_linux_test.go
@@ -87,7 +87,7 @@ func TestRunIpcHost(t *testing.T) {
 }
 
 func TestRunAddHost(t *testing.T) {
-	t.Parallel()
+	// Not parallelizable (https://github.com/containerd/nerdctl/issues/1127)
 	base := testutil.NewBase(t)
 	base.Cmd("run", "--rm", "--add-host", "testing.example.com:10.0.0.1", testutil.AlpineImage, "cat", "/etc/hosts").AssertOutWithFunc(func(stdout string) error {
 		var found bool

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ func TestRunIpcHost(t *testing.T) {`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`func TestRunAddHost(t *testing.T) {`
`90`		`- t.Parallel()`
	`90`	`+ // Not parallelizable (https://github.com/containerd/nerdctl/issues/1127)`
`91`	`91`	`base := testutil.NewBase(t)`
`92`	`92`	`base.Cmd("run", "--rm", "--add-host", "testing.example.com:10.0.0.1", testutil.AlpineImage, "cat", "/etc/hosts").AssertOutWithFunc(func(stdout string) error {`
`93`	`93`	`var found bool`