temporal icon indicating copy to clipboard operation
temporal copied to clipboard

Fix TestWorkflowTaskRedirectInRetryFirstTask flake

Open carlydf opened this issue 1 year ago • 0 comments

What changed?

  • Wait for build id search attribute v1.1 to land in visibility before validating entire build id search attribute structure (this solves the observed flake)
  • Wait for build id search attribute v1.2 before validating it (not sure if this would flake, but it could)
  • Add comment explaining that we should wait before validating

Why?

Here's a run with the latest flake: https://github.com/temporalio/temporal/actions/runs/9575636250/job/26400905169 We need to make sure that the build ID search attribute has enough time to update before we validate all the parts of it.

How did you test it?

Tested locally, testing in CI now

Potential risks

None

Documentation

Commented function

Is hotfix candidate?

No

carlydf avatar Jun 22 '24 00:06 carlydf