----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56195/#review163956 -----------------------------------------------------------
Patch looks great! Reviews applied: [56195] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Feb. 2, 2017, 5:29 a.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56195/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2017, 5:29 a.m.) > > > Review request for mesos, Alexander Rukletsov, Gastón Kleiman, Gilbert Song, > and Jie Yu. > > > Bugs: MESOS-7050 > https://issues.apache.org/jira/browse/MESOS-7050 > > > Repository: mesos > > > Description > ------- > > Previously, if the containizer launch path failed before actually > launching the container, the FDs allocated to the container by the > IOSwitchboard isolator would be leaked. This would lead to deadlock in > the destroy path because the IOSwitchboard does not shutdown until the > FDs it allocates to the container have been closed. Since the > switchboard doesn't shutdown, the future returned by its 'cleanup()' > function is never satisfied. > > This commit makes sure to close the FDs under all failure cases in the > launch path. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > 4f0a773676da45fa40ad1ad9cdfab2a19249247d > > Diff: https://reviews.apache.org/r/56195/diff/ > > > Testing > ------- > > Linux CentOS 7: > ``` > GTEST_FILTER="" make -j check > src/mesos-tests > [----------] Global test environment tear-down > [==========] 1477 tests from 167 test cases ran. (390804 ms total) > [ PASSED ] 1477 tests. > ``` > > Mac OS X (El Capitan) > ``` > GTEST_FILTER="" make -j check > src/mesos-tests > [ FAILED ] 3 tests, listed below: > [ FAILED ] ExamplesTest.V1JavaFramework > [ FAILED ] ExamplesTest.PythonFramework > ``` > > > Thanks, > > Kevin Klues > >
