Re: Review Request 54074: Added isolator nested aware user doc.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54074/ --- (Updated Nov. 30, 2016, 11:11 a.m.) Review request for mesos, Benjamin Mahler,

Re: Review Request 54013: Added user doc for nested container and task group.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54013/ --- (Updated Nov. 30, 2016, 11:11 a.m.) Review request for mesos, Benjamin Mahler,

Re: Review Request 54216: Fixed unsufficient root privileges check by geteuid().

2016-11-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54216/#review157440 --- Bad patch! Reviews applied: [54216, 50871, 54215, 54214, 54213,

Re: Review Request 53974: Added support to handle ATTACH_CONTAINER_OUPUT in the io switchbaord.

2016-11-30 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53974/ --- (Updated Nov. 30, 2016, 6:56 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 53939: Added implementation for containerizer 'attach()' call.

2016-11-30 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53939/ --- (Updated Nov. 30, 2016, 6:55 p.m.) Review request for mesos, Benjamin Hindman,

Review Request 54211: Fixed overlay backend symlink error message.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54211/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao Li.

Review Request 54213: Supported more layers through symlink for aufs backend.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54213/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao Li.

Re: Review Request 50871: Supported auto backend in Unified Containerizer.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50871/ --- (Updated Nov. 30, 2016, 10:55 a.m.) Review request for mesos, Artem

Review Request 54216: Fixed unsufficient root privileges check by geteuid().

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54216/ --- Review request for mesos, Artem Harutyunyan, James DeFelice, and Jie Yu. Bugs:

Re: Review Request 54148: Updated the io switchboard to launch an external io switchboard server.

2016-11-30 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54148/ --- (Updated Nov. 30, 2016, 6:54 p.m.) Review request for mesos and Jie Yu.

Review Request 54215: Fixed duplicate image layer ids returned by docker store.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54215/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao Li.

Review Request 54214: Added unit test for aufs backend supporting many layers.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54214/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao Li.

Review Request 54212: Fixed overlay backend provisioning multi images symlink.

2016-11-30 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54212/ --- Review request for mesos, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao Li.

Re: Review Request 54147: Added a server side component for the IOSwitchboard.

2016-11-30 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54147/ --- (Updated Nov. 30, 2016, 6:53 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 53803: Added a new libprocess HTTP test.

2016-11-30 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53803/ --- (Updated Nov. 30, 2016, 5:57 p.m.) Review request for mesos, Benjamin Mahler

Re: Review Request 52645: Harden Mesos

2016-11-30 Thread Aaron Wood
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/ --- (Updated Nov. 30, 2016, 5:19 p.m.) Review request for mesos, James Peach,

Re: Review Request 52695: Harden libprocess

2016-11-30 Thread Aaron Wood
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52695/ --- (Updated Nov. 30, 2016, 5:12 p.m.) Review request for mesos, James Peach,

Re: Review Request 52696: Harden stout

2016-11-30 Thread Aaron Wood
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52696/ --- (Updated Nov. 30, 2016, 5:10 p.m.) Review request for mesos, James Peach,

Re: Review Request 54035: Extended test coverage of posix/rlimits isolator.

2016-11-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54035/#review157420 --- Patch looks great! Reviews applied: [54034, 54035] Passed

Re: Review Request 54179: Updated master to use BoundedHashMap.

2016-11-30 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54179/#review157412 --- src/master/master.hpp (line 2587)

Re: Review Request 54035: Extended test coverage of posix/rlimits isolator.

2016-11-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54035/ --- (Updated Nov. 30, 2016, 4:16 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 54035: Extended test coverage of posix/rlimits isolator.

2016-11-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54035/#review157402 --- Fix it, then Ship it!

Re: Review Request 54034: Introduced common fixture to PosixRLimitsIsolatorTest.

2016-11-30 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54034/#review157406 --- Ship it! Ship It! - Till Toenshoff On Nov. 23, 2016, 4:35

Re: Review Request 53299: Fixed memory leak in implementation of Future::after().

2016-11-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53299/#review157404 --- Ship it! - Benjamin Bannier On Oct. 31, 2016, 3:48 p.m.,

Review Request 54205: Improved equality check in SlaveTest.StateEndpoint.

2016-11-30 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54205/ --- Review request for mesos, Joseph Wu and Till Toenshoff. Bugs: MESOS-4695

Re: Review Request 54200: Updated H-DRF allocator to handle multi-role framework.

2016-11-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54200/#review157399 --- Patch looks great! Reviews applied: [54061, 54062, 54197, 54198,

Re: Review Request 53541: Added authorization actions for Nested Container and Debug API.

2016-11-30 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53541/ --- (Updated Nov. 30, 2016, 11:48 a.m.) Review request for mesos, Adam B, Kapil

Re: Review Request 54196: Added API handler for LAUNCH_NESTED_CONTAINER_SESSION.

2016-11-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54196/#review157391 --- Patch looks great! Reviews applied: [54193, 54194, 54196]

Review Request 54197: Introduced Resource.AllocationInfo.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54197/ --- Review request for mesos, Guangya Liu and Qiang Zhang. Bugs: MESOS-6633

Re: Review Request 54062: Added logic to validate FrameworkInfo roles.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54062/ --- (Updated Nov. 30, 2016, 9:57 a.m.) Review request for mesos, Benjamin Mahler,

Re: Review Request 54061: Augmented FrameworkInfo to support multiple roles.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54061/ --- (Updated Nov. 30, 2016, 9:58 a.m.) Review request for mesos, Benjamin Mahler,

Review Request 54200: Updated H-DRF allocator to handle multi-role framework.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54200/ --- Review request for mesos, Guangya Liu and Qiang Zhang. Bugs: MESOS-6635

Review Request 54199: Minor style fix.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54199/ --- Review request for mesos and Guangya Liu. Repository: mesos Description

Re: Review Request 53541: Added authorization actions for Nested Container and Debug API.

2016-11-30 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53541/#review157387 --- Fix it, then Ship it! Looks great! Just a few remaining nits.

Review Request 54198: Added Resource.Allocation in Offer.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54198/ --- Review request for mesos, Guangya Liu and Qian Zhang. Bugs: MESOS-6634

Re: Review Request 51879: Autodetect value of resource when not specified in static resources.

2016-11-30 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51879/#review157057 --- include/mesos/resources.hpp (lines 138 - 148)

<    1   2