Re: Review Request 37172: Maintenance Primitives: Set offer `unavailability` if slave is scheduled for maintenance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37172/ --- (Updated Sept. 14, 2015, 12:25 p.m.) Review request for mesos, Benjamin

Re: Review Request 37173: Maintenance Primitives: Added unavailability to Allocator's Slave struct.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37173/ --- (Updated Sept. 14, 2015, 12:40 p.m.) Review request for mesos, Benjamin

Re: Review Request 37180: Maintenance Primitives: Implemented Master::inverseOffer.

> On Aug. 31, 2015, 7:49 a.m., Jian Qiu wrote: > > src/master/master.cpp, line 4904 > > > > > > If the allocator calls inverse callback followed by an offer callback, > > will the two messages be handled in two

Re: Review Request 37505: Fix broken health check in docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37505/ --- (Updated Sept. 14, 2015, 12:59 p.m.) Review request for mesos and Timothy

Re: Review Request 38287: Check if the futrue is failed before dispatch in freeze()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38287/#review98840 --- Patch looks great! Reviews applied: [38287] All tests passed. -

Re: Review Request 37177: Maintenance Primitives: Added inverse offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37177/ --- (Updated Sept. 14, 2015, 12:51 p.m.) Review request for mesos, Benjamin

Re: Review Request 37622: Maintenance Primitives: Shutdown & remove slave when maintenance is started.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37622/ --- (Updated Sept. 14, 2015, 1:08 p.m.) Review request for mesos, Benjamin

Re: Review Request 37175: Maintenance Primitives: Added updateUnavailability to master.

> On Sept. 14, 2015, 12:37 a.m., Qian Zhang wrote: > > src/master/master.cpp, lines 4131-4134 > > > > > > I think we have done this in Master::Http::maintenanceSchedule(), so > > why we do it again here? Maybe

Re: Review Request 37175: Maintenance Primitives: Added updateUnavailability to master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37175/ --- (Updated Sept. 14, 2015, 12:46 p.m.) Review request for mesos, Benjamin

Re: Review Request 37282: Maintenance Primitives: Added InverseOffer to V1 API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37282/ --- (Updated Sept. 14, 2015, 12:54 p.m.) Review request for mesos, Benjamin

Re: Review Request 37827: Added a test for converting JSON arrays to repeated protobufs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37827/ --- (Updated Sept. 14, 2015, 10:52 a.m.) Review request for mesos, haosdent huang,

Re: Review Request 37170: Maintenance Primitives: Added `Machine` to Slave struct in Master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37170/ --- (Updated Sept. 14, 2015, 12:11 p.m.) Review request for mesos, Benjamin

Re: Review Request 37280: Maintenance Primitives: Added updateInverseOffer to Allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37280/ --- (Updated Sept. 14, 2015, 12:57 p.m.) Review request for mesos, Benjamin

Re: Review Request 37622: Maintenance Primitives: Shutdown & remove slave when maintenance is started.

> On Sept. 14, 2015, 1:53 a.m., Qian Zhang wrote: > > src/master/http.cpp, line 1606 > > > > > > s/machineInfos/machines Nice catch! Will be fixed in updated review. - Joris

Re: Review Request 37284: Maintenance Primitives: Added support for Accept / Decline of InverseOffers in master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37284/ --- (Updated Sept. 14, 2015, 1:03 p.m.) Review request for mesos, Benjamin

Re: Review Request 37168: MESOS-3063 (Add an example framework using dynamic reservation)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review98818 --- src/Makefile.am (line 1463)

Re: Review Request 37168: MESOS-3063 (Add an example framework using dynamic reservation)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/ --- (Updated Sept. 14, 2015, 1:48 p.m.) Review request for mesos and Michael Park.

Re: Review Request 37622: Maintenance Primitives: Shutdown & remove slave when maintenance is started.

> On Sept. 14, 2015, 3:36 a.m., Benjamin Hindman wrote: > > src/master/http.cpp, lines 1622-1624 > > > > > > Why don't we want to send lost slave messages? > > Benjamin Hindman wrote: > Sorry, I meant: why

Re: Review Request 38279: [MESOS-3366] Allow resources/attributes discovery

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38279/ --- (Updated Sept. 14, 2015, 10:38 a.m.) Review request for mesos, Connor Doyle

Re: Review Request 38279: Enabled resources/attributes discovery

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38279/ --- (Updated Sept. 14, 2015, 10:39 a.m.) Review request for mesos, Connor Doyle

Re: Review Request 38253: Add containerId to ResourceUsage to enable QoS controller to target a container

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38253/ --- (Updated Sept. 14, 2015, 10:41 a.m.) Review request for mesos and Niklas

