Hello Todd Lipcon,
I'd like you to do a code review. Please visit
to review the following change.
Change subject: Fix flaky disk_reservation-itest
Fix flaky disk_reservation-itest
There are two fixes in this patch for two separate types of failures
seen on Jenkins for this test:
1. Fix a data race in DiskReservationITest.TestFillMultipleDisks
We can't override gflag strings at runtime in a thread-safe manner,
although this test was attempting to.
Take what used to be a single parsed string gflag and replace it with 2
path strings and 2 integer overrides, one for each path. That makes 4
new test-only gflags total. Only the integer flags are modified at
2. Fix a startup race between the TestWorkload client thread and
SetFlags() in DiskReservationITest.TestWalWriteToFullDiskAborts
We need to wait for some rows to be written after starting up the
TestWorkload threads in TestWalWriteToFullDiskAborts before we allow the
TS to crash by setting gflags. If we don't, the test gets confused
because the TestWorkload client thread may not be able to resolve where
the tablet is located. The previous failures were because we sometimes
managed to crash the TS before it sent its tablet report to the master.
After applying these changes, I looped disk_reservation-itest 1000x in
TSAN mode and got no failures.
2 files changed, 61 insertions(+), 29 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/52/3652/1
To view, visit http://gerrit.cloudera.org:8080/3652
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>