----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65034/#review195005 -----------------------------------------------------------
PASS: Mesos patch 65034 was successfully built and tested. Reviews applied: `['65034']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/65034 - Mesos Reviewbot Windows On Jan. 8, 2018, 4:21 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65034/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2018, 4:21 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Gaston Kleiman, and Jie Yu. > > > Repository: mesos > > > Description > ------- > > When forwarding operation status updates from resource providers > to the agent, the manager was setting the framework ID > unconditionally. This is a problem when OPERATION_DROPPED updates > with no framework ID are generated by the resource provider. > > > Diffs > ----- > > src/resource_provider/manager.cpp 643469339bc1ff702e215b7bb2663c946ca03778 > > > Diff: https://reviews.apache.org/r/65034/diff/1/ > > > Testing > ------- > > Tried `sudo make check`, seeing some CHECK failures. Need to investigate. > > > Thanks, > > Greg Mann > >