Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review108151 --- Ship it! Ship It! - Till Toenshoff On Nov. 26, 2015, 10:12 a.m.

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review108145 --- Ship it! Ship It! - Bernd Mathiske On Nov. 26, 2015, 2:12 a.m.,

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review108139 --- Patch looks great! Reviews applied: [40679] Passed command: expor

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/ --- (Updated Nov. 26, 2015, 10:12 a.m.) Review request for mesos, Bernd Mathiske, M

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review108110 --- Ship it! Ship It! - haosdent huang On Nov. 26, 2015, 10:06 a.m.

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/ --- (Updated Nov. 26, 2015, 10:06 a.m.) Review request for mesos, Bernd Mathiske, M

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review108056 --- Patch looks great! Reviews applied: [40679] Passed command: expor

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-25 Thread Greg Mann
> On Nov. 25, 2015, 7:06 a.m., Timothy Chen wrote: > > src/tests/environment.cpp, line 147 > > > > > > Cfs should be CFS_ right? Currently, the tests are not prefixed with `CFS_` the way the CGROUP, etc. tests are.

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-24 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review107933 --- src/tests/environment.cpp (line 147)

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-24 Thread Greg Mann
> On Nov. 25, 2015, 1:35 a.m., haosdent huang wrote: > > src/tests/environment.cpp, line 129 > > > > > > use `/sys/fs/cgroup` maybe have some problems. centos 6 mount cgroup > > under `/cgroup` directly. Could refer

Re: Review Request 40679: Added filtering of CFS cgroup tests.

2015-11-24 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40679/#review107892 --- src/tests/environment.cpp (line 129)