Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10229 )
Change subject: test_util: reset flags before cleaning test directory ...................................................................... test_util: reset flags before cleaning test directory TabletServerDiskFailureTest.TestRandomOpSequence didn't reset env_inject_eio to 0 before finishing. Rather than fixing that, let's change KuduTest to reset its flags _before_ cleaning out the test directory. This obviates the need for all such workarounds. Change-Id: I4b4843cbc2c3bc2cbeb3a4d5959f80a35da7714a Reviewed-on: http://gerrit.cloudera.org:8080/9944 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> (cherry picked from commit 3f194c3a17f11a70e64bf2668a96e1e93fed00df) Reviewed-on: http://gerrit.cloudera.org:8080/10229 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Grant Henke <[email protected]> --- M src/kudu/fs/block_manager-test.cc M src/kudu/fs/data_dirs-test.cc M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/log_block_manager-test.cc M src/kudu/tablet/diskrowset-test.cc M src/kudu/util/test_util.cc M src/kudu/util/test_util.h 7 files changed, 31 insertions(+), 28 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Grant Henke: Verified -- To view, visit http://gerrit.cloudera.org:8080/10229 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: merged Gerrit-Change-Id: I4b4843cbc2c3bc2cbeb3a4d5959f80a35da7714a Gerrit-Change-Number: 10229 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins
