> On Aug. 12, 2020, 3:17 a.m., Qian Zhang wrote: > > src/examples/test_csi_plugin.cpp > > Lines 174-177 (patched) > > <https://reviews.apache.org/r/72727/diff/3/?file=2237824#file2237824line174> > > > > Why do we need this flag? Without this flag, would the unit test in > > https://reviews.apache.org/r/72728 be affected?
Whoops this is no longer needed, sorry. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72727/#review221554 ----------------------------------------------------------- On Aug. 18, 2020, 1:23 a.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72727/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2020, 1:23 a.m.) > > > Review request for mesos, Andrei Budnik and Qian Zhang. > > > Bugs: MESOS-10163 > https://issues.apache.org/jira/browse/MESOS-10163 > > > Repository: mesos > > > Description > ------- > > This patch adds additional configuration flags to the > test CSI plugin which are necessary in order to test > the agent's CSI server. > > > Diffs > ----- > > src/examples/test_csi_plugin.cpp 214a3ee4481512b025ff054db6ffe15d476b0264 > > > Diff: https://reviews.apache.org/r/72727/diff/4/ > > > Testing > ------- > > These new flags are used in a subsequent test in this chain. > > > Thanks, > > Greg Mann > >
