Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-04 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192799 --- FAIL: Some Mesos libprocess-tests failed. Reviews applied:

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-04 Thread John Kordich via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/ --- (Updated Dec. 4, 2017, 6:54 p.m.) Review request for mesos, Akash Gupta,

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-04 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192730 --- src/tests/health_check_tests.cpp Lines 852-853 (original),

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-01 Thread Andrew Schwartzmeyer
> On Nov. 30, 2017, 6:04 p.m., Andrew Schwartzmeyer wrote: > > src/tests/health_check_tests.cpp > > Lines 2491-2492 (original), 2491-2492 (patched) > > > > > > This seems like it would be broken on other platforms

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-01 Thread Akash Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192559 --- src/tests/health_check_tests.cpp Line 565 (original), 565

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-12-01 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192485 --- FAIL: Failed to apply the current review. Failed command:

Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests.

2017-11-30 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64104/#review192433 --- Fix it, then Ship it! src/slave/CMakeLists.txt Lines 25-28