----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58820/#review173378 -----------------------------------------------------------
include/mesos/slave/containerizer.proto Line 186 (original), 186 (patched) <https://reviews.apache.org/r/58820/#comment246384> s/sandbox/sandbox in the host (or is it container?) mount namespace/ ? also, not clear what "Mesos sandbox" means? do you mean the value of 'MESOS_SANDBOX' env variable? include/mesos/slave/containerizer.proto Lines 187 (patched) <https://reviews.apache.org/r/58820/#comment246383> s/the image/an image/ - Vinod Kone On April 28, 2017, 4:14 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58820/ > ----------------------------------------------------------- > > (Updated April 28, 2017, 4:14 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > include/mesos/slave/containerizer.proto > c30b1fc659ee9b3cd343899638ced6408d8b51a2 > src/slave/containerizer/mesos/containerizer.cpp > b58baed64480e22f640a4852537f85922ed382ae > > > Diff: https://reviews.apache.org/r/58820/diff/2/ > > > Testing > ------- > > None: not a functional change. > > > Thanks, > > Alexander Rukletsov > >
