mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <>
Subject Review Request 58903: Combined Mesos containerizer's launch methods.
Date Tue, 02 May 2017 02:04:36 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos and Jie Yu.

Bugs: MESOS-7449

Repository: mesos


This simplifies the container launch path by removing combining
the nested and non-nested container code paths into one.

The Mesos containerizer was originally translating the two
`containerizer->launch` entrypoints into a common method (also
called `launch`).  The previous commits moved this translation
logic into the caller (i.e. the Agent).

The end result has some slight changes:
  * It is now possible for the Agent to specify some more combinations
    of `ContainerConfig`.  For example, specifying a TaskInfo with
    a DEBUG-class container.  Or a nested container with Resources.
    We may need to add extra validation around this.
  * The `bool checkpoint` argument was replaced with a `Option<string>`
    which optionally contains an absolute path.  This allows us to
    remove the `SlaveID` field.


  src/slave/containerizer/mesos/containerizer.hpp 29a99f33e646593127b9dc126f398f7bca88e21d

  src/slave/containerizer/mesos/containerizer.cpp b58baed64480e22f640a4852537f85922ed382ae



See last patch in chain.


Joseph Wu

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