----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58950/#review176299 -----------------------------------------------------------
Patch looks great! Reviews applied: [58950] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On May 30, 2017, 10:22 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58950/ > ----------------------------------------------------------- > > (Updated May 30, 2017, 10:22 a.m.) > > > Review request for mesos, Alexander Rukletsov and Benjamin Mahler. > > > Bugs: MESOS-7388 > https://issues.apache.org/jira/browse/MESOS-7388 > > > Repository: mesos > > > Description > ------- > > The tests in the case often require an agent ID. To obtain the ID they > were using a mock allocator to grab agent ID, but not other operations > with the allocator were performed. > > Instead we now just capture the SlaveRegisteredMessage in order to > learn an agent's ID. > > > Diffs > ----- > > src/tests/persistent_volume_endpoints_tests.cpp > 21136b29d7eeee24959527b889f52611baee0668 > > > Diff: https://reviews.apache.org/r/58950/diff/2/ > > > Testing > ------- > > `make check`, also in repetition. > > While this patch is part of this series since later patches depend on it, it > could be commited independently. > > > Thanks, > > Benjamin Bannier > >