Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-14 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/ --- (Updated Oct. 14, 2015, 7:38 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/#review102588 --- Patch looks great! Reviews applied: [39300] All tests passed. -

Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-13 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/ --- (Updated Oct. 14, 2015, 4:21 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-13 Thread Michael Park
> On Oct. 14, 2015, 12:44 a.m., Ben Mahler wrote: > > src/tests/reservation_endpoints_tests.cpp, line 187 > > > > > > Shall we put this down before stopping the driver to be a bit clearer? > > A comment there would

Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-13 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/#review102565 --- Ship it! Really appreciate the follow up, thanks Michael!

Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-13 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-3411

Re: Review Request 39300: Fixed flaky ReservationEndpointsTest.AvailableResources test.

2015-10-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39300/#review102568 --- Patch looks great! Reviews applied: [39300] All tests passed. -