Todd Lipcon has submitted this change and it was merged. Change subject: benchmarks: add set -o pipefail ......................................................................
benchmarks: add set -o pipefail It seems that our automated benchmarks job was failing to build for the last several days, but we didn't notice because it didn't use 'pipefail'. Thus the 'make | tee build.log' failure didn't actually cause the script to exit with a bad status code, and we kept running the binaries from a previous revision. Change-Id: I6ab66e85cf39c6513da61e472c85e34da77205fc Reviewed-on: http://gerrit.cloudera.org:8080/7123 Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Adar Dembo <[email protected]> Tested-by: Todd Lipcon <[email protected]> --- M src/kudu/scripts/benchmarks.sh 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Todd Lipcon: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7123 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ab66e85cf39c6513da61e472c85e34da77205fc Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
