----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44414/#review122308 -----------------------------------------------------------
docs/container-image.md (line 46) <https://reviews.apache.org/r/44414/#comment184267> Do you mean `OCI` here? docs/container-image.md (lines 54 - 60) <https://reviews.apache.org/r/44414/#comment184268> Should we explain why do we allow multiple types? because users may ask `why don't I specify all possible image providers if I want to use unified containerizer`. docs/container-image.md (line 64) <https://reviews.apache.org/r/44414/#comment184269> s/needs/may need/g to make distinction between appc runtime vs docker runtime in the future. docs/container-image.md (lines 82 - 85) <https://reviews.apache.org/r/44414/#comment184271> Should we add `--launcher=linux`? docs/container-image.md (line 150) <https://reviews.apache.org/r/44414/#comment184273> sbin -> /bin docs/container-image.md (line 154) <https://reviews.apache.org/r/44414/#comment184274> sbin -> /bin docs/container-image.md (line 166) <https://reviews.apache.org/r/44414/#comment184277> make consensus with the agent cli commands. docs/container-image.md (line 185) <https://reviews.apache.org/r/44414/#comment184287> should we memtion `~/.curlrc`? docs/container-image.md (line 194) <https://reviews.apache.org/r/44414/#comment184288> We have updated this. Should remove `file://`. docs/container-image.md (line 200) <https://reviews.apache.org/r/44414/#comment184290> ditto. - Gilbert Song On March 4, 2016, 6:20 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44414/ > ----------------------------------------------------------- > > (Updated March 4, 2016, 6:20 p.m.) > > > Review request for mesos, Gilbert Song, Jojy Varghese, Neil Conway, Timothy > Chen, and Vinod Kone. > > > Bugs: MESOS-4873 > https://issues.apache.org/jira/browse/MESOS-4873 > > > Repository: mesos > > > Description > ------- > > Added documentation about container image support. > > > Diffs > ----- > > docs/container-image.md PRE-CREATION > docs/mesos-provisioner.md 1b19406cb93bbc3f5330eaf9d29b1be98a674136 > > Diff: https://reviews.apache.org/r/44414/diff/ > > > Testing > ------- > > Tested the formatting in Mou > > > Thanks, > > Jie Yu > >
