----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68417/#review207551 -----------------------------------------------------------
Ship it! Ship It! - Jie Yu On Aug. 17, 2018, 9:12 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68417/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2018, 9:12 p.m.) > > > Review request for mesos, Benjamin Bannier and Jie Yu. > > > Bugs: MESOS-9166 > https://issues.apache.org/jira/browse/MESOS-9166 > > > Repository: mesos > > > Description > ------- > > If a pre-existing volume is known to SLRP, i.e., the SLRP keeps a CSI > volume state checkpoint for the volume, but there is no corresponding > resource checkpoint, then this means the volume was created by a > previous SLRP instance, but the SLRP lost the state checkpoint > (typically happens when the agent ID changes). In such a case, we should > not report this volume as an unmanaged pre-existing volume. For now we > simply ingore such a volume. > > See MESOS-9167 for future work. > > > Diffs > ----- > > src/resource_provider/storage/provider.cpp > fc48072aac531bac3cbffc3ba089b8dfa2a2f200 > > > Diff: https://reviews.apache.org/r/68417/diff/1/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Chun-Hung Hsiao > >
