pulumi / pulumi-terraform-bridge

A library allowing Terraform providers to be bridged into Pulumi.
Apache License 2.0
194 stars 43 forks source link

TestStacktraceDisplayed flaky test #2388

Open VenelinMartinov opened 3 weeks ago

VenelinMartinov commented 3 weeks ago

What happened?

Looks like TestStacktraceDisplayed is flaky:

it failed on master: https://github.com/pulumi/pulumi-terraform-bridge/issues/2378

and on a recent PR: https://github.com/pulumi/pulumi-terraform-bridge/actions/runs/10686173033/job/29620708700?pr=2380

Example

.

Output of pulumi about

.

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction. To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

iwahbe commented 3 weeks ago

This doesn’t seem to be an issue reproducible with only that test. I ran the test 10,000 times locally on my laptop, and it saw no issues.

$ go test -ldflags "-X google.golang.org/protobuf/reflect/protoregistry.conflictPolicy=ignore" -test.run \^TestStacktraceDisplayed\$ -count 10000

I’m concerned that this is only reproducible by interacting TestStacktraceDisplayed with other tests.