Re: Review Request 38348: Correct typo in http.cpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38348/#review98870 --- That's not a typo; "iff" is short for "if and only if". - Joseph

Re: Review Request 38289: Handle bad request in Docker registry client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38289/#review98872 --- Ship it! Ship It! - Jojy Varghese On Sept. 11, 2015, 7:34 p.m.,

Re: Review Request 38364: Minor refactor for MesosContainerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38364/ --- (Updated Sept. 14, 2015, 5:54 p.m.) Review request for mesos, Connor Doyle,

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/#review98926 --- include/mesos/mesos.proto (line 1163)

Review Request 38378: Minor cleanup in perf_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38378/ --- Review request for mesos and Jie Yu. Repository: mesos Description ---

Review Request 38382: Fixed the perf event isolator to continue sampling in the presence of failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38382/ --- Review request for mesos and Jie Yu. Bugs: MESOS-3423

Review Request 38375: Fixed process::collect and process::await to do discard propagation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38375/ --- Review request for mesos and Jie Yu. Bugs: MESOS-3426

Review Request 38380: Minor cleanups in the perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38380/ --- Review request for mesos and Jie Yu. Repository: mesos Description ---

Re: Review Request 38374: Added logics in LinuxFilesystemIsolator to recover and cleanup orphans.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38374/#review98942 --- Ship it! We should probably have a test case (or some checks in

Re: Review Request 38348: Correct typo in http.cpp

> On 九月 14, 2015, 5:27 p.m., Joseph Wu wrote: > > That's not a typo; "iff" is short for "if and only if". Thanks Joseph for pointing this oout. It is difficult for non native english speakers understand this, what about updating "iff" to "if and only if"? - Guangya

Review Request 38364: Minor refactor for MesosContainerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38364/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. Bugs:

Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. Bugs:

Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/ --- Review request for mesos, Connor Doyle, Jie Yu, Niklas Nielsen, and Vinod Kone.

Re: Review Request 38366: Added helper to model Labels message for state.json.

> On Sept. 14, 2015, 5:40 p.m., Niklas Nielsen wrote: > > High level comment; would it be worth change JSON::Protobuf() to support > > Labels instead? Probably not. We are trying a shortcut when modelling Labels for state.json. Thus, I wouldn't want to change JSON::Protobuf which is supposed

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/#review98936 --- src/slave/slave.cpp (line 2760)

Re: Review Request 38382: Fixed the perf event isolator to continue sampling in the presence of failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38382/#review98959 --- src/slave/containerizer/isolators/cgroups/perf_event.cpp (line

Re: Review Request 37283: Maintenance Primitives: Refactored Master maintenance test to use V1 API.

