----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69572/#review211371 -----------------------------------------------------------
Ship it! Ship It! - Chun-Hung Hsiao 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/1/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >