> On Dec. 4, 2017, 10:05 p.m., Vinod Kone wrote: > > src/tests/hierarchical_allocator_tests.cpp > > Lines 6338 (patched) > > <https://reviews.apache.org/r/64011/diff/6/?file=1908065#file1908065line6338> > > > > I wonder what caused the allocation to happen given you didn't set > > `updated` to true in `updateSlave` for attribute changes?
Doesn't allocation happen periodically every second or so? - Benno ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64011/#review192773 ----------------------------------------------------------- On Dec. 4, 2017, 4:30 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64011/ > ----------------------------------------------------------- > > (Updated Dec. 4, 2017, 4:30 p.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > When an agent reregisters, the master will now always update > the agent information it holds in memory, and will write any > changes back to the registry if necessary. > > Note that most tests for this are added in a later review, since they > require the capability to actually restart the agent with > a changed state to effectively test the masters response to that. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.hpp > 3c87dc797cf70f3aa48b1ed9f86d673d4ea2fe76 > src/master/allocator/mesos/hierarchical.cpp > 715650ee9cb15aed1d1e58badf70fc09e26d13c1 > src/master/master.hpp 5d2ae658070d9c5a0bc630c15ff89dc449857f46 > src/master/master.cpp 883793a226849434eb833c3027d121635a86fdc4 > src/tests/hierarchical_allocator_tests.cpp > 0309074bab180be122c9b0074981e6f69c97feee > > > Diff: https://reviews.apache.org/r/64011/diff/6/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >