mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jie Yu <yujie....@gmail.com>
Subject Re: Review Request 60888: Added recovery logic for standalone containers.
Date Tue, 07 Nov 2017 22:52:06 GMT


> On Oct. 17, 2017, 5:39 p.m., Jie Yu wrote:
> > src/slave/paths.hpp
> > Lines 53-54 (patched)
> > <https://reviews.apache.org/r/60888/diff/3/?file=1858655#file1858655line53>
> >
> >     Putting this path here is a bit wierd because this directory is kind of containerizer
specific.
> >     
> >     I'd suggest we create a `src/slave/containerizer/paths.hpp` to put those path
helper functions.
> >     
> >     `src/slave/paths.hpp` should probably only have paths that are related to slave,
but not containers. In the future, the sandbox for executors can be a symlink to the actual
container sandbox.
> 
> Joseph Wu wrote:
>     Leaving this open as I didn't change the directory structure...
>     
>     Sandbox directories (for normal and standalone containers) are currently specified
by the agent, so it makes sense to have this directory in the agent's helper.  The containerizer
only specifies sandboxes for nested containers by appending onto an existing sandbox (which
itself is directly or indirectly (for multi-level nested containers) specified by the agent).

ok, fair enough. I think we can revisit this once we want to support just running containerizer
alone without the agent.


- Jie


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


On Nov. 2, 2017, 3:43 p.m., Joseph Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60888/
> -----------------------------------------------------------
> 
> (Updated Nov. 2, 2017, 3:43 p.m.)
> 
> 
> Review request for mesos, Gilbert Song and Jie Yu.
> 
> 
> Bugs: MESOS-7305
>     https://issues.apache.org/jira/browse/MESOS-7305
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Although there is no way to launch standalone containers yet,
> this commit outlines the expected layout of container metadata
> which should be populated when launching standalone containers.
> 
> The layout is fairly simple, as standalone containers have no
> framework, executor, or tasks to worry about.  The sandbox directory
> will live under a new top-level directory `containers` and there is
> no metadata to checkpoint at the moment.
> 
> The containerizer will checkpoint a marker file (in the runtime
> directory) so that it knows to recover all standalone containers.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/containerizer.cpp 100e3bbda543d87808da9ff6bea42da5099ea8c5

>   src/slave/containerizer/mesos/paths.hpp a03f15e01e8eb8a1326baad8d7db96a79d785482 
>   src/slave/containerizer/mesos/paths.cpp 0c61c20c345a327ec469b382558aaeed0280e754 
>   src/slave/paths.hpp f000508d414daf9f943561f89c7105503a8a98b3 
>   src/slave/paths.cpp fd546525b900cb6524fb9196d19616ef18de0f30 
> 
> 
> Diff: https://reviews.apache.org/r/60888/diff/4/
> 
> 
> Testing
> -------
> 
> See next patch and later ones too.
> 
> 
> Thanks,
> 
> Joseph Wu
> 
>


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