----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48430/#review136716 -----------------------------------------------------------
Bad patch! Reviews applied: [48430, 48429, 48428, 48427, 48426, 48425, 48424, 48423, 48422] Failed command: ./support/apply-review.sh -n -r 48423 Error: 2016-06-08 22:12:33 URL:https://reviews.apache.org/r/48423/diff/raw/ [1772/1772] -> "48423.patch" [1] Total errors found: 0 Checking 1 files Error: No line in the commit message summary may exceed 72 characters. Full log: https://builds.apache.org/job/mesos-reviewbot/13633/console - Mesos ReviewBot On June 8, 2016, 7:26 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48430/ > ----------------------------------------------------------- > > (Updated June 8, 2016, 7:26 p.m.) > > > Review request for mesos and Jie Yu. > > > Repository: mesos > > > Description > ------- > > This wraps 'docker kill', which will be of use in the Docker > executor for controlling signal escalation ourselves. Currently, > by using 'docker stop --timeout', we cannot adjust the kill > policy as straightforwardly as directly using 'docker kill'. > > > Diffs > ----- > > src/docker/docker.hpp d9ffc18496718701fac8182506a8c36e21e9c319 > src/docker/docker.cpp ffbd7ef56d02dfe550a60851251e6645d2ca5925 > src/tests/containerizer/docker_tests.cpp > 7ef52ade0d3389f9e24e3c5c7dda4f8809b9d83f > > Diff: https://reviews.apache.org/r/48430/diff/ > > > Testing > ------- > > Added a test. > > > Thanks, > > Benjamin Mahler > >
