Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-03-01 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/ --- (Updated March 1, 2016, 10:17 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-03-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/#review121508 --- Patch looks great! Reviews applied: [43914, 43915, 44182, 43916]

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-03-01 Thread Avinash sridharan
> On March 1, 2016, 5:13 p.m., Jie Yu wrote: > > src/tests/containerizer/isolator_tests.cpp, line 387 > > > > > > I think we set a default for the secondary so that you don't have to do > > this here? Updated the

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-03-01 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/ --- (Updated March 1, 2016, 7:59 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-03-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/#review121449 --- Fix it, then Ship it!

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-02-29 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/ --- (Updated Feb. 29, 2016, 10:16 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-02-24 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/#review120578 --- Patch looks great! Reviews applied: [43914, 43915, 43916]

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-02-24 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/ --- (Updated Feb. 24, 2016, 4:54 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-02-23 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/#review120458 --- Patch looks great! Reviews applied: [43913, 43914, 43915, 43916]

Review Request 43916: Modified testcase to test `--cgroups_net_cls_secondary_handles` flag.

2016-02-23 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43916/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4700