> On Jan. 12, 2015, 5:30 p.m., Bernd Mathiske wrote: > > src/slave/containerizer/docker.cpp, line 112 > > <https://reviews.apache.org/r/29336/diff/2/?file=813856#file813856line112> > > > > According to review #29341 this is wrong. See your updated solution > > there! The latter seems to be the correct one to me.
Yes I fix this in a later review. I uploaded the whole commit chain, where the latest has some fixes. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29336/#review67660 ----------------------------------------------------------- On Jan. 9, 2015, 1:25 a.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29336/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2015, 1:25 a.m.) > > > Review request for mesos, Benjamin Hindman and Bernd Mathiske. > > > Repository: mesos-git > > > Description > ------- > > Recover docker containers that launched in containers. > > > Diffs > ----- > > src/slave/containerizer/docker.hpp b7bf54a > src/slave/containerizer/docker.cpp 5f4b4ce > > Diff: https://reviews.apache.org/r/29336/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy Chen > >
