----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55810/#review162684 -----------------------------------------------------------
src/slave/containerizer/mesos/io/switchboard.cpp <https://reviews.apache.org/r/55810/#comment234028> Hum, then, it's possible that we install two timers and sending sigterm twice? - Jie Yu On Jan. 24, 2017, 10:45 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55810/ > ----------------------------------------------------------- > > (Updated Jan. 24, 2017, 10:45 p.m.) > > > Review request for mesos, Anand Mazumdar, Benjamin Bannier, Greg Mann, Jie > Yu, and Vinod Kone. > > > Bugs: MESOS-6948 > https://issues.apache.org/jira/browse/MESOS-6948 > > > Repository: mesos > > > Description > ------- > > Fixed bug allowing IOSwitchboard::connect() after container destruction. > > > Diffs > ----- > > src/slave/containerizer/mesos/io/switchboard.cpp > 4b134f8420e1ef5cd139e0e4b8b08b1ff32e3a30 > > Diff: https://reviews.apache.org/r/55810/diff/ > > > Testing > ------- > > GTEST_FILTER="ContentType/AgentAPITest.LaunchNestedContainerSession/1" > src/mesos-tests --gtest_repeat=-1 --gtest_break_on_failure > > > Thanks, > > Kevin Klues > >
