----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56778/#review165959 -----------------------------------------------------------
Fix it, then Ship it! src/tests/default_executor_tests.cpp (line 1317) <https://reviews.apache.org/r/56778/#comment237800> Is this still needed? Thought we allowed implicit roles now? src/tests/default_executor_tests.cpp (lines 1324 - 1335) <https://reviews.apache.org/r/56778/#comment237801> move this to #1380 src/tests/default_executor_tests.cpp (lines 1416 - 1417) <https://reviews.apache.org/r/56778/#comment237804> hmm. why didn't you just start with a v1::ExecutorInfo like you did with v1::TaskGroup? - Vinod Kone On Feb. 17, 2017, 5:24 a.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56778/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2017, 5:24 a.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-7137 > https://issues.apache.org/jira/browse/MESOS-7137 > > > Repository: mesos > > > Description > ------- > > We were not unallocating the resources before checking if the > executor resources were contained in the checkpointed resources > on the agent. > > > Diffs > ----- > > src/slave/slave.cpp ebba8e16bc9ec45781183e78cb5a3c351a5f65f5 > src/tests/default_executor_tests.cpp > ffb69e9d6745c267704195056500edd4d8a4ca3f > > Diff: https://reviews.apache.org/r/56778/diff/ > > > Testing > ------- > > make check + added test that fails without the change. > > > Thanks, > > Anand Mazumdar > >
