----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69572/#review211398 -----------------------------------------------------------
PASS: Mesos patch 69572 was successfully built and tested. Reviews applied: `['69569', '69572']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2721/mesos-review-69572 - Mesos Reviewbot Windows On Dec. 17, 2018, 9:56 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69572/ > ----------------------------------------------------------- > > (Updated Dec. 17, 2018, 9:56 a.m.) > > > Review request for mesos and Chun-Hung Hsiao. > > > Bugs: MESOS-9479 > https://issues.apache.org/jira/browse/MESOS-9479 > > > Repository: mesos > > > Description > ------- > > We expect resource providers to set their IDs in operation status > messages. This patch adds some assertion that the IDs match our > expectations, and adjusts some test code to honor these assumptions. > > > Diffs > ----- > > src/resource_provider/manager.cpp 103f5458ec882a37357265bd680a59da0bbad6b3 > src/tests/master_slave_reconciliation_tests.cpp > de6e382ed16d8c02d90dd6157ded1aaae9b593aa > src/tests/resource_provider_manager_tests.cpp > b61c50f839b7a0f49a781a6f44aa4f10ad7ebafe > > > Diff: https://reviews.apache.org/r/69572/diff/2/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >