mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From haosdent huang <haosd...@gmail.com>
Subject Re: Review Request 49828: Added default methods implementations for `Subsystem` base class.
Date Sat, 06 Aug 2016 16:49:06 GMT


> On July 31, 2016, 5:57 p.m., haosdent huang wrote:
> > src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp, lines 85-97
> > <https://reviews.apache.org/r/49828/diff/10/?file=1458774#file1458774line85>
> >
> >     Suppose we use `cgroups/cpu` as isolation first, then restart the agent with
`cgroups/cpu,cgroups/mem` in `isolation` flag. For those recovered containers, because they
don't appear in the memory subsystem before, they should return `Failure` here.
> 
> Qian Zhang wrote:
>     I think in memory subsystem (https://reviews.apache.org/r/49851/diff), you already
return `Failure` in `update()`/`usage()`, right?
>     
>     And actually I think we do not need to care about the case that agent is restarted
with a different sets of isolators at this moment, because currently Mesos does not officially
support it yet. In future, we may support it with checkpointing what isolators (including
their version) have been used to launch the container, but for now, we do not need to worry
about it.

Just a reference, for use a `cgroupMap` to store the relation between `ContainerID` and `cgroup`,
I implemented it in this branch. https://github.com/haosdent/mesos/blob/MESOS-4697-cgroups/src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp


- haosdent


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


On Aug. 3, 2016, 5:13 p.m., haosdent huang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49828/
> -----------------------------------------------------------
> 
> (Updated Aug. 3, 2016, 5:13 p.m.)
> 
> 
> Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha.
> 
> 
> Bugs: MESOS-5039
>     https://issues.apache.org/jira/browse/MESOS-5039
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added default methods implementations for `Subsystem` base class.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/isolators/cgroups/subsystem.hpp 5f52a076a1fa3a21d886cb961ddeed5046a38d7c

>   src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp a30ecafcbecc9d3b6eeea2b04dcb4d278750af41

> 
> Diff: https://reviews.apache.org/r/49828/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> haosdent huang
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message