mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anindya Sinha <anindya_si...@apple.com>
Subject Re: Review Request 45963: Allow tasks to set persistent volume as readonly or readwrite resource.
Date Sun, 22 May 2016 07:21:20 GMT

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

(Updated May 22, 2016, 7:21 a.m.)


Review request for mesos and Jiang Yan Xu.


Changes
-------

Cosmetic changes.


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


Repository: mesos


Description
-------

Allow the task to specify the persistent volume access to be read-only
or read-write. Note that the persistent volume is always created as
read-write.
If the task is the first consumer of the shared persistent volume, then
set the ownership of the persistent volume to match that of the task.
Otherwise, allow the task to be executed only if the ownership of the
persistent volume matches that of the task.
Added an option to run the test in mixed (default) mode or shared-only
mode. In mixed mode, multiple shards alternate between shared and
unshared persistent volumes for the tasks. In shared-only mode, all
shards use shared persistent volumes for the tasks.


Diffs (updated)
-----

  src/examples/persistent_shared_volume_framework.cpp PRE-CREATION 
  src/master/validation.hpp f29f9753c75e5abc3402ed23381edcea26517ab3 
  src/master/validation.cpp f490b899758bdac9676a6f6939918efa6ac52781 
  src/slave/containerizer/mesos/isolators/filesystem/linux.hpp c99f33a77e4db5407cc26361a2f253b00e91f5b5

  src/slave/containerizer/mesos/isolators/filesystem/linux.cpp 5edbce95cc9eb4d6d22f9ab1528902cc745780af

  src/slave/containerizer/mesos/isolators/filesystem/posix.hpp c6cea98e16f2bdea2da0220c235468080bbcd17b

  src/slave/containerizer/mesos/isolators/filesystem/posix.cpp 01c0ad6dbb6d509e62e769365586b3d23dcb240d


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


Testing
-------

Tests successful.


Thanks,

Anindya Sinha


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