----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33865/#review83962 -----------------------------------------------------------
Ship it! src/master/validation.cpp <https://reviews.apache.org/r/33865/#comment135024> Do you want to add tests for that (and DiskInfo) in master_validation_tests.cpp (ResourceValidationTest)? src/master/validation.cpp <https://reviews.apache.org/r/33865/#comment135022> Revocable resources shouldn't be used with Persistent volume. We might want to support temp volume in the future and I think revocable resources are compatible with that. So either add a TODO here, or move the check under `if (resource.disk().has_persistence()` block. - Jie Yu On May 14, 2015, 11:57 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33865/ > ----------------------------------------------------------- > > (Updated May 14, 2015, 11:57 p.m.) > > > Review request for mesos, Jie Yu, Joris Van Remoortere, and Niklas Nielsen. > > > Bugs: MESOS-2691 > https://issues.apache.org/jira/browse/MESOS-2691 > > > Repository: mesos > > > Description > ------- > > RevocableInfo currently supports OVERSUBSCRIBED resources. In the future it > can be easily extended to use other types of revocable reosurces (e.g., > resources allocated to other roles). > > Disabled the ability to use revocable resources for reservation or > persistence because the semantics seem weird. We can enable it in the future > if there is a use case for that. > > > Diffs > ----- > > include/mesos/mesos.proto 15f55a3c54be3475e77d561f106e00ea6e53c2fa > src/common/resources.cpp 843a06d6c4d3e9ff0d1665360bae7c57bcfecb83 > src/master/validation.cpp c3e96ae0e684f3f365e9aa365bccc953d32b0452 > src/tests/resources_tests.cpp a7ec59ea217ad71f7d1e93ca6039d5b2491b3237 > > Diff: https://reviews.apache.org/r/33865/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >