Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-11 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/#review211209 --- Ship it! Ship It! - Gastón Kleiman On Dec. 10, 2018, 3:34

Re: Review Request 69505: Added an operation status update manager to the agent.

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

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/ --- (Updated Dec. 10, 2018, 11:34 p.m.) Review request for mesos, Benno Evers and

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/ --- (Updated Dec. 10, 2018, 11:03 p.m.) Review request for mesos, Benno Evers and

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-10 Thread Greg Mann
> On Dec. 6, 2018, 12:32 a.m., Gastón Kleiman wrote: > > src/master/master.cpp > > Lines 8539-8541 (original) > > > > > > Should we replace this with something like the following? > > > > ``` > >

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/ --- (Updated Dec. 10, 2018, 11:01 p.m.) Review request for mesos, Benno Evers and

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-05 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/#review211067 --- Fix it, then Ship it! src/master/master.cpp Lines 8539-8541

Re: Review Request 69505: Added an operation status update manager to the agent.

2018-12-04 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/#review211033 --- PASS: Mesos patch 69505 was successfully built and tested.

Review Request 69505: Added an operation status update manager to the agent.

2018-12-04 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69505/ --- Review request for mesos and Gastón Kleiman. Bugs: MESOS-9278