Hello Alexey Serbin, Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14341

to look at the new patch set (#2).

Change subject: test: deflake TestBackgroundFailureDuringMaintenanceMode
......................................................................

test: deflake TestBackgroundFailureDuringMaintenanceMode

We previously weren't accounting for the possibility of a slow
bootstrap, asserting early that we had the expected number of running
replicas.

This also fixes the check for 0 failed replicas. Really, we should have
been checking for explicit healthy/failed states, rather than "not
failed".

Without this, the test failed 6/1000 times in debug mode. With it, it
passed 4997/5000 times (failures due to a different issue).

Change-Id: I67eed008a983cd9ed528496777ffe3d30126a55f
---
M src/kudu/integration-tests/maintenance_mode-itest.cc
1 file changed, 38 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/14341/2
--
To view, visit http://gerrit.cloudera.org:8080/14341
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I67eed008a983cd9ed528496777ffe3d30126a55f
Gerrit-Change-Number: 14341
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to