----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68145/#review207611 -----------------------------------------------------------
PASS: Mesos patch 68145 was successfully built and tested. Reviews applied: `['68407', '68143', '68144', '68146', '68147', '68362', '68145']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2195/mesos-review-68145 - Mesos Reviewbot Windows On Aug. 20, 2018, 2:52 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68145/ > ----------------------------------------------------------- > > (Updated Aug. 20, 2018, 2:52 a.m.) > > > Review request for mesos, Chun-Hung Hsiao and Jan Schlicht. > > > Bugs: MESOS-8403 > https://issues.apache.org/jira/browse/MESOS-8403 > > > Repository: mesos > > > Description > ------- > > In order to allow proper cleanup the resource provider manager sends a > `REMOVED` to a resource provider when it is being removed. > > The event is not sent reliably, i.e., if the resource provider was > e.g., not subscribed when it was removed we currently will never > attempt to resend the event. > > > Diffs > ----- > > include/mesos/resource_provider/resource_provider.proto > 7c6833378401751a8997ecd3279cf3bf2ad3cbf4 > include/mesos/v1/resource_provider/resource_provider.proto > 535b898b03cd3d81261d62523916cf9a6410a30d > src/resource_provider/manager.cpp abd7e38e5517ea600f9fc9b8a96c7d0d26df0620 > src/resource_provider/storage/provider.cpp > fc48072aac531bac3cbffc3ba089b8dfa2a2f200 > src/tests/mesos.hpp 75c5fae8ed30f64c973b1cc290f8540a072cb8a8 > > > Diff: https://reviews.apache.org/r/68145/diff/7/ > > > Testing > ------- > > `make check` > > Additional testing with the test case added in > https://reviews.apache.org/r/68147/. > > > Thanks, > > Benjamin Bannier > >