Skip to content

Commit

Permalink
testing: examples/ecs/nodejs add missing dependsOn
Browse files Browse the repository at this point in the history
Pulumi was not recognizing that the underlying aws.ecs.Service should wait for the load balancer to be provisioned,
causing a race that sporadically would flake up the test. This is now compensated by with a dependsOn option.

Fixes #1418
  • Loading branch information
t0yv0 committed Dec 16, 2024
1 parent c1a760e commit 2a6ccd1
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions examples/ecs/nodejs/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ const service = new ecs.FargateService("my-service", {
subnets: vpc.publicSubnetIds,
assignPublicIp: true,
},
}, {
dependsOn: [lb.loadBalancer],
});

// Export the load balancer's address so that it's easy to access.
Expand Down

0 comments on commit 2a6ccd1

Please sign in to comment.