mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joseph Wu" <>
Subject Re: Review Request 41369: Logger Module: Add ContainerLogger support to the Mesos Docker executor
Date Tue, 15 Dec 2015 20:41:16 GMT

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

(Updated Dec. 15, 2015, 12:41 p.m.)

Review request for mesos, Benjamin Hindman and Artem Harutyunyan.


Remove `options` in favor of module parameters.

Bugs: MESOS-4137

Repository: mesos


Adds `ContainerLogger` flags, creation, and initialization to the Mesos Docker executor. 
This `ContainerLogger` instance runs in the executor itself.

Diffs (updated)

  src/docker/executor.hpp abbc419533ab40312e917931a2fc2ce78b38da41 
  src/docker/executor.cpp 4042cec0acbe03d937ea3c53ffde745cbba552d2 
  src/slave/containerizer/docker.cpp 2d2dd4e0df36207c5f3cbb4fe2c50df51c0f3e9e 



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


Joseph Wu

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