fdb-kubernetes-operator icon indicating copy to clipboard operation
fdb-kubernetes-operator copied to clipboard

Remove sleep in tests in replace_failed_process_groups_test.go

Open xumengpanda opened this issue 1 year ago • 0 comments

What would you like to be added/changed?

The taint related test uses sleep to verify that the feature can replace pods on a timely manner. Using sleep() in unit tests is suboptimal because it can slow down the test speed and can become flaky when running on slower hardware.

Task:

  • [ ] We should consider if there is a better way to validate the timing behavior of the taint feature.
  • [ ] We should fix this flaky test .

xumengpanda avatar May 04 '23 21:05 xumengpanda