Upcoming Renaming of Memory Statistics from /monitor/statistics.json (MESOS-2104)

2015-06-02 Thread Chi Zhang
If you are not using mem_rss_bytes, mem_file_bytes and mem_anon_bytes from /monitor/statistics.json prior to 0.23.0, you can stop reading. Hi Community, I would like to send out an update from MESOS-2014: Targeted for 0.23.0: mem_total_bytes will be introduced to replace mem_rss_bytes.

Re: Review Request 33329: Removed unnecessary freeaddrinfo in getIP if getaddrinfo returns error.

2015-04-21 Thread Chi Zhang
, visit: https://reviews.apache.org/r/33329/#review80984 --- On April 20, 2015, 7:16 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 33329: Removed unnecessary freeaddrinfo in getIP if getaddrinfo returns error.

2015-04-20 Thread Chi Zhang
3622165af064a1f0a9d461af0e1d0d88a352d4a8 Diff: https://reviews.apache.org/r/33329/diff/ Testing --- Thanks, Chi Zhang

Review Request 33329: Removed unnecessary freeaddrinfo in getIP if getaddrinfo returns error.

2015-04-17 Thread Chi Zhang
: https://reviews.apache.org/r/33329/diff/ Testing --- Thanks, Chi Zhang

Review Request 32955: Simplified ROOT_CGROUPS_Listen test.

2015-04-07 Thread Chi Zhang
://issues.apache.org/jira/browse/mesos-2573 Repository: mesos Description --- Simplified ROOT_CGROUPS_Listen test. Diffs - src/tests/cgroups_tests.cpp e18aed1feca182da89a117f81bed0897a00fb0ef Diff: https://reviews.apache.org/r/32955/diff/ Testing --- Thanks, Chi Zhang

Review Request 32956: Simplified MemUsage test.

2015-04-07 Thread Chi Zhang
://issues.apache.org/jira/browse/mesos-2573 Repository: mesos Description --- Simplified MemUsage test. Diffs - src/tests/isolator_tests.cpp 535e9afd942a2be507f69bf8bd9bb32f41a85a3a Diff: https://reviews.apache.org/r/32956/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-02 Thread Chi Zhang
/network/port_mapping.cpp e691d463515084518c94cdec3fbdf37be4a72977 src/tests/port_mapping_tests.cpp f4124c3e880e043729579a829e1057727741d131 Diff: https://reviews.apache.org/r/32744/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31012: [1/5] cgroups: added tests for memory statistics.

2015-04-01 Thread Chi Zhang
- src/tests/cgroups_tests.cpp e18aed1feca182da89a117f81bed0897a00fb0ef Diff: https://reviews.apache.org/r/31012/diff/ Testing --- Thanks, Chi Zhang

Review Request 32756: [4/5] Added a memory statistics test for active anonymous memory.

2015-04-01 Thread Chi Zhang
11712d7f378d9426f160d53b0387c698a28a4207 src/tests/memory_test_helper.cpp cdf769b32036c746ec4aa90841ea45c9a4159b51 Diff: https://reviews.apache.org/r/32756/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31012: [1/5] Added memory statisics test fixure and a test for RSS.

2015-04-01 Thread Chi Zhang
- src/tests/cgroups_tests.cpp e18aed1feca182da89a117f81bed0897a00fb0ef Diff: https://reviews.apache.org/r/31012/diff/ Testing --- Thanks, Chi Zhang

Review Request 32754: [2/5] Added a memory statistics test for page cache.

2015-04-01 Thread Chi Zhang
a memory statistics test for page cache. Diffs - src/tests/cgroups_tests.cpp e18aed1feca182da89a117f81bed0897a00fb0ef Diff: https://reviews.apache.org/r/32754/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32754: [2/5] Added a memory statistics test for page cache.

2015-04-01 Thread Chi Zhang
/diff/ Testing --- Thanks, Chi Zhang

Review Request 32757: [5/5] Added a memory statistics test for writeback.

2015-04-01 Thread Chi Zhang
/tests/memory_test_helper.cpp cdf769b32036c746ec4aa90841ea45c9a4159b51 Diff: https://reviews.apache.org/r/32757/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31012: cgroups: added tests for memory statistics.

2015-04-01 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31012/#review77678 --- On April 1, 2015, 11:09 p.m., Chi Zhang wrote

Re: Review Request 31012: cgroups: added tests for memory statistics.

2015-04-01 Thread Chi Zhang
://reviews.apache.org/r/31012/diff/ Testing --- Thanks, Chi Zhang

Review Request 32755: [3/5] Added a memory statistics test for memory-mapped file.

2015-04-01 Thread Chi Zhang
11712d7f378d9426f160d53b0387c698a28a4207 src/tests/memory_test_helper.cpp cdf769b32036c746ec4aa90841ea45c9a4159b51 Diff: https://reviews.apache.org/r/32755/diff/ Testing --- Thanks, Chi Zhang

Review Request 32728: Fixed indentation in mesos.proto.

2015-04-01 Thread Chi Zhang
--- Fixed indentation in mesos.proto. Diffs - include/mesos/mesos.proto 0cbee3b188c2f55f6da16d064e1ae9266832d35b Diff: https://reviews.apache.org/r/32728/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32660: Report network isolator statistics on a per container basis (MESOS-2332)

2015-04-01 Thread Chi Zhang
generated e-mail. To reply, visit: https://reviews.apache.org/r/32660/ --- (Updated April 1, 2015, 3:45 p.m.) Review request for mesos, Chi Zhang, Ian Downes, and Cong Wang. Bugs: mesos-2332 https://issues.apache.org/jira/browse

Re: Review Request 32660: Report network isolator statistics on a per container basis (MESOS-2332)

2015-04-01 Thread Chi Zhang
achieve some high-level consensus before diving right into it? - Chi Zhang On April 1, 2015, 3:45 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32660

Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
e691d463515084518c94cdec3fbdf37be4a72977 src/tests/port_mapping_tests.cpp f4124c3e880e043729579a829e1057727741d131 Diff: https://reviews.apache.org/r/32744/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
/network/port_mapping.cpp e691d463515084518c94cdec3fbdf37be4a72977 src/tests/port_mapping_tests.cpp f4124c3e880e043729579a829e1057727741d131 Diff: https://reviews.apache.org/r/32744/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31914: cgroups: added memsw_usage_in_bytes.

2015-04-01 Thread Chi Zhang
df3211a0c25d7a16f36814886d14f81caaef2b9c Diff: https://reviews.apache.org/r/31914/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31915: MemIsolator: Improved some statistics naming. (MESOS-2104)

2015-04-01 Thread Chi Zhang
/containerizer/isolators/cgroups/mem.cpp a7a83ef9ad4726aa139a92fc7f5917ed687d33f5 Diff: https://reviews.apache.org/r/31915/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31915: MemIsolator: Improved some statistics naming. (MESOS-2104)

2015-04-01 Thread Chi Zhang
-mail. To reply, visit: https://reviews.apache.org/r/31915/#review76770 --- On April 1, 2015, 10:28 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 32744: PortMapping: change to not host namespace symlink handles in /var/run/netns.

2015-04-01 Thread Chi Zhang
/network/port_mapping.cpp e691d463515084518c94cdec3fbdf37be4a72977 src/tests/port_mapping_tests.cpp f4124c3e880e043729579a829e1057727741d131 Diff: https://reviews.apache.org/r/32744/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32659: Pull the common container definitions out of PortIsolatorMappingTest for reuse.

2015-03-31 Thread Chi Zhang
-guide/ - Chi Zhang On March 31, 2015, 11:11 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32659

Re: Review Request 32656: Refactor statistics helper out of PortMappingIsolatorTest for easier reuse.

