Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-06-01 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated June 1, 2017, 5:48 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/#review176402 --- Fix it, then Ship it! src/tests/slave_recovery_tests.cpp

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-30 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 30, 2017, 11:29 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 27, 2017, 1:32 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 26, 2017, 5:17 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-24 Thread Greg Mann
> On May 24, 2017, 1:27 a.m., Benjamin Mahler wrote: > > src/tests/slave_recovery_tests.cpp > > Lines 955 (patched) > > > > > > Rather than pausing, resuming and pausing again, have you tried leaving > > the clock

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-24 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 24, 2017, 5:54 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-23 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 24, 2017, 4:30 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-23 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/#review175888 --- Fix it, then Ship it! src/tests/slave_recovery_tests.cpp

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-23 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/ --- (Updated May 23, 2017, 11:59 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-23 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/#review175714 --- src/tests/slave_recovery_tests.cpp Lines 870 (patched)

Re: Review Request 59463: Added test for agent ping timeout during agent recovery.

2017-05-22 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59463/#review175720 --- src/tests/slave_recovery_tests.cpp Lines 891-898 (patched)