Re: Review Request 65232: Resumed the clock if necessary when destroying test agent.

2018-01-19 Thread Greg Mann
> On Jan. 19, 2018, 5:03 p.m., Benjamin Bannier wrote: > > Thanks for this fix Greg. This will help remove some unsafe boilerplate > > from tests. We should probably follow up with a tech-debt ticket removing > > unnecessary clock resumes in tests. Created

Re: Review Request 65232: Resumed the clock if necessary when destroying test agent.

2018-01-19 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/ --- (Updated Jan. 19, 2018, 5:36 p.m.) Review request for mesos, Benjamin Bannier

Re: Review Request 65232: Resumed the clock if necessary when destroying test agent.

2018-01-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/#review195836 --- Fix it, then Ship it! Thanks for this fix Greg. This will help

Re: Review Request 65232: Resumed the clock if necessary when destroying test agent.

2018-01-18 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/#review195760 --- Ship it! Ship It! - Gaston Kleiman On Jan. 18, 2018, 4:40

Review Request 65232: Resumed the clock if necessary when destroying test agent.

2018-01-18 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65232/ --- Review request for mesos, Benjamin Bannier and Gaston Kleiman. Repository: