mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 68145: Sent an event to resource providers when they are removed.
Date Mon, 20 Aug 2018 09:52:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68145/
-----------------------------------------------------------

(Updated Aug. 20, 2018, 11:52 a.m.)


Review request for mesos, Chun-Hung Hsiao and Jan Schlicht.


Changes
-------

Add comment.


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 (updated)
-----

  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/

Changes: https://reviews.apache.org/r/68145/diff/6-7/


Testing
-------

`make check`

Additional testing with the test case added in https://reviews.apache.org/r/68147/.


Thanks,

Benjamin Bannier


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message