Review Request 65712: Added a unit test for docker daemon hangs while pulling.

2018-02-19 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65712/ --- Review request for mesos, Andrei Budnik, Greg Mann, Jie Yu, and Vinod Kone.

Review Request 65711: Addressed TODOs in the agent to set executor TERMINATING state.

2018-02-19 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65711/ --- Review request for mesos, Andrei Budnik, Greg Mann, Jie Yu, and Vinod Kone.

Re: Review Request 65689: Fixed the task update issue on framework due to docker daemon hangs.

2018-02-19 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65689/ --- (Updated Feb. 19, 2018, 11:20 p.m.) Review request for mesos, Andrei Budnik,

Re: Review Request 65710: Fixed build with CSI enabled.

2018-02-19 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65710/#review197761 --- Patch looks great! Reviews applied: [65710] Passed command:

Re: Review Request 65710: Fixed build with CSI enabled.

2018-02-19 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65710/#review197760 --- Ship it! Ship It! - James Peach On Feb. 19, 2018, 5:47

Re: Review Request 65710: Fixed build with CSI enabled.

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

Review Request 65710: Fixed build with CSI enabled.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65710/ --- Review request for mesos, Chun-Hung Hsiao and James Peach. Repository: mesos

Re: Review Request 65594: Checkpointed profiles in storage local resource provider.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65594/#review197757 --- src/resource_provider/state.proto Lines 19 (patched)

Re: Review Request 65706: Added Docker daemon to the Mesos mini container.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65706/#review197755 --- LGTM. I cannot say a lot on the systemd and docker setup files;

Re: Review Request 65707: Enabled docker containerizer in Mesos mini container.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65707/#review197756 --- Fix it, then Ship it!

Re: Review Request 65127: Windows: Enabled docker health checks.

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

Re: Review Request 65705: Fixed CLI bootstrap script to work with long workspace paths.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65705/#review197752 --- Ship it! src/python/cli_new/bootstrap Line 27 (original), 27

Re: Review Request 64211: Added options to build the Python CLI and run unit tests.

2018-02-19 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64211/#review197748 --- cmake/CompilationConfigure.cmake Lines 113-118 (patched)

Re: Review Request 65045: Tested correct operation handling during master failover.

2018-02-19 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65045/#review197751 --- Patch looks great! Reviews applied: [65671, 65672, 65044, 65045]

Re: Review Request 65045: Tested correct operation handling during master failover.

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

Re: Review Request 65045: Tested correct operation handling during master failover.

2018-02-19 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65045/ --- (Updated Feb. 19, 2018, 12:09 p.m.) Review request for mesos, Benjamin Bannier

Re: Review Request 65044: Added the v1 API 'GET_OPERATIONS' call for master and agent.

2018-02-19 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65044/ --- (Updated Feb. 19, 2018, 12:08 p.m.) Review request for mesos, Benjamin

Re: Review Request 65707: Enabled docker containerizer in Mesos mini container.

2018-02-19 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65707/#review197743 --- Patch looks great! Reviews applied: [65706, 65707] Passed