> On Sept. 14, 2015, 4:51 p.m., Qian Zhang wrote: > > src/tests/master_maintenance_tests.cpp, line 418 > > > > > > I checked v1/mesos.proto, and found regular offer also has > > unavailability: > > ``` > >

Review Request 38386: Fix 'operator=' spacing in the registry_client.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38386/ --- Review request for mesos, Joris Van Remoortere, Joseph Wu, and Timothy Chen.

Re: Review Request 37585: Maintenance primitives: Add a user doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37585/ --- (Updated Sept. 14, 2015, 2:32 p.m.) Review request for mesos, Benjamin

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/#review98932 --- include/mesos/mesos.proto (lines 1163 - 1164)

Review Request 38374: Added logics in LinuxFilesystemIsolator to recover and cleanup orphans.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38374/ --- Review request for mesos, haosdent huang, Timothy Chen, Vinod Kone, and Jiang

Re: Review Request 38378: Minor cleanup in perf_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38378/ --- (Updated Sept. 14, 2015, 11:08 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 38364: Minor refactor for MesosContainerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38364/#review98938 --- Ship it! src/slave/containerizer/mesos/containerizer.cpp (lines

Re: Review Request 38379: Handle empty set of cgroups as a no-op in perf::sample.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38379/#review98945 --- Not sure how much value this patch has, because upper layer always

Re: Review Request 38378: Minor cleanup in perf_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38378/#review98947 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 11:08 p.m., Ben

Re: Review Request 38377: Removed 'using namespace process' in perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38377/#review98946 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 37024: Exposes mesos version information in components.

> On Aug. 31, 2015, 10:47 p.m., Ben Mahler wrote: > > Couple of comments: > > > > Let's not call the actor VersionInfo, since that sounds like a protobuf > > message. How about 'VersionProcess' and we avoid the wrapper type entirely > > since Version is taken in stout? > > > > Also, can we

Re: Review Request 37283: Maintenance Primitives: Refactored Master maintenance test to use V1 API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37283/#review98964 --- src/tests/master_maintenance_tests.cpp (line 405)

Re: Review Request 38376: Fix 'operator=' spacing in the V1 Scheduler interface.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38376/#review98963 --- Ship it! Ship It! - Guangya Liu On 九月 14, 2015, 10:49 p.m.,

Re: Review Request 38137: Added Docker provisioner and local store

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38137/#review98965 --- src/slave/containerizer/provisioners/docker/paths.cpp (line 35)

Re: Review Request 37585: Maintenance primitives: Add a user doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37585/ --- (Updated Sept. 14, 2015, 2:32 p.m.) Review request for mesos, Benjamin

Re: Review Request 38366: Added helper to model Labels message for state.json.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38366/#review98922 --- High level comment; would it be worth change JSON::Protobuf() to

Re: Review Request 38364: Minor refactor for MesosContainerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38364/#review98921 --- Can you expand on the change (before and after) effect in the RR

Review Request 38376: Fix 'operator=' spacing in the V1 Scheduler interface.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38376/ --- Review request for mesos and Joris Van Remoortere. Repository: mesos

Re: Review Request 38375: Fixed process::collect and process::await to do discard propagation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38375/#review98944 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 38380: Minor cleanups in the perf event isolator.

> On Sept. 14, 2015, 11:48 p.m., Cong Wang wrote: > > src/slave/containerizer/isolators/cgroups/perf_event.cpp, line 374 > > > > > > I don't think cgroup can be destroyed while perf running, kernel should > >

Re: Review Request 38382: Fixed the perf event isolator to continue sampling in the presence of failures.

> On Sept. 14, 2015, 11:38 p.m., Cong Wang wrote: > > src/slave/containerizer/isolators/cgroups/perf_event.cpp, line 413 > > > > > > s/sampling will be halted// ? Great catch, thank you! - Ben

Re: Review Request 38289: Handle bad request in Docker registry client.

> On Sept. 14, 2015, 6:11 p.m., Anand Mazumdar wrote: > > src/slave/containerizer/provisioners/docker/registry_client.cpp, line 277 > > > > > > Why are we parsing the error JSON to extract the error string from JSON

Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/#review98934 --- src/tests/containerizer/isolator_tests.cpp (line 202)

Re: Review Request 38102: MESOS-3046 (Stout's UUID re-seeds a new random generator during each call to UUID::random)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38102/#review98950 --- 3rdparty/libprocess/3rdparty/stout/include/stout/uuid.hpp (line

Re: Review Request 38381: Removed hard-coded reap interval from perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38381/#review98952 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 38379: Handle empty set of cgroups as a no-op in perf::sample.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38379/#review98949 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 38380: Minor cleanups in the perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38380/#review98951 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 38333: Made container sandbox a shared mount to address MESOS-3349.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38333/ --- (Updated Sept. 14, 2015, 10:29 p.m.) Review request for mesos, haosdent huang,

Re: Review Request 38378: Minor cleanup in perf_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38378/#review98939 --- src/tests/containerizer/perf_tests.cpp (line 51)

Re: Review Request 37168: MESOS-3063 (Add an example framework using dynamic reservation)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review98943 --- Patch looks great! Reviews applied: [37168] All tests passed. -

Re: Review Request 37969: Maintenance primitives: Tweak validation error messages to return JSON rather than protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37969/ --- (Updated Sept. 14, 2015, 5:10 p.m.) Review request for mesos, Benjamin

Review Request 38366: Added helper to model Labels message for state.json.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38366/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen.

Review Request 38368: Replaced slaveTaskStatusLabelDecorator hook with slaveTaskStatusDecorator hook.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38368/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. Bugs:

Review Request 38365: Updated Isolator::prepare to return list of required namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38365/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. Bugs:

Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38370/#review98917 --- src/docker/executor.cpp (line 162)

Re: Review Request 38368: Replaced slaveTaskStatusLabelDecorator hook with slaveTaskStatusDecorator hook.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38368/#review98928 --- src/examples/test_hook_module.cpp (line 197)

Re: Review Request 38365: Updated Isolator::prepare to return list of required namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38365/#review98937 --- Ship it! include/mesos/slave/isolator.proto (line 75)

Review Request 38381: Removed hard-coded reap interval from perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38381/ --- Review request for mesos and Jie Yu. Bugs: MESOS-3423

Review Request 38377: Removed 'using namespace process' in perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38377/ --- Review request for mesos and Jie Yu. Repository: mesos Description ---

Review Request 38379: Handle empty set of cgroups as a no-op in perf::sample.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38379/ --- Review request for mesos and Jie Yu. Repository: mesos Description ---

Re: Review Request 38379: Handle empty set of cgroups as a no-op in perf::sample.

> On Sept. 14, 2015, 11:21 p.m., Cong Wang wrote: > > Not sure how much value this patch has, because upper layer always calls it > > with non-empty cgroups. Also, you probably want to check for events.empty() > > too while you are on it. This was done for https://reviews.apache.org/r/38380/,

Re: Review Request 38382: Fixed the perf event isolator to continue sampling in the presence of failures.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38382/#review98956 --- Ship it! Ship It! - Jie Yu On Sept. 14, 2015, 10:57 p.m., Ben

Re: Review Request 38374: Added logics in LinuxFilesystemIsolator to recover and cleanup orphans.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38374/#review98958 --- Ship it! Ship It! - Timothy Chen On Sept. 14, 2015, 10:30 p.m.,

Re: Review Request 38381: Removed hard-coded reap interval from perf event isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38381/#review98957 --- Ship it! Ship It! - Guangya Liu On 九月 14, 2015, 10:57 p.m., Ben

Re: Review Request 38376: Fix 'operator=' spacing in the V1 Scheduler interface.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38376/#review98969 --- Ship it! Ship It! - Anand Mazumdar On Sept. 14, 2015, 10:49

Re: Review Request 37540: Add perf event API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37540/#review98924 --- my main question is regarding division of responsibilities between

Re: Review Request 38386: Fix 'operator=' spacing in the registry_client.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38386/#review98973 --- Ship it! Ship It! - Joseph Wu On Sept. 14, 2015, 5:14 p.m.,

Re: Review Request 38348: Correct typo in http.cpp

> On Sept. 14, 2015, 10:27 a.m., Joseph Wu wrote: > > That's not a typo; "iff" is short for "if and only if". > > Guangya Liu wrote: > Thanks Joseph for pointing this oout. It is difficult for non native > english speakers understand this, what about updating "iff" to "if and only > if"?

Re: Review Request 38172: Stout: Simplified hashset implementation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38172/#review98974 --- Just a drive by on comment wrapping changes.

Re: Review Request 38259: [MESOS-3340] Command-line flags should take precedence over OS Env variables

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38259/#review98983 --- LGTM Can you please add a test with a few vars defined in Env and

Re: Review Request 38259: [MESOS-3340] Command-line flags should take precedence over OS Env variables

> On Sept. 15, 2015, 4:54 a.m., Marco Massenzio wrote: > > LGTM > > Can you please add a test with a few vars defined in Env and then in Cmd > > Line too, and verify that it "works as intended"? > > > > Thanks for bearing with me and changing your code. > > I think it looks clean and nice now

Re: Review Request 38279: Enabled resources/attributes discovery

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38279/#review98879 --- Per Connors comment, we need a test to exercise the new code

Re: Review Request 38031: [3/5] Integer Precision for JSON <-> Protobuf conversions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38031/ --- (Updated Sept. 14, 2015, 1:37 p.m.) Review request for mesos, Ben Mahler,

Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38370/#review98923 --- +1 on Tim's comment. I think you can just generalize the text to

Re: Review Request 38287: Check if the futrue is failed before dispatch in freeze()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38287/ --- (Updated 九月 14, 2015, 7:28 a.m.) Review request for mesos, Ian Downes, Jie Yu,

Re: Review Request 38343: mesos: Fixed punctuation in log message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38343/#review98878 --- Ship it! Ship It! - Niklas Nielsen On Sept. 13, 2015, 10:09

Re: Review Request 37505: Fix broken health check in docker executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37505/#review98892 --- Patch looks great! Reviews applied: [37505] All tests passed. -

Re: Review Request 38306: Libprocess: Removed namespace pollution from ssl gtest.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38306/#review98893 --- Ship it! 3rdparty/libprocess/include/process/ssl/gtest.hpp (lines

Re: Review Request 38333: Made container sandbox a shared mount to address MESOS-3349.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38333/#review98863 --- Ship it! src/slave/containerizer/isolators/filesystem/linux.cpp

Re: Review Request 38279: Enabled resources/attributes discovery

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38279/#review98884 --- src/examples/test_hook_module.cpp (line 208)

Re: Review Request 38279: Enabled resources/attributes discovery

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38279/#review98886 --- src/hook/manager.cpp (line 261)

Re: Review Request 38289: Handle bad request in Docker registry client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38289/#review98876 --- LGTM. Just a minor query around why do we want to parse the error

Re: Review Request 38253: Add containerId to ResourceUsage to enable QoS controller to target a container

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38253/#review98880 --- include/mesos/mesos.proto (line 820)

  1   2   >