Adar Dembo has posted comments on this change. Change subject: Fixes for flaky-test build ......................................................................
Patch Set 1: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/2422/1/build-support/jenkins/build-and-test.sh File build-support/jenkins/build-and-test.sh: Line 252: echo Could not fetch flaky tests list. Did you test this failure manually? I don't trust "if <complex thing>; then ..." to maintain the exit status after that other issue you ran into. Line 260: echo No tests are flaky. I admire your optimism if you think this code path will ever be taken. -- To view, visit http://gerrit.cloudera.org:8080/2422 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id6b284e9643fc336e3dbd6a997f0bd60759e97cd Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
