----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62472/#review185923 -----------------------------------------------------------
PASS: Mesos patch 62472 was successfully built and tested. Reviews applied: `['62472']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62472 - Mesos Reviewbot Windows On Sept. 21, 2017, 7:47 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62472/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2017, 7:47 p.m.) > > > Review request for mesos, Benjamin Mahler, Gilbert Song, Jie Yu, and Kevin > Klues. > > > Bugs: MESOS-7643 > https://issues.apache.org/jira/browse/MESOS-7643 > > > Repository: mesos > > > Description > ------- > > Historically, isolators were invoked in the order listed by > the operator in the `--isolation` flag. This changed to an > internal ordering in 6fb9c024, back to the operator ordering > in af474a6fa, and then to an undefined ordering in 9642d3c67b. > > This commit switches back to an internal ordering for all the built-in > isolators. Custom isolators loaded in modules are run in operator order > after all the built-in ones. The rationale for an internal ordering is > expressed in MESOS-5581; basically we should not burden the operator > with having to figure out how to make the order correct. In the case of > custom isolators there's no way for us to know the correct ordering so > we make an arbitrary choice. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > 4d5dc13f363f5d8886983d7dd06a5cecc177c345 > > > Diff: https://reviews.apache.org/r/62472/diff/1/ > > > Testing > ------- > > make check (Fedora 26) > > Posting this as an RFC. If reviewers are OK with the approach, I'll float it > on users@ and dev@ and update the documentation. > > > Thanks, > > James Peach > >
