Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9502 )
Change subject: dist_test: fix the ability to pass flags to the 'run' option ...................................................................... dist_test: fix the ability to pass flags to the 'run' option The python 'argparse' module ends up considering the special '--' flag an actual flag in the "remainder". So, for a typical command line like: dist_test.py run -- --stress_cpu_threads 3 ... it was ending up passing too many '--' through to the underlying binary, which caused the arguments after it to be ignored. This simply removes the extra '--' when it is detected. Change-Id: Id191a6f35520678ce48e78be0507d9f4607a2ea8 Reviewed-on: http://gerrit.cloudera.org:8080/9502 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M build-support/dist_test.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9502 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id191a6f35520678ce48e78be0507d9f4607a2ea8 Gerrit-Change-Number: 9502 Gerrit-PatchSet: 2 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
