Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 )
Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork ...................................................................... Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/7938/12/bin/bootstrap_system.sh File bin/bootstrap_system.sh: http://gerrit.cloudera.org:8080/#/c/7938/12/bin/bootstrap_system.sh@111 PS12, Line 111: if ! { service --status-all | grep -E '^ \[ \+ \] ssh$'; } > The apt-get on line 106 is calling the function on line 79. Maybe the probl Aha, looks like that was the problem. When running sudo apt-get in L79, we didn't transfer the environment variables into the sudo scope. I fixed that by adding -E. I also tested that it works. -- To view, visit http://gerrit.cloudera.org:8080/7938 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9cea56cc6e7412d87f4c2e92399a2f91ea6af6c7 Gerrit-Change-Number: 7938 Gerrit-PatchSet: 13 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Comment-Date: Thu, 26 Oct 2017 06:10:34 +0000 Gerrit-HasComments: Yes