> On Jan. 24, 2017, 11:04 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/io/switchboard.cpp, lines 761-762 > > <https://reviews.apache.org/r/55810/diff/1/?file=1611305#file1611305line761> > > > > Hum, then, it's possible that we install two timers and sending sigterm > > twice?
yes, and i think that's safe? - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55810/#review162684 ----------------------------------------------------------- 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 > >
