You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Anindya Sinha <an...@apple.com> on 2016/04/15 01:04:20 UTC

Review Request 46228: Create persistent volume with a supplied user.

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

Review request for mesos and Jiang Yan Xu.


Bugs: MESOS-4893
    https://issues.apache.org/jira/browse/MESOS-4893


Repository: mesos


Description
-------

If user is specified in Resource::DiskInfo::Persistence, set the
ownership of the volume to that user. Tasks should have to comply to
ownership of the volume before they can use the volume.
If user is not specified in Resource::DiskInfo::Persistence, it is
created with the user mesos-slave process runs as. When a task is
launched, it updates the ownership of the persistent volume with its
user so as to be able to write into that persistent volume.


Diffs
-----

  src/common/resources.cpp f6ff92b591c15bc8e93fd85e1896349c3a7bb968 
  src/slave/containerizer/mesos/isolators/filesystem/linux.cpp 9fc7c48f99155750fd3c18c7c102507e2726362b 
  src/slave/containerizer/mesos/isolators/filesystem/posix.cpp 01c0ad6dbb6d509e62e769365586b3d23dcb240d 
  src/slave/slave.cpp de99e9eb5cc812b2e07deb749b98b4f4db363728 
  src/v1/resources.cpp 8c3f2d1c1529915a59d47fe37bb3fc7a3267079a 

Diff: https://reviews.apache.org/r/46228/diff/


Testing
-------

All tests passed.


Thanks,

Anindya Sinha