> On Dec. 1, 2018, 4:32 a.m., Greg Mann wrote: > > src/slave/slave.cpp > > Lines 8209-8210 (patched) > > <https://reviews.apache.org/r/68147/diff/12/?file=2111262#file2111262line8210> > > > > Some high-level feedback after my first glance at this patch: for the > > MARK_AGENT_GONE call, the master checkpoints a list of gone agents so that > > it can prevent agents in that list from reregistering in the future. While > > the size of this list is constrained, and thus it is possible that a gone > > agent can reregister in the future if it has been GC'd from the list, this > > still allows the master to provide a best-effort guarantee that such agents > > will not return. > > > > Should the agent checkpoint a list of gone RPs so that it can attempt > > to provide the same guarantee?
In the current design this would happen not in the agent, but the RP manager. With this patch we already check if resubscribing RPs do not use a gone RP ID. Limiting the size of that list is currently not implemented. Dropping. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68147/#review210994 ----------------------------------------------------------- On Oct. 25, 2018, 12:47 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68147/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2018, 12:47 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Gastón Kleiman, Greg Mann, and Jan > Schlicht. > > > Bugs: MESOS-8403 > https://issues.apache.org/jira/browse/MESOS-8403 > > > Repository: mesos > > > Description > ------- > > This patch adds support for triggering permanent removal of local > resource providers. We also add authorization and tests as part of this > patch. > > > Diffs > ----- > > include/mesos/agent/agent.proto 74488e873cbf99ca487403b70691912cf3788288 > include/mesos/v1/agent/agent.proto 5d1ab6fd7f6f21593444467c66ed35ee90aea07f > src/common/protobuf_utils.cpp a45607eed4c4bae5010bcc3f3ffeabd6d911062a > src/master/master.cpp 3b3824a67f46866cd64e32d7f9f92484b5891aa2 > src/slave/http.hpp 5b113fa2f7e3421d4219e0ece113010937b204c1 > src/slave/http.cpp ba389263ef7ab311682d542424eb56360841d24c > src/slave/slave.hpp edf7269d4057ec8c95bb54c855210ad00d002a50 > src/slave/slave.cpp 858b78620e1ef33f3587d0bd95a684917aaf5bbb > src/slave/validation.cpp df5e1373dbe497bc859455dcaf4e064e923bd72e > src/tests/api_tests.cpp edde48a4b4b46c9f47bc44de389c97b77322b8e8 > > > Diff: https://reviews.apache.org/r/68147/diff/12/ > > > Testing > ------- > > `make check` > > > Thanks, > > Benjamin Bannier > >
