mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilbert Song <>
Subject Re: Review Request 52412: Supported logger with nested containers in Mesos Containerizer.
Date Wed, 12 Oct 2016 00:09:09 GMT

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

(Updated Oct. 11, 2016, 5:09 p.m.)

Review request for mesos, Artem Harutyunyan, Jie Yu, Joris Van Remoortere, and Joseph Wu.

Bugs: MESOS-6290

Repository: mesos

Description (updated)

Currently, there is an issue in mesos containerizer using logger
for nested contaienrs:

An empty executorinfo is passed to logger when launching a nested
container, it would potentially break some logger modules if any
module tries to access the required proto field (e.g., executorId).

We should pass the ExecutorInfo of a nested container's top level
parent container to the logger when launching a nested container.

Diffs (updated)

  src/slave/containerizer/mesos/containerizer.cpp 32058c35ea9ca95f0a2665994c1ebccd5c840345



make check


Gilbert Song

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