Todd Lipcon has submitted this change and it was merged. Change subject: Fix flakiness in remote_bootstrap-itest ......................................................................
Fix flakiness in remote_bootstrap-itest This test deleted a tablet, and then checked to see that the tablet data state was TABLET_DATA_COPYING. However, it was possible for the remote bootstrap to complete so quickly that it would already be TABLET_DATA_READY by the time we checked, making the test flaky. This changes the check to allow either COPYING or READY states. Without the patch, it was around 8% flaky[1]. With the patch, it's not flaky[2]. [1] http://dist-test.cloudera.org/job?job_id=todd.1457557419.27817 [2] http://dist-test.cloudera.org/job?job_id=todd.1457559502.30323 Change-Id: I07b53a4209316b6a543f4d3d39ff17bed3890dd9 Reviewed-on: http://gerrit.cloudera.org:8080/2505 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/integration-tests/delete_table-test.cc M src/kudu/integration-tests/external_mini_cluster_fs_inspector.cc M src/kudu/integration-tests/external_mini_cluster_fs_inspector.h M src/kudu/integration-tests/remote_bootstrap-itest.cc M src/kudu/integration-tests/tablet_replacement-itest.cc M src/kudu/tools/kudu-ts-cli-test.cc 6 files changed, 52 insertions(+), 33 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2505 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I07b53a4209316b6a543f4d3d39ff17bed3890dd9 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
