Grant Henke has uploaded this change for review. (
http://gerrit.cloudera.org:8080/17514
Change subject: [test] minor clean up on
TestUnsafeChangeConfigLeaderWithPendingConfig
......................................................................
[test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig
When looking at one of recent pre-commit test failure [1][2], I found
that AdminCliTest.TestUnsafeChangeConfigLeaderWithPendingConfig failed
but it was hard to tell what was the actual status code returned:
src/kudu/tools/kudu-admin-test.cc:881: Failure
Value of: s.IsTimedOut()
Actual: false
Expected: true
I reran the scenario multiple times with TSAN-enabled binaries, but
was not able to reproduce the test failure yet. Anyways, I added more
diagnostics about the actual status code returned and cleaned up the
code of the test scenario a bit. Hopefully, next time it fails it
will be clearer what's going on.
[1] http://jenkins.kudu.apache.org/job/kudu-gerrit/23918/BUILD_TYPE=TSAN
[2] http://dist-test.cloudera.org/job?job_id=jenkins-slave.1621893605.3746155
Change-Id: Id2bf354764f3ddb371de19910c5f879c1a06c78b
Reviewed-on: http://gerrit.cloudera.org:8080/17504
Tested-by: Kudu Jenkins
Reviewed-by: Mahesh Reddy <[email protected]>
Reviewed-by: Andrew Wong <[email protected]>
Reviewed-by: Grant Henke <[email protected]>
(cherry picked from commit a0bda0989d97fac182f409ecaef3357a1fb5329f)
---
M src/kudu/tools/kudu-admin-test.cc
1 file changed, 7 insertions(+), 13 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/17514/1
--
To view, visit http://gerrit.cloudera.org:8080/17514
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.15.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2bf354764f3ddb371de19910c5f879c1a06c78b
Gerrit-Change-Number: 17514
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
