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:03:25 UTC

Review Request 46227: Added an user to indicate owner of persistent volume.

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

Review request for mesos and Jiang Yan Xu.


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


Repository: mesos


Description
-------

When a persistent volume is created, it is created with the user
indicated in Resource::DiskInfo::Persistence. If missing, it would
fall back to make the user of the mesos-slave process as the owner
of the persistent volume.


Diffs
-----

  include/mesos/mesos.proto 87af4a06db8cc3889fe4d3b314206103f5ce5f2d 
  include/mesos/v1/mesos.proto 34da0a1484dc2f71262d8b97484b8edaae35bb6c 
  src/CMakeLists.txt 06f58c4a88e3c527df727df1efe11ed3ab77efa8 
  src/Makefile.am 1ae0ea7f3ff6f136d58ef08e7b11185ea19f37a8 
  src/slave/containerizer/mesos/isolators/docker/volume/isolator.hpp 786f917c0ec04b6839bbd524fa7c8de3729f9bdb 
  src/slave/containerizer/mesos/isolators/docker/volume/isolator.cpp 0ad473dc3bd45e122fba55a670e1a893e61c977a 

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


Testing
-------

All tests passed.


Thanks,

Anindya Sinha


Re: Review Request 46227: Added an user to indicate owner of persistent volume.

Posted by Anindya Sinha <an...@apple.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46227/
-----------------------------------------------------------

(Updated April 14, 2016, 11:07 p.m.)


Review request for mesos and Jiang Yan Xu.


Changes
-------

rebased to HEAD.


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


Repository: mesos


Description
-------

When a persistent volume is created, it is created with the user
indicated in Resource::DiskInfo::Persistence. If missing, it would
fall back to make the user of the mesos-slave process as the owner
of the persistent volume.


Diffs (updated)
-----

  include/mesos/mesos.proto 87af4a06db8cc3889fe4d3b314206103f5ce5f2d 
  include/mesos/v1/mesos.proto 34da0a1484dc2f71262d8b97484b8edaae35bb6c 

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


Testing
-------

All tests passed.


Thanks,

Anindya Sinha