---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/
---
(Updated Oct. 14, 2014, 1:31 p.m.)
Review request for mesos, Jie Yu and Vinod
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review56567
---
src/linux/cgroups.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/
---
(Updated Oct. 2, 2014, 11:29 a.m.)
Review request for mesos, Jie Yu and Vinod
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54553
---
Ship it!
src/linux/cgroups.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54643
---
Ship it!
lgtm modulo comments.
- Vinod Kone
On Sept. 23, 2014,
On Sept. 22, 2014, 12:12 p.m., Ben Mahler wrote:
Why do you need this?
Seems like one would only watch for cgroup emptiness when destroying a
cgroup, in which case, why split emptiness waiting from a successful
destroy?
This is for use in conjunction with pid namespaces: we kill
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/
---
(Updated Sept. 23, 2014, 4:39 p.m.)
Review request for mesos, Jie Yu and Vinod
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54362
---
Ship it!
Ship It!
- Timothy Chen
On Sept. 23, 2014, 11:39 p.m.,
On Sept. 22, 2014, 7:12 p.m., Ben Mahler wrote:
Why do you need this?
Seems like one would only watch for cgroup emptiness when destroying a
cgroup, in which case, why split emptiness waiting from a successful
destroy?
Ian Downes wrote:
This is for use in conjunction with
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/
---
Review request for mesos, Jie Yu and Vinod Kone.
Repository: mesos-git
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54166
---
src/linux/cgroups.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54170
---
Why do you need this?
Seems like one would only watch for cgroup
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25864/#review54211
---
Patch looks great!
Reviews applied: [25864]
All tests passed.
-
13 matches
Mail list logo