You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Ben Mahler <be...@gmail.com> on 2013/01/15 01:08:05 UTC
Review Request: Recursively chown the executor work directory after
extracting resources.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8947/
-----------------------------------------------------------
Review request for mesos, Benjamin Hindman and Vinod Kone.
Description
-------
Executes 'chown -R user:group .' while in the work directory, in order to chown the extracted executor resources.
For some chown subtlety: http://aplawrence.com/Unixart/chown.html
This addresses bug MESOS-332.
https://issues.apache.org/jira/browse/MESOS-332
Diffs
-----
src/launcher/launcher.cpp 3ce8081edba669157a14b815503322ea8d367d43
Diff: https://reviews.apache.org/r/8947/diff/
Testing
-------
make check
Thanks,
Ben Mahler
Re: Review Request: Recursively chown the executor work directory after
extracting resources.
Posted by Ben Mahler <be...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8947/
-----------------------------------------------------------
(Updated Jan. 15, 2013, 2:42 a.m.)
Review request for mesos, Benjamin Hindman and Vinod Kone.
Changes
-------
Updated os::chown to include a recursive flag.
Description
-------
Executes 'chown -R user:group .' while in the work directory, in order to chown the extracted executor resources.
For some chown subtlety: http://aplawrence.com/Unixart/chown.html
This addresses bug MESOS-332.
https://issues.apache.org/jira/browse/MESOS-332
Diffs (updated)
-----
src/launcher/launcher.cpp 3ce8081edba669157a14b815503322ea8d367d43
third_party/libprocess/include/stout/os.hpp 5b5b33636cb396c3324ab7f3f42144f4ab30ef95
Diff: https://reviews.apache.org/r/8947/diff/
Testing
-------
make check
Thanks,
Ben Mahler