> On June 9, 2016, 7:43 p.m., Joseph Wu wrote: > > src/docker/executor.cpp, lines 307-314 > > <https://reviews.apache.org/r/48429/diff/2/?file=1411852#file1411852line307> > > > > If the `docker inspect` times out, is it possible that we still try to > > `docker stop` on a container docker does not know about? > > > > And is it possible to leak a container if: > > 1) `docker run ...` starts. > > 2) `docker inspect` does not complete. > > 3) Kill is issued. > > 4) `docker inspect` times out. > > 5) `_killTask` sends a `TASK_KILLED` and `docker stop`. > > 6) `docker stop` does not kill the container because it hasn't started > > yet.
Chatted with joseph, this case is not possible because we'll only discard the inspect when the container has terminated (in reap). When we try to stop the container we do not discard the 'inspect' due to danger of leaving a container running. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48429/#review136881 ----------------------------------------------------------- On June 9, 2016, 4:04 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48429/ > ----------------------------------------------------------- > > (Updated June 9, 2016, 4:04 a.m.) > > > Review request for mesos and Jie Yu. > > > Bugs: MESOS-4279 and MESOS-5195 > https://issues.apache.org/jira/browse/MESOS-4279 > https://issues.apache.org/jira/browse/MESOS-5195 > > > Repository: mesos > > > Description > ------- > > Currently, the docker executor will SIGKILL the 'docker run' > subprocess immediately upon receiving a kill task request. > This is problematic in that the 'docker run' subprocess is > responsible for streaming the stdout and stderr from the > container. When we kill the 'docker run' subprocess, we no > longer stream the output. > > > Diffs > ----- > > src/docker/executor.cpp 1b3a7795b1db83394d4b884c1041c341f88a7df1 > > Diff: https://reviews.apache.org/r/48429/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Benjamin Mahler > >
