-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64612/
-----------------------------------------------------------
(Updated Dec. 18, 2017, 6:35 p.m.)
Review request for mesos, Chun-Hung Hsiao and Jie Yu.
Changes
-------
Changed the injection method for the module. This involves much fewer
interface changes, but is also unlike most of our other patterns.
Bugs: MESOS-8251
https://issues.apache.org/jira/browse/MESOS-8251
Repository: mesos
Description (updated)
-------
This adds an agent flag to choose the VolumeProfileAdaptor module
to load and passes this information down to the storage resource
providers. A single module can potentially be used by multiple
storage resource providers.
We choose to pass the module along as a global variable because the
module is not needed by many of the components in the injection path.
Specifically, the Storage Local Resource Provider will need the module,
but its parents, the Local Resource Provider and Resource Provider
Daemons will not.
Diffs (updated)
-----
include/mesos/resource_provider/volume_profile.hpp PRE-CREATION
src/resource_provider/storage/provider.cpp
158b6b408002209aa9a79a6772da30c984aad61a
src/resource_provider/volume_profile.cpp PRE-CREATION
src/slave/flags.hpp 5918484825c3c813883c42465f3e5a5693d6a2cd
src/slave/flags.cpp d4b95b7542733b22299bc19235cf99220534c022
src/slave/slave.hpp 75cc5836e39b2413eaefc5a5c9e905134db83743
src/slave/slave.cpp e69d42a2bfae09e2defcd8333c0434a9fd8bac4c
Diff: https://reviews.apache.org/r/64612/diff/2/
Changes: https://reviews.apache.org/r/64612/diff/1-2/
Testing
-------
make check
Basically just a plumbing change to pass the module into the SLRP.
Thanks,
Joseph Wu
