Re: Review Request 43323: Added unit-test for checking `NetCls` classid in `state.json.

2016-02-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43323/#review118678 --- Patch looks great! Reviews applied: [42782, 42947, 42982, 42983,

Re: Review Request 43323: Added unit-test for checking `NetCls` classid in `state.json.

2016-02-10 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43323/#review118677 --- Ship it! Ship It! - Kapil Arya On Feb. 10, 2016, 2:05 a.m.,

Re: Review Request 43323: Added unit-test for checking `NetCls` classid in `state.json.

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

Re: Review Request 43323: Added unit-test for checking `NetCls` classid in `state.json.

2016-02-09 Thread Avinash sridharan
> On Feb. 10, 2016, 1:26 a.m., Jie Yu wrote: > > src/tests/containerizer/isolator_tests.cpp, line 1089 > > > > > > Can you do > > ``` > > namespace http = process::http; > > ``` > > in the begining

Re: Review Request 43323: Added unit-test for checking `NetCls` classid in `state.json.

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