Skip to content

Commit

Permalink
Increase timeout for TestAccEcsParallel (#1489)
Browse files Browse the repository at this point in the history
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
#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 #1488
  • Loading branch information
flostadler authored Jan 16, 2025
1 parent 7afa022 commit 2ccd2a0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion examples/examples_nodejs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit 2ccd2a0

Please sign in to comment.