2015-03-31 Thread Chi Zhang
/32656/#comment127117 how about also extract this out to be a 'helperPath', like what you did for launchHelper in the other patch? src/tests/port_mapping_tests.cpp https://reviews.apache.org/r/32656/#comment127116 having no output here isn't expected; why delay reporting it? - Chi Zhang

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-31 Thread Chi Zhang
: https://reviews.apache.org/r/30546/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 32653: Replace busy look on ready file with a more relaxed loop

2015-03-31 Thread Chi Zhang
you can save the timeout logic here. It'd be great see this in the library code! - Chi Zhang On March 31, 2015, 3:33 a.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 32655: Refactor out launchHelper to make is usable outside PortMappingIsolatorTest class

2015-03-31 Thread Chi Zhang
/32655/#comment127087 Could you make it a bit more explict to say this variable is mesos-containerizer *path*? is the indent off by 1? ;) - Chi Zhang On March 31, 2015, 5:35 a.m., Paul Brett wrote: --- This is an automatically

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-30 Thread Chi Zhang
/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-26 Thread Chi Zhang
-mail. To reply, visit: https://reviews.apache.org/r/30546/#review77820 --- On March 27, 2015, 3:24 a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-26 Thread Chi Zhang
/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-26 Thread Chi Zhang
--- On March 27, 2015, 3:24 a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30546

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-03-26 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30545/#review77819 --- On March 27, 2015, 3:18 a.m., Chi Zhang wrote

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-03-26 Thread Chi Zhang
/cgroups.cpp a533b319fc75abb0fc45b8f5f473f257912d21ac Diff: https://reviews.apache.org/r/30545/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-26 Thread Chi Zhang
75c61aad80f894acb92a9752e8d1b6af70e5b9a6 src/tests/memory_test_helper.hpp PRE-CREATION src/tests/memory_test_helper.cpp PRE-CREATION src/tests/memory_test_helper_main.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31276/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-26 Thread Chi Zhang
., Jie Yu wrote: src/tests/memory_test_helper.cpp, line 160 https://reviews.apache.org/r/31276/diff/6/?file=904736#file904736line160 Move this down after validation is done. Also, s/memory/buffer/ Chi Zhang wrote: that won't change the function frame size though

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-26 Thread Chi Zhang
://reviews.apache.org/r/31276/#review9 --- On March 24, 2015, 10:15 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31276

Re: Review Request 32405: Fixed the non-POD global variables in port mapping isolator.

2015-03-25 Thread Chi Zhang
://reviews.apache.org/r/32405/#comment125957 curious why the const is dropped? - Chi Zhang On March 23, 2015, 5:05 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 32426: Symlink the namespace handle with ContainerID for the port mapping isolator.

2015-03-25 Thread Chi Zhang
Is this by design that you don't want to do this section in the above for loop (over all files from ls)? Also I think you can achieve this without using _linkers explictly. - Chi Zhang On March 25, 2015, 6:43 p.m., Jie Yu wrote

Re: Review Request 32466: Cleaned up and simplified PortMappingMesosTest.

2015-03-25 Thread Chi Zhang
/32466/#comment126124 What's the need for unsetting it here? - Chi Zhang On March 25, 2015, 12:04 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32466

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-24 Thread Chi Zhang
75c61aad80f894acb92a9752e8d1b6af70e5b9a6 src/tests/memory_test_helper.hpp PRE-CREATION src/tests/memory_test_helper.cpp PRE-CREATION src/tests/memory_test_helper_main.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31276/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-24 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31276/#review76978 --- On March 24, 2015, 10:15 p.m., Chi Zhang wrote

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-24 Thread Chi Zhang
--- On March 25, 2015, 12:03 a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30546/ --- (Updated March 25, 2015

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-03-20 Thread Chi Zhang
://reviews.apache.org/r/31444/ --- (Updated March 17, 2015, 10:44 p.m.) Review request for mesos, Chi Zhang, Dominic Hamon, Jay Buffington, Jie Yu, and James Peach. Bugs: MESOS-2350 https://issues.apache.org/jira/browse/MESOS-2350

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-03-20 Thread Chi Zhang
points under newRoot private)? src/slave/containerizer/mesos/launch.cpp https://reviews.apache.org/r/31444/#comment125230 why not here just do the pivot_root . old chroot . like the man page recommends? - Chi Zhang On March 17, 2015, 10:44 p.m., Ian Downes wrote

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-03-20 Thread Chi Zhang
On March 21, 2015, 12:21 a.m., Chi Zhang wrote: src/slave/containerizer/mesos/launch.cpp, line 296 https://reviews.apache.org/r/31444/diff/2/?file=898403#file898403line296 From my testing, you only need to make-slave on the newRoot for the pivot_root to work

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-18 Thread Chi Zhang
/ Testing --- Thanks, Chi Zhang

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-18 Thread Chi Zhang
/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-18 Thread Chi Zhang
--- On March 18, 2015, 6:32 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30546/ --- (Updated March 18, 2015, 6:32

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-18 Thread Chi Zhang
On March 11, 2015, 9:59 p.m., Jie Yu wrote: src/tests/memory_test_helper.cpp, line 100 https://reviews.apache.org/r/31276/diff/4/?file=890584#file890584line100 You need a comment explaining why fsync is necessary here. Chi Zhang wrote: confirmed it's not needed. i likedly

Re: Review Request 32133: Refactor port isolator tests to break out helper functions for testing of bandwidth limit statistics

2015-03-18 Thread Chi Zhang
understanding is they are not from gtest, so personally i think we should only use expect and assert in tests which are gtest utilities. But this is obviously not the case currently. - Chi Zhang On March 17, 2015, 6:56 p.m., Paul Brett wrote

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-13 Thread Chi Zhang
:). - Chi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31276/#review76118 --- On March 10, 2015, 7:26 p.m., Chi Zhang wrote

Re: Review Request 31503: (2/5) Add the FlowId action

2015-03-12 Thread Chi Zhang
On March 4, 2015, 10:01 p.m., Chi Zhang wrote: src/linux/routing/filter/action.hpp, line 87 https://reviews.apache.org/r/31503/diff/1-2/?file=879070#file879070line87 could be just parent and child directly. Cong Wang wrote: What is mesos preferred coding style here? I think

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-12 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31276/#review76118 --- On March 10, 2015, 7:26 p.m., Chi Zhang wrote

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-12 Thread Chi Zhang
/31276/#review76118 --- On March 10, 2015, 7:26 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31276

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-11 Thread Chi Zhang
:43 a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30546/ --- (Updated Feb. 28, 2015, 1:43 a.m.) Review

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-11 Thread Chi Zhang
: https://reviews.apache.org/r/30546/#review74647 --- On Feb. 28, 2015, 1:43 a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-03-11 Thread Chi Zhang
/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-03-11 Thread Chi Zhang
/cgroups.cpp a533b319fc75abb0fc45b8f5f473f257912d21ac Diff: https://reviews.apache.org/r/30545/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-03-10 Thread Chi Zhang
://reviews.apache.org/r/31276/diff/ Testing --- Thanks, Chi Zhang

Review Request 31915: MemIsolator: Improved some statistics naming. (MESOS-2104)

2015-03-10 Thread Chi Zhang
6299ca4ba01b65daa3d75c64150e2738e32b841e Diff: https://reviews.apache.org/r/31915/diff/ Testing --- Thanks, Chi Zhang

Review Request 31914: cgroups: added memsw_usage_in_bytes.

2015-03-10 Thread Chi Zhang
/31914/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31784: type_utils: Relaxened the equality check of CommandInfo to allow 'unset' environment == 'empty' environment.

2015-03-06 Thread Chi Zhang
(updated) - src/Makefile.am d299f07d865080676ca8a550cf6005c6ab32839f src/common/type_utils.cpp a1704c67d04d19f65d94dbe56a61bb28561e5bf3 src/tests/type_utils_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31784/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-06 Thread Chi Zhang
: https://reviews.apache.org/r/31250/#review73835 --- On March 6, 2015, 6:36 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-06 Thread Chi Zhang
(updated) --- Thanks, Chi Zhang

Review Request 31784: type_utils: Relaxened the equality check of CommandInfo to allow 'unset' environment == 'empty' environment.

2015-03-05 Thread Chi Zhang
d299f07d865080676ca8a550cf6005c6ab32839f src/common/type_utils.cpp a1704c67d04d19f65d94dbe56a61bb28561e5bf3 src/tests/type_utils_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31784/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-05 Thread Chi Zhang
: https://reviews.apache.org/r/31250/diff/ Testing --- added a new test. the test is end-to-end so that future replacement of the actual logic won't require a change. Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-04 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31250/#review75026 --- On March 4, 2015, 1:47 a.m., Chi Zhang wrote

Re: Review Request 31502: (1/6) Add fq_codel qdisc for egress

2015-03-04 Thread Chi Zhang
/r/31502/#comment122211 unfold IOW? - Chi Zhang On March 4, 2015, 7:58 p.m., Cong Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31502

Re: Review Request 31503: (2/6) Add the FlowId action

2015-03-04 Thread Chi Zhang
/internal.hpp https://reviews.apache.org/r/31503/#comment122206 Do you need to undo the previous step if this is an error? - Chi Zhang On March 4, 2015, 8:06 p.m., Cong Wang wrote: --- This is an automatically generated e-mail

Re: Review Request 31501: (5/6) Add flowid to IP classifier

2015-03-04 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31501/#review75251 --- no diffs :) - Chi Zhang On March 4, 2015, 8:16 p.m., Cong Wang

Re: Review Request 31501: (5/6) Add flowid to IP classifier

2015-03-04 Thread Chi Zhang
to set it to NONE. flowId_(None()) - Chi Zhang On March 4, 2015, 8:16 p.m., Cong Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31501

Re: Review Request 31505: (6/6) Add flow classifiers for fq_codel on egress

2015-03-04 Thread Chi Zhang
will be inserted to your set?? - Chi Zhang On March 4, 2015, 8:18 p.m., Cong Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31505

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-03 Thread Chi Zhang
://reviews.apache.org/r/31250/diff/ Testing --- added a new test. the test is end-to-end so that future replacement of the actual logic won't require a change. Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-03 Thread Chi Zhang
and tids. - Chi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31250/#review74837 --- On March 3, 2015, 7:37 p.m., Chi Zhang wrote

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-03 Thread Chi Zhang
offline. - Chi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31250/#review74837 --- On March 4, 2015, 1:47 a.m., Chi Zhang wrote

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-03-03 Thread Chi Zhang
: https://reviews.apache.org/r/31250/diff/ Testing --- added a new test. the test is end-to-end so that future replacement of the actual logic won't require a change. Thanks, Chi Zhang

Re: Review Request 31503: (2/4) Add the Classifier action

2015-03-02 Thread Chi Zhang
document kernel issues in port_mapping.cpp) src/tests/routing_tests.cpp https://reviews.apache.org/r/31503/#comment121602 kill this line src/tests/routing_tests.cpp https://reviews.apache.org/r/31503/#comment121604 kill this line. - Chi Zhang On Feb. 27, 2015, 9:39 p.m., Cong Wang

Re: Review Request 31503: (2/4) Add the Classifier action

2015-03-02 Thread Chi Zhang
On March 2, 2015, 9:36 p.m., Chi Zhang wrote: src/tests/routing_tests.cpp, line 446 https://reviews.apache.org/r/31503/diff/1/?file=879078#file879078line446 If you can have a way to programmatically determine the existence of this condition, this test should be checked before

Re: Review Request 31505: (4/4) Add flow classifiers for fq_codel on egress

2015-03-02 Thread Chi Zhang
you use. src/slave/containerizer/isolators/network/port_mapping.cpp https://reviews.apache.org/r/31505/#comment121675 add a period at the end of the line. - Chi Zhang On March 2, 2015, 5:21 p.m., Cong Wang wrote

Re: Review Request 31502: (1/4) Add fq_codel qdisc for egress

2015-02-27 Thread Chi Zhang
are these numbers available from kernel header files? - Chi Zhang On Feb. 27, 2015, 9:38 p.m., Cong Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31502

Re: Review Request 30546: MemIsolator: expose memory pressures for containers.

2015-02-27 Thread Chi Zhang
/ Testing --- Thanks, Chi Zhang

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-02-26 Thread Chi Zhang
On Feb. 26, 2015, 11:19 p.m., Chi Zhang wrote: A lot of tricky manuevers; likely have grasped it fully :) s/have/haven not/ - Chi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31444

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-02-26 Thread Chi Zhang
/dev/pts followed by either $ rm /dev/ptmx $ ln -s pts/ptmx /dev/ptmx $ chmod 666 /dev/pts/ptmx or $ mount -o bind /dev/pts/ptmx /dev/ptmx - Chi Zhang On Feb. 25, 2015, 10:48 p.m., Ian Downes wrote

Re: Review Request 31378: Improved NsTest.ROOT_sentns:

2015-02-25 Thread Chi Zhang
/tests/setns_test_helper_main.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31378/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-02-25 Thread Chi Zhang
75c61aad80f894acb92a9752e8d1b6af70e5b9a6 src/tests/memory_test_helper.hpp PRE-CREATION src/tests/memory_test_helper.cpp PRE-CREATION src/tests/memory_test_helper_main.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31276/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31276: Added cgroup memory pressure listening tests.

2015-02-25 Thread Chi Zhang
75c61aad80f894acb92a9752e8d1b6af70e5b9a6 src/tests/memory_test_helper.hpp PRE-CREATION src/tests/memory_test_helper.cpp PRE-CREATION src/tests/memory_test_helper_main.cpp PRE-CREATION Diff: https://reviews.apache.org/r/31276/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30958: PortMappingIsolator: Better ingress qdisc management on eth0 and lo.

2015-02-25 Thread Chi Zhang
the next container comes in. This makes sure after a clean shutdown, the sytem will be clean. Diffs - src/slave/containerizer/isolators/network/port_mapping.cpp b860b17e59486cd6097183a4d3139fbd5c897b44 Diff: https://reviews.apache.org/r/30958/diff/ Testing --- Thanks, Chi Zhang

Review Request 31378: Improved NsTest.ROOT_sentns:

2015-02-24 Thread Chi Zhang
/31378/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31308: mesos: Fixed compilation problems caused by mesos::internal namespace removal.

2015-02-23 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31308/#review73637 --- Ship it! confirmed this should fix the errors. - Chi Zhang

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-02-23 Thread Chi Zhang
. 23, 2015, 8:06 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30545/ --- (Updated Feb. 23, 2015, 8:06 p.m

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-02-23 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30545/#review73587 --- On Feb. 23, 2015, 8:06 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-02-23 Thread Chi Zhang
/cgroups.cpp a533b319fc75abb0fc45b8f5f473f257912d21ac Diff: https://reviews.apache.org/r/30545/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-02-23 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31250/#review73591 --- On Feb. 24, 2015, 12:45 a.m., Chi Zhang wrote

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-02-23 Thread Chi Zhang
://reviews.apache.org/r/31250/diff/ Testing --- added a new test. the test is end-to-end so that future replacement of the actual logic won't require a change. Thanks, Chi Zhang

Re: Review Request 31250: Expose the number of processes and threads in a container when cgroup is enabled.

2015-02-23 Thread Chi Zhang
a.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31250/ --- (Updated Feb. 24, 2015, 12:45 a.m.) Review

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-02-22 Thread Chi Zhang
/cgroups.cpp a307e27c5840270e28cced2d5aeb90c6679bff1d Diff: https://reviews.apache.org/r/30545/diff/ Testing --- Thanks, Chi Zhang

Re: Review Request 30545: cgroups: added support to listen on memory pressures.

2015-02-21 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30545/#review73359 --- On Feb. 22, 2015, 5:13 a.m., Chi Zhang wrote

  1   2   3   >