> On July 19, 2018, 11:09 p.m., Gastón Kleiman wrote: > > Can you add a test? At least one that calls suppress/revive, but ideally > > also one that adds and removes roles.
I updated `SchedulerTest.Suppress` to check that the metric returns the correct value, and updated `MasterTest.MultiRoleSchedulerUnsubscribeFromRole` to check that metrics for a role will be removed when the framework unsubscribes from the role. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66870/#review206253 ----------------------------------------------------------- On July 24, 2018, 3:40 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66870/ > ----------------------------------------------------------- > > (Updated July 24, 2018, 3:40 p.m.) > > > Review request for mesos, Benjamin Mahler, Gastón Kleiman, Gilbert Song, and > Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Added per-framework metrics for suppressed roles. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > 707dd6bd0f255a64d759ce87cbf75be57d86b392 > src/master/allocator/mesos/metrics.hpp > 6d386225c301d5ab44f3cc0ecdd1478fb5162e5b > src/master/allocator/mesos/metrics.cpp > 82990b2dc0b827a43a392d898667eaf58c77ea36 > > > Diff: https://reviews.apache.org/r/66870/diff/6/ > > > Testing > ------- > > Testing details can be found at the end of this chain. > > > Thanks, > > Greg Mann > >
