-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54682/#review158986
-----------------------------------------------------------




src/tests/containerizer/io_switchboard_tests.cpp (line 471)
<https://reviews.apache.org/r/54682/#comment229897>

    Note that this flag only exists on Linux, so this change suddenly causes 
this test to fail on e.g., OS X. `#ifdef`'ing these statements leads to passing 
tests.
    
    Here and below.


- Benjamin Bannier


On Dec. 13, 2016, 12:42 a.m., Jie Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54682/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2016, 12:42 a.m.)
> 
> 
> Review request for mesos and Kevin Klues.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Agent subsystems should be not be used if the corresponding cgroups
> isolator is not used. Ideally, we should add a check in Mesos to
> disallow such a combination.
> 
> 
> Diffs
> -----
> 
>   src/tests/containerizer/io_switchboard_tests.cpp 
> a1ac75549697ba13cbb7c38fcab4a724aefbb402 
> 
> Diff: https://reviews.apache.org/r/54682/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Jie Yu
> 
>

Reply via email to