----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58950/#review177029 -----------------------------------------------------------
Bad patch! Reviews applied: [58950] Failed command: python support/apply-reviews.py -n -r 58950 Error: error: patch failed: src/tests/persistent_volume_endpoints_tests.cpp:1576 error: src/tests/persistent_volume_endpoints_tests.cpp: patch does not apply Full log: http://mesos-winbot.westus.cloudapp.azure.com/logs/59/console - Mesos Reviewbot Windows On June 6, 2017, 12:38 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58950/ > ----------------------------------------------------------- > > (Updated June 6, 2017, 12:38 p.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 > 8d0a7970ac03d0dbff6acccce894b922951d4e5d > > > Diff: https://reviews.apache.org/r/58950/diff/3/ > > > 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 > >
