Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-22 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review198157 --- Patch looks great! Reviews applied: [65393, 65394, 65395, 65396,

Re: Review Request 65728: Add test to check quotes in docker executor health check.

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

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review198095 --- Ship it! Ship It! - Andrew Schwartzmeyer On Feb. 21, 2018,

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review198094 --- Patch looks great! Reviews applied: [65393, 65394, 65395, 65396,

Re: Review Request 65728: Add test to check quotes in docker executor health check.

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

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Akash Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/ --- (Updated Feb. 22, 2018, 1:51 a.m.) Review request for mesos, Alexander

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review198011 --- src/tests/health_check_tests.cpp Lines 2716 (patched)

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review197910 --- FAIL: Failed to apply the dependent review: 65615. Failed

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review197907 --- FAIL: Failed to get dependent review IDs for the current patch.

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review197899 --- FAIL: Failed to apply the dependent review: 65615. Failed

Re: Review Request 65728: Add test to check quotes in docker executor health check.

2018-02-21 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65728/#review197871 --- Bad patch! Reviews applied: [65728, 64604, 64387, 65615, 65127,