----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51650/#review148666 -----------------------------------------------------------
Ship it! Ship It! - Michael Park On Sept. 8, 2016, 11:57 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51650/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2016, 11:57 a.m.) > > > Review request for mesos and Michael Park. > > > Repository: mesos > > > Description > ------- > > In order minimize external dependency modify the implementation of > `support/mesos-tidy.sh` to build the mesos-tidy Docker container on > the fly just before executing tests. > > We follow the same basic idea already employed in > `support/docker_build.sh` where the image is build, executed, and > removed. Note that this approach likely does not take full advantage > of Docker's caching capabilities. > > > Diffs > ----- > > support/mesos-tidy.sh 36e7f03ba2d9aa4ed982a82d8bbac3416dd52ad7 > > Diff: https://reviews.apache.org/r/51650/diff/ > > > Testing > ------- > > Confirmed that executing > > $ ./support/mesos-tidy.sh > > give same results with external, or ad hoc container image. > > > Thanks, > > Benjamin Bannier > >