----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68145/#review207553 -----------------------------------------------------------
Patch looks great! Reviews applied: [68407, 68143, 68144, 68146, 68410, 68147, 68362, 68145] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Aug. 17, 2018, 1:53 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68145/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2018, 1:53 p.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 > 22c184f45ea43335c3e1e4598f7409a4f05fb38e > src/tests/mesos.hpp 75c5fae8ed30f64c973b1cc290f8540a072cb8a8 > > > Diff: https://reviews.apache.org/r/68145/diff/6/ > > > Testing > ------- > > `make check` > > Additional testing with the test case added in > https://reviews.apache.org/r/68147/. > > > Thanks, > > Benjamin Bannier > >
