Review Request 54368: Added helpers to checkpoint a 'destroy-on-recovery' file for containers.

2016-12-05 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54368/ --- Review request for mesos and Jie Yu. Bugs: MESOS-6718 https://issues.apache

Review Request 54367: Added support to destroy running DEBUG containers on agent recovery.

2016-12-05 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54367/ --- Review request for mesos and Jie Yu. Bugs: MESOS-6718 https://issues.apache

Re: Review Request 53982: Added documentation for posix/rlimit isolator.

2016-12-05 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53982/ --- (Updated Dec. 5, 2016, 10:37 a.m.) Review request for mesos, Jie Yu and Neil Co

Re: Review Request 53982: Added documentation for posix/rlimit isolator.

2016-12-05 Thread Benjamin Bannier
> On Nov. 29, 2016, 4:08 p.m., Neil Conway wrote: > > I'd still like to see more discussion of how someone might actually use > > this feature. For example, when you say "With these constraints operators > > can use a > > global hard limit to contain resource limits set by users" -- how would t

Re: Review Request 54356: Added implementation of `watch()` to the 'IOSwitchboard' isolator.

2016-12-05 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54356/ --- (Updated Dec. 5, 2016, 8:54 a.m.) Review request for mesos and Jie Yu. Change

Re: Review Request 54365: Fixed indentation of a function argument in master.cpp.

2016-12-05 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54365/ --- (Updated Dec. 5, 2016, 8:48 a.m.) Review request for mesos, Benjamin Mahler and

Review Request 54365: Fixed indentation of a function argument in master.cpp.

2016-12-05 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54365/ --- Review request for mesos, Benjamin Mahler and Guangya Liu. Repository: mesos

Re: Review Request 54291: Added os::ptsname to stout.

2016-12-05 Thread Benjamin Bannier
> On Dec. 2, 2016, 10:16 a.m., Benjamin Bannier wrote: > > 3rdparty/stout/include/stout/posix/os.hpp, line 480 > > > > > > We should be able to use a simple function-local static here and avoid > > the global leak o

<    1   2