----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72805/#review221713 -----------------------------------------------------------
Fix it, then Ship it! src/tests/mesos.hpp Lines 873-874 (patched) <https://reviews.apache.org/r/72805/#comment310764> A newline between. src/tests/mesos.hpp Lines 2097 (patched) <https://reviews.apache.org/r/72805/#comment310765> Do we need another `createVolumeHostPath` in the `internal` namespace at line 1804? - Qian Zhang On Aug. 26, 2020, 1:50 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72805/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2020, 1:50 p.m.) > > > Review request for mesos and Qian Zhang. > > > Repository: mesos > > > Description > ------- > > Added a test helper for CSI volumes. > > > Diffs > ----- > > src/tests/mesos.hpp 8f89d7ca5642a475ecbc176d8bba277a6774a8a1 > > > Diff: https://reviews.apache.org/r/72805/diff/1/ > > > Testing > ------- > > Testing details at the end of this chain. > > > Thanks, > > Greg Mann > >
