> On Jan. 19, 2016, 7:53 p.m., Guangya Liu wrote: > > src/tests/hierarchical_allocator_tests.cpp, lines 1817-1829 > > <https://reviews.apache.org/r/42289/diff/3/?file=1201483#file1201483line1817> > > > > What about make the test cases cover both `addSlave()` before and after > > `createFrameworkInfo`?
The case after `createFrameworkInfo` is enough to cover both cases. - Klaus ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42289/#review115143 ----------------------------------------------------------- On Jan. 19, 2016, 8:03 p.m., Klaus Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42289/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2016, 8:03 p.m.) > > > Review request for mesos, Alexander Rukletsov, Ben Mahler, Joris Van > Remoortere, and Neil Conway. > > > Bugs: MESOS-4102 > https://issues.apache.org/jira/browse/MESOS-4102 > > > Repository: mesos > > > Description > ------- > > __Phenomenon__: > Quota doesn't allocate resources on slave joining. > > __Root Cause__: > Event-triggered allocations do not include all available agents. If we > calculate remaining resources in the cluster using the partial view, > we may overlook already laid away resources for quota'ed roles and lay > away more. Hence we may unnecessarily deprive non-quota'ed frameworks > of resources. > > Refer to AlexR's comments for more detail: > https://issues.apache.org/jira/browse/MESOS-4102?focusedCommentId=15048495&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15048495 > > __Solution/Fix__: > Calcuated 'remainingClusterResources' by all activated slaves. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > 48acde69b1a2f305b568a7e322a58708063dd30a > src/tests/hierarchical_allocator_tests.cpp > 9362dd306497ba01e0f387c3862456cdcac6f863 > > Diff: https://reviews.apache.org/r/42289/diff/ > > > Testing > ------- > > make > make check > ./bin/mesos-tests.sh --gtest_filter="HierarchicalAllocatorTest.*" --verbose > --gtest_repeat=100 --gtest_shuffle > > > Thanks, > > Klaus Ma > >
