----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45231/#review125638 -----------------------------------------------------------
Ship it! - Joris Van Remoortere On March 28, 2016, 11:43 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45231/ > ----------------------------------------------------------- > > (Updated March 28, 2016, 11:43 a.m.) > > > Review request for mesos and Joris Van Remoortere. > > > Repository: mesos > > > Description > ------- > > This review follows the previous one and removes most > (see following reviews) usages setup functions throughout > the code. > > > Diffs > ----- > > src/docker/docker.cpp 4d35513cdd9c044d37d876a6db7dd9321ceaca53 > src/health-check/main.cpp 36955dd5f77a056213762d92c39be4ca7dd5780c > src/slave/container_loggers/lib_logrotate.cpp > cf5f238eb3b6217b848cf23f8c9cd1848bf9a9f0 > src/slave/containerizer/docker.cpp 0133628f5b37dc0d3aa12f210c61dfc07deac18b > src/slave/containerizer/external_containerizer.cpp > fe368dd52644b89e47ffd4d947de290b3998fb1a > src/slave/containerizer/fetcher.cpp > 2b1e022c8dcd9734663f20fe963a8de0d6233715 > src/slave/containerizer/mesos/containerizer.cpp > e7f7e7fd1304e14dbfaab8b53cea16efc0417911 > src/slave/containerizer/mesos/isolators/posix/disk.cpp > b38d83fbb29f46552ffbda7b17cbc85af15550e1 > src/slave/containerizer/mesos/launcher.hpp > 8afac20c5a4093162f764412dbf16c4230dcb141 > src/slave/containerizer/mesos/launcher.cpp > ba012b1d577a770948a1bfcad5ad2d4977f81b2f > src/slave/containerizer/mesos/linux_launcher.hpp > 1763d8d48614e0a1e187505d90920e97c8e476fb > src/slave/containerizer/mesos/linux_launcher.cpp > 550c58263df257989568debe2858ef599e5f96e8 > src/tests/containerizer/isolator_tests.cpp > df506fc793e480e825b476e43c683ef8bcf676b2 > src/tests/containerizer/launch_tests.cpp > e0f934cf93f51316b35f34d34f3d0c5f004565d9 > src/tests/containerizer/launcher.hpp > 36405a97893f352147af4794fb39b6e621d95b58 > src/tests/containerizer/launcher.cpp > a92d9890f0931425d69ef8ce0896d081b8722079 > src/tests/containerizer/ns_tests.cpp > 4bf45e970bede713fa4ffce205627b149232fe2b > src/tests/slave_tests.cpp 1f1a31020096efa5db698e86ac74e61dfdb4b94a > > Diff: https://reviews.apache.org/r/45231/diff/ > > > Testing > ------- > > tested complete chain (see https://reviews.apache.org/r/45236/). > > > Thanks, > > Joerg Schad > >
