Skip to content

Commit f84fa27

Browse files
AnilAltinaygvisor-bot
authored andcommitted
Fix attempt for flakiness of testDockerComposeRun test
Make sure that docker compose completed successfully before checking it's output. PiperOrigin-RevId: 862874657
1 parent 80a9cb6 commit f84fa27

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/image/image_test.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -792,6 +792,11 @@ func testDockerComposeRun(ctx context.Context, t *testing.T, d *dockerutil.Conta
792792
if err != nil {
793793
t.Fatalf("docker compose run failed: %v", err)
794794
}
795+
waitCtx, cancel := context.WithTimeout(ctx, defaultWait)
796+
defer cancel()
797+
if status, err := execProc.WaitExitStatus(waitCtx); err != nil || status != 0 {
798+
t.Fatalf("docker compose run exited with error: %v, status: %d, docker logs: %v", err, status, dockerLogs(ctx, d))
799+
}
795800
output, err := execProc.Logs()
796801
if err != nil {
797802
t.Fatalf("docker logs failed: %v", err)

0 commit comments

Comments
 (0)