mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joseph Wu" <jos...@mesosphere.io>
Subject Re: Review Request 41378: Logger Module: Update tests that use the DockerContainerizer to pass in a ContainerLogger.
Date Tue, 22 Dec 2015 01:35:04 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41378/
-----------------------------------------------------------

(Updated Dec. 21, 2015, 5:35 p.m.)


Review request for mesos, Benjamin Hindman and Artem Harutyunyan.


Changes
-------

Remove `ContainerLogger` injection from the `Docker` class.


Bugs: MESOS-4137
    https://issues.apache.org/jira/browse/MESOS-4137


Repository: mesos


Description
-------

See summary.


Diffs (updated)
-----

  src/tests/containerizer/docker_containerizer_tests.cpp a43e355405b37443601c0993d36821c50056c24f

  src/tests/hook_tests.cpp e034534295fe6ddd280e25f1d9ee5182cddae187 

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


Testing
-------

make (OSX, Centos7)
make check (OSX, Centos7)
sudo bin/mesos-tests.sh (Centos7)


Thanks,

Joseph Wu


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