> On June 8, 2017, 11:24 a.m., Neil Conway wrote: > > src/tests/upgrade_tests.cpp > > Lines 755 (patched) > > <https://reviews.apache.org/r/59860/diff/1/?file=1743640#file1743640line755> > > > > Any particular reason we're setting `MULTI_ROLE` capability here?
I think mainly because the `FrameworkInfo.role` field is deprecated. But I've removed this from the test for now. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59860/#review177341 ----------------------------------------------------------- On June 13, 2017, 1:22 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59860/ > ----------------------------------------------------------- > > (Updated June 13, 2017, 1:22 a.m.) > > > Review request for mesos, Benjamin Mahler and Neil Conway. > > > Bugs: MESOS-7633 > https://issues.apache.org/jira/browse/MESOS-7633 > > > Repository: mesos > > > Description > ------- > > Prevent allocating non-capable agents' resources to hierarchical roles. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > 73d0b42433a1ff3e853e851b8191864b4e233666 > src/tests/upgrade_tests.cpp b07426fa1e402c88a8a647eafdb77f6ebadd9959 > > > Diff: https://reviews.apache.org/r/59860/diff/2/ > > > Testing > ------- > > New tests + `make check`. > > > Thanks, > > Michael Park > >