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

(Updated Dec. 14, 2015, 6:14 p.m.)

Review request for mesos, Benjamin Hindman, haosdent huang, and Artem 


Finish up the meat of the logic.  Migrated ContainerLogger into the 
containerizer itself.

Summary (updated)

Logger Module: Adds the ContainerLogger into the DockerContainerizer.

Bugs: MESOS-4137

Repository: mesos

Description (updated)

The `DockerContainerizer` creates and initializes its own `ContainerLogger` and 
passes it into the `Docker` wrapper.

The `ContainerLogger` is used before launching executors, in two two call sites 
depending on whether the agent is running in a container or not.

Diffs (updated)

  src/docker/docker.hpp 33d6fb3e82ff7328ad093648a45546a18ec7b8cb 
  src/docker/docker.cpp 5dc4667d93b143b54841d85606affe3e4926757a 
  src/slave/containerizer/docker.hpp 35712f599395b5f5fbc311a37c6e29b33bac0c8e 
  src/slave/containerizer/docker.cpp 2d2dd4e0df36207c5f3cbb4fe2c50df51c0f3e9e 

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

Testing (updated)

Tests will be modified and run later in the review chain.


Joseph Wu

Reply via email to