Ver Fonte

Merge pull request #45103 from vvoland/testlogs-longer-timeout

TestLogs: Increase stop check poll timeout on Windows
Sebastiaan van Stijn há 2 anos atrás
pai
commit
7cda1fc4b6

+ 8 - 1
integration/container/logs_test.go

@@ -124,6 +124,11 @@ func testLogs(t *testing.T, logDriver string) {
 		},
 	}
 
+	pollTimeout := time.Second * 10
+	if testEnv.OSType == "windows" {
+		pollTimeout = StopContainerWindowsPollTimeout
+	}
+
 	for _, tC := range testCases {
 		tC := tC
 		t.Run(tC.desc, func(t *testing.T) {
@@ -136,7 +141,9 @@ func testLogs(t *testing.T, logDriver string) {
 			)
 			defer client.ContainerRemove(ctx, id, types.ContainerRemoveOptions{Force: true})
 
-			poll.WaitOn(t, container.IsStopped(ctx, client, id), poll.WithDelay(time.Millisecond*100))
+			poll.WaitOn(t, container.IsStopped(ctx, client, id),
+				poll.WithDelay(time.Millisecond*100),
+				poll.WithTimeout(pollTimeout))
 
 			logs, err := client.ContainerLogs(ctx, id, tC.logOps)
 			assert.NilError(t, err)

+ 3 - 0
integration/container/stop_test.go

@@ -11,6 +11,9 @@ import (
 	"gotest.tools/v3/poll"
 )
 
+// hcs can sometimes take a long time to stop container.
+const StopContainerWindowsPollTimeout = 75 * time.Second
+
 func TestStopContainerWithRestartPolicyAlways(t *testing.T) {
 	defer setupTest(t)()
 	client := testEnv.APIClient()

+ 1 - 1
integration/container/wait_test.go

@@ -170,7 +170,7 @@ func TestWaitConditions(t *testing.T) {
 				assert.NilError(t, err)
 			case waitRes := <-waitResC:
 				assert.Check(t, is.Equal(int64(99), waitRes.StatusCode))
-			case <-time.After(75 * time.Second):
+			case <-time.After(StopContainerWindowsPollTimeout):
 				info, _ := cli.ContainerInspect(ctx, containerID)
 				t.Fatalf("Timed out waiting for container exit code (status = %q)", info.State.Status)
 			}