You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mesos.apache.org by Kapil Arya <ka...@mesosphere.io> on 2015/02/05 04:15:26 UTC

Re: Review Request 29597: Removed #include slave/flags.hpp from isolator.hpp

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

(Updated Feb. 4, 2015, 10:15 p.m.)


Review request for mesos, Niklas Nielsen, Till Toenshoff, and Timothy Chen.


Changes
-------

rebased


Repository: mesos


Description
-------

Slave flags are not being used in isolator.hpp. Removed #include slave/flags.hpp
from isolator.hpp and included it directly in only those files that need it.


Diffs (updated)
-----

  src/slave/containerizer/isolator.hpp a27c3e955ff25b87599af0ac2c97427a88b786f6 
  src/slave/containerizer/isolators/cgroups/cpushare.hpp 4ded0c4678ad61742a69e14233006a448f3fc3ca 
  src/slave/containerizer/isolators/cgroups/mem.hpp 2fa755571b4d21b8b13301fcfd57ae05ea66e6e6 
  src/slave/containerizer/isolators/cgroups/perf_event.hpp e511c3efe4cc4ec13cc74bdcda993477a4db2e36 
  src/slave/containerizer/isolators/filesystem/shared.hpp 727f63e16fa69293c472a4b6b95fb0be04e689c8 
  src/slave/containerizer/isolators/namespaces/pid.hpp 8da6cccff04edf11fa37f5a33648821299cf49d5 
  src/slave/containerizer/isolators/posix.hpp 7a667e3e30712b98484947520b58965d3b768659 
  src/slave/containerizer/isolators/posix/disk.hpp cf4e143eb50cf5ee59ca0d47dd6f8d0a845c3732 

Diff: https://reviews.apache.org/r/29597/diff/


Testing
-------

make check


Thanks,

Kapil Arya