----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63971/#review191944 -----------------------------------------------------------
include/mesos/resource_provider/volume_profile.hpp Lines 28 (patched) <https://reviews.apache.org/r/63971/#comment269893> Currently the CSI spec is internal (under `src/`). Do we want to expose it as a public interface? include/mesos/resource_provider/volume_profile.hpp Lines 86 (patched) <https://reviews.apache.org/r/63971/#comment269892> It seems to me that the purpose of this function is that SLRP can request the capabilities/params for a given profile dynamically. The alternative is to have a function to provide a static map between profile names and capabilities/params, and SLRP calls it once during startup. The advantage of this function over the alternative is that when a profile is dynamically generated, there is no need to restart SLRP. However, when a new profile is generated, SLRP still needs to be notified so that it can report the RAW resource for the new profile. Since there is not callback mechanism for the module, we still need to restart SLRP so that it will report the RAW resource for the new profile. So I'd suggest that we have another function that returns a map from profile names to capabilities/params that would be called during SLRP startup instead. - Chun-Hung Hsiao On Nov. 27, 2017, 9:33 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63971/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2017, 9:33 p.m.) > > > Review request for mesos, Chun-Hung Hsiao and Jie Yu. > > > Bugs: MESOS-8251 > https://issues.apache.org/jira/browse/MESOS-8251 > > > Repository: mesos > > > Description > ------- > > This module is currently intended for use by the Storage Local > Resource Provider (SLRP), but may be used by other components if > those components use any of the affected Container Storage Interface > (CSI) requests. The affected calls are listed in the module's comments. > > > Diffs > ----- > > include/mesos/mesos.proto e194093e490741acc552fd3ad328fd710b4b4435 > include/mesos/resource_provider/volume_profile.hpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/63971/diff/1/ > > > Testing > ------- > > > Thanks, > > Joseph Wu > >