----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59605/#review176271 -----------------------------------------------------------
Ship it! LGTM! It would be great if you could mention MESOS-7586 in the commit message as documentation of the pattern you followed. src/docker/executor.cpp Line 814 (original), 814 (patched) <https://reviews.apache.org/r/59605/#comment250220> Note that in e.g., `src/slave/main.cpp` parsing errors from either the flags or env vars are reported with glog (via `EXIT(EXIT_FAILURE)`). The pattern here does make sense to me (report setup messages from `main` vio stdout/stderr; from functions via glog), and it would be great if you could mention MESOS-7586 in the commit message as documentation of the pattern. Here and elsewhere. - Benjamin Bannier On May 29, 2017, 4:59 p.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59605/ > ----------------------------------------------------------- > > (Updated May 29, 2017, 4:59 p.m.) > > > Review request for mesos, Alexander Rukletsov, Benjamin Bannier, Benjamin > Mahler, and Till Toenshoff. > > > Bugs: MESOS-6961 > https://issues.apache.org/jira/browse/MESOS-6961 > > > Repository: mesos > > > Description > ------- > > Use glog for logging in executors. > > > Diffs > ----- > > src/docker/executor.cpp 464a74859e0f477ec2e5525a5fc726ea420a00a2 > src/launcher/default_executor.cpp 1a60e3bb2b0e8312b349cddedb4cd55716c1b574 > src/launcher/executor.cpp 9ac3c3d84c0ec47954c5c72228ad0b8795ff3eec > > > Diff: https://reviews.apache.org/r/59605/diff/1/ > > > Testing > ------- > > 1. make check (mac os x 10.12, fedora 25) > 2. jenkins: > https://jenkins.mesosphere.com/service/jenkins/job/mesos/job/Mesos_CI-build/1087/ > > 3. I've modified NoTaskKillingCapability from command_executor_tests.cpp to > launch command "sleep 0.1" (as well as "sleep 0"), > then to wait for a task completion. The test was launched in an endless loop: > > GLOG_v=1 ./bin/mesos-tests.sh --gtest_repeat=-1 --gtest_break_on_failure > --gtest_filter="HTTPCommandExecutor/CommandExecutorTest.NoTaskKillingCapability/0" > --verbose > > I've run this test multiple times for a while both on mac os and linux > (gru1.hw.ca1.mesosphere.com). > Race condition problem described in a jira ticket wasn't reproduced. > > > Thanks, > > Andrei Budnik > >