Re: Review Request 69712: Added a CNI reboot test.

2019-01-11 Thread Deepak Goel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/#review211898 --- Ship it! Ship It! - Deepak Goel On Jan. 11, 2019, 6:49

Re: Review Request 69712: Added a CNI reboot test.

2019-01-11 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/ --- (Updated Jan. 11, 2019, 6:49 p.m.) Review request for mesos, Deepak Goel,

Re: Review Request 69712: Added a CNI reboot test.

2019-01-11 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/ --- (Updated Jan. 11, 2019, 6:09 p.m.) Review request for mesos, Deepak Goel,

Re: Review Request 69712: Added a CNI reboot test.

2019-01-11 Thread Jie Yu
> On Jan. 11, 2019, 9:38 a.m., Qian Zhang wrote: > > src/tests/containerizer/cni_isolator_tests.cpp > > Lines 2612-2613 (patched) > > > > > > `%s` is the total size, should we check the device ID instead? err,

Re: Review Request 69712: Added a CNI reboot test.

2019-01-11 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/#review211871 --- src/tests/containerizer/cni_isolator_tests.cpp Lines 2612-2613

Re: Review Request 69712: Added a CNI reboot test.

2019-01-10 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/#review211856 --- FAIL: Some of the unit tests failed. Please check the relevant

Review Request 69712: Added a CNI reboot test.

2019-01-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69712/ --- Review request for mesos, Deepak Goel, Gilbert Song, James Peach, and Qian