Skip to content

Commit

Permalink
cephfs admin: add additional delay when module is enabled
Browse files Browse the repository at this point in the history
There was continued flakiness when the tests that enabled the cephfs
mirroring module were enabled. This change adds another 200 ms delay
to hopefully cover the time that might occur as the mgr has a new
module enabled and restart. But unfortunately this is still a bit of
a wild guess.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
  • Loading branch information
phlogistonjohn committed Dec 6, 2022
1 parent 98c41c3 commit 2a593d1
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cephfs/admin/mirror_workflow_test.go
Expand Up @@ -33,6 +33,8 @@ func waitForMirroring(t *testing.T, fsa *FSAdmin) {
require.NoError(t, err)
for _, emod := range modinfo.EnabledModules {
if emod == "mirroring" {
// give additional time for mgr to restart(?)
time.Sleep(200 * time.Millisecond)
return
}
}
Expand Down

0 comments on commit 2a593d1

Please sign in to comment.