> On Oct. 7, 2016, 5:20 p.m., Greg Mann wrote: > > Given how similar the code in the new example framework is to the existing > > persistent volumes framework, I think it could make sense to add shared > > volume features to the existing framework, which could be enabled/disabled > > via command-line flags. Especially since you add some nice new DESTROY > > logic in the new framework; it would be great to have that change in the > > existing one, and avoid duplicating a lot of code. What do you think? > > Anindya Sinha wrote: > Agreed. Makes sense. Will do that in the next update.
Updated with this. - Anindya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45962/#review151822 ----------------------------------------------------------- On Sept. 28, 2016, 12:15 a.m., Anindya Sinha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45962/ > ----------------------------------------------------------- > > (Updated Sept. 28, 2016, 12:15 a.m.) > > > Review request for mesos and Jiang Yan Xu. > > > Bugs: MESOS-4431 > https://issues.apache.org/jira/browse/MESOS-4431 > > > Repository: mesos > > > Description > ------- > > Added a persistent volume test framework for shared volumes. > > > Diffs > ----- > > src/Makefile.am fba488f9d676851dd046a8b8c7dd175b3c0d9ef0 > src/examples/persistent_shared_volume_framework.cpp PRE-CREATION > src/tests/examples_tests.cpp 52fac33733cc86dd718c7836c5031999f1597a5c > src/tests/persistent_shared_volume_framework_test.sh PRE-CREATION > > Diff: https://reviews.apache.org/r/45962/diff/ > > > Testing > ------- > > New test framework for shared resources added. > Tests successful. > > > Thanks, > > Anindya Sinha > >