From 2ccd2a0c256f8e5f6e779104a1deae48a7829c82 Mon Sep 17 00:00:00 2001 From: Florian Stadler Date: Thu, 16 Jan 2025 14:14:49 +0100 Subject: [PATCH] Increase timeout for `TestAccEcsParallel` (#1489) This test verifies that components like ECS services are created/updated in parallel. This took 1+ hours to run before fixing pulumi/pulumi#7629. The test started to become more flaky recently (see https://github.com/pulumi/pulumi-awsx/issues/1488), this coincides with upgrading the GitHub runners to Ubuntu 24.04 and onboarding awsx to ci-mgmt. Those changes might've affected the test runtime and 15 minutes were probably a bit too aggressive. Updated the timeout to 30min, which is still well below the 1 hour it took before. Fixes https://github.com/pulumi/pulumi-awsx/issues/1488 --- examples/examples_nodejs_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/examples/examples_nodejs_test.go b/examples/examples_nodejs_test.go index 0c494329e..9ffd55fcc 100644 --- a/examples/examples_nodejs_test.go +++ b/examples/examples_nodejs_test.go @@ -350,8 +350,9 @@ func TestVpc(t *testing.T) { integration.ProgramTest(t, &test) } +// Verifies that ECS services are created/updated in parallel. This took 1+ hours to run before fixing pulumi/pulumi#7629 func TestAccEcsParallel(t *testing.T) { - maxDuration(15*time.Minute, t, func(t *testing.T) { + maxDuration(30*time.Minute, t, func(t *testing.T) { test := getNodeJSBaseOptions(t). With(integration.ProgramTestOptions{ RunUpdateTest: false,