----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45104/#review124544 -----------------------------------------------------------
Master (b24619b) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On March 21, 2016, 1:21 p.m., Pierre Cheynier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45104/ > ----------------------------------------------------------- > > (Updated March 21, 2016, 1:21 p.m.) > > > Review request for Aurora, Joshua Cohen and Bill Farner. > > > Repository: aurora > > > Description > ------- > > When using `--executor_environment_variables` without explicitely > passing LD_LIBRARY_PATH, `sys.executable` returns an empty string > resulting in a '[Errno 13] Permission denied' error for every launched > task. > > Moreover, it seems that this feature is coming in 0.30: "Executors no > longer inherit environment variables from the agent". > > This patch partially revert back 07ce21d where chmod_x method was > removed in favor of using sys.executable. > > > Diffs > ----- > > src/main/python/apache/aurora/executor/thermos_task_runner.py > 3896e3841562600379705dbf78a6f62728246348 > > Diff: https://reviews.apache.org/r/45104/diff/ > > > Testing > ------- > > Make Aurora run on CentOS7 > > > Thanks, > > Pierre Cheynier > >