Page MenuHomePhabricator

Fix server id test flakiness
ClosedPublic

Authored by sashank on Thu, Nov 19, 11:30 PM.

Details

Summary

title

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nice - sometimes when I am trying to verify a flaky test is unflaked i'll wrap it in. for trial in range(50), you could try that here too, but i think we're also pretty confident this will fix it

This revision is now accepted and ready to land.Thu, Nov 19, 11:34 PM

that's a really good idea, I'll try that. you should share that trick with the team :)

This revision was landed with ongoing or failed builds.Fri, Nov 20, 12:05 AM
This revision was automatically updated to reflect the committed changes.