----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67146/#review203198 -----------------------------------------------------------
PASS: Mesos patch 67146 was successfully built and tested. Reviews applied: `['67146']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67146 - Mesos Reviewbot Windows On May 15, 2018, 5:44 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67146/ > ----------------------------------------------------------- > > (Updated May 15, 2018, 5:44 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Joseph Wu, and Till Toenshoff. > > > Bugs: MESOS-8920 > https://issues.apache.org/jira/browse/MESOS-8920 > > > Repository: mesos > > > Description > ------- > > Contianer logger's prepare() method now takes `ContainerConfig` to be > consistent with the isolator interfaces. This allows the framework to > control the logger behavior for nested containers, as well as standalone > containers (by using environment variables). > > > Diffs > ----- > > CHANGELOG c7173fc887433422b890bf13809ea2e3fcaaa1ff > docs/logging.md 28039141fecf9d84baa9d5f419c33419647de5c8 > docs/upgrades.md baf4a0f9ef4ee96debd3673777c12a0b4d84c6db > include/mesos/slave/container_logger.hpp > e8f8838d42df8e63a4c30f75e2a10b8930a71661 > src/slave/container_loggers/lib_logrotate.hpp > 5c6d3a980a718e9ab087be37f913cb5357eab61b > src/slave/container_loggers/lib_logrotate.cpp > 575ed6e4cf54641689c6c7005f113939312aaeb3 > src/slave/container_loggers/sandbox.hpp > fb06a70f8ffad0ddd29c48a8d73e4962268a650c > src/slave/container_loggers/sandbox.cpp > b01e359b6a9827c2fa7dbb46eda574b24d47a13c > src/slave/containerizer/docker.cpp 7171cb5f2cdbfcceb35f7d58c0669fbc575d683e > src/slave/containerizer/mesos/io/switchboard.cpp > 517d45903bc5095f2f954b95feb61c8717d395f0 > > > Diff: https://reviews.apache.org/r/67146/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >