----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63555/#review193083 -----------------------------------------------------------
src/slave/slave.hpp Lines 556 (patched) <https://reviews.apache.org/r/63555/#comment271569> typo src/slave/slave.hpp Lines 559 (patched) <https://reviews.apache.org/r/63555/#comment271572> I'd call this `publishResources` and comment here saying this will publish all resources that are needed to run the current set of tasks/executors on the agent. src/slave/slave.cpp Lines 2279-2281 (patched) <https://reviews.apache.org/r/63555/#comment271571> This is no longer needed because `needCheckpointing` has been updated to only look at agent default resources. src/slave/slave.cpp Lines 7198 (patched) <https://reviews.apache.org/r/63555/#comment271573> Can you comment on why you don't use `framework->allocatedResources()` here (because you don't want to consider `queuedTasks` which hasn't been authorized yet. src/tests/slave_tests.cpp Lines 8897-8900 (patched) <https://reviews.apache.org/r/63555/#comment271578> I would just use sleep 1000 here. src/tests/slave_tests.cpp Lines 8961-8964 (patched) <https://reviews.apache.org/r/63555/#comment271581> Can we call `subscribedDefault`? - Jie Yu On Dec. 6, 2017, 8:07 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63555/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2017, 8:07 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, Joseph Wu, and Jan Schlicht. > > > Bugs: MESOS-7550 > https://issues.apache.org/jira/browse/MESOS-7550 > > > Repository: mesos > > > Description > ------- > > `Slave::publishAllocatedResources()` will compute the total allocated > resources for all currently running executor containers, and takes an > `extra` argument for resources that will be used by the executor that > is about to launch, then sums them up and asks the resource provider > manager to publish the resources. > > > Diffs > ----- > > src/resource_provider/manager.cpp 8d8b2f1396230d6edba590b8b7e9b4ca51366efe > src/slave/slave.hpp 643d8559244f0842e82b293c1ef99cb26e111059 > src/slave/slave.cpp c07e25f668bef6b6fa3cae4b69ba90de3eb3bdcc > src/tests/mesos.hpp 3a9b1fbd4301c6cccbc770329ff71ed8ff7e86a2 > src/tests/slave_tests.cpp a7f6658fa467914cd74691b5631d26d85ba7c1eb > > > Diff: https://reviews.apache.org/r/63555/diff/2/ > > > Testing > ------- > > The test is still broken. Will fix it later. > > > Thanks, > > Chun-Hung Hsiao > >
