Re: Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58280/#review171396 --- Patch looks great! Reviews applied: [58278, 58279, 58280]

Re: Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58280/#review171395 --- Ship it! Ship It! - Gilbert Song On April 7, 2017, 5:27

Re: Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58280/ --- (Updated April 8, 2017, 12:27 a.m.) Review request for mesos, Gilbert Song,

Re: Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Jie Yu
> On April 8, 2017, 12:17 a.m., Zhitao Li wrote: > > Can you confirm that this test fails without the fix in r/58278? Updated the "test" section. - Jie --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58280/#review171389 --- Can you confirm that this test fails without the fix in r/58278?

Review Request 58280: Added a test to verify persistent volume mount points removal.

2017-04-07 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58280/ --- Review request for mesos, Gilbert Song, Jason Lai, and Zhitao Li. Bugs: