David Ribeiro Alves has submitted this change and it was merged. Change subject: [flaky tests] Fix ClientTest.TestFailedDnsResolution ......................................................................
[flaky tests] Fix ClientTest.TestFailedDnsResolution In this test we're testing that we get dns resolution error back when a dns resolution error fails. However there is a narrow window in which the returned error might be a master GetTabletLocations() RPC instead, because of KUDU-1466. This changes the test to retry until it gets the appropriate error. Change-Id: I9fa3ceb7215d60cd23efabf729731254524e1625 Reviewed-on: http://gerrit.cloudera.org:8080/4643 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: David Ribeiro Alves <[email protected]> --- M src/kudu/client/client-test.cc 1 file changed, 24 insertions(+), 10 deletions(-) Approvals: David Ribeiro Alves: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4643 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9fa3ceb7215d60cd23efabf729731254524e1625 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
