mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Greg Mann" <>
Subject Re: Review Request 40255: [5/7] Added framework authorization for persistent volumes.
Date Sat, 14 Nov 2015 00:25:16 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 14, 2015, 12:25 a.m.)

Review request for mesos, Jie Yu, Michael Park, and Neil Conway.



Bugs: MESOS-3065

Repository: mesos


Added framework authorization for persistent volumes.

Diffs (updated)

  src/master/master.cpp 7bac0fea4bcd040307fdfdcd002387d5baee46d1 
  src/tests/persistent_volume_tests.cpp 8b791ac0861171b0c655307e965165d9ad7ba966 



This is the fifth in a chain of 7 patches. New tests were added to `persistent_volume_tests.cpp`
in order to test a framework attempting both successful and failed authorizations for `CREATE`
and `DESTROY` offer operations. `make check` was used to test after all patches were applied.

Note that this chain of patches touches many of the same files as another chain beginning
with Review #39985 and ending with Review #39989, which is currently in review as well. To
avoid conflicts, the beginning of this chain begins on top of Review #39989.


Greg Mann

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