aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pierre Cheynier ...@pierre-cheynier.net>
Subject Review Request 45104: Use chmod+x to make termos_runner.pex executable
Date Mon, 21 Mar 2016 13:20:37 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45104/
-----------------------------------------------------------

Review request for Aurora.


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message