Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14341 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14341 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/integration-tests/maintenance_mode-itest.cc 1 file changed, 38 insertions(+), 18 deletions(-) Approvals: Adar Dembo: Looks good to me, but someone else must approve Andrew Wong: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I67eed008a983cd9ed528496777ffe3d30126a55f Gerrit-Change-Number: 14341 Gerrit-PatchSet: 3 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)
