----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62877/#review187799 -----------------------------------------------------------
Fix it, then Ship it! include/mesos/mesos.proto Lines 958-966 (patched) <https://reviews.apache.org/r/62877/#comment264840> I do not feel very strongly about this, but it still feels to me that this capability is less about resource providers and more about the agent being able to provide a protocol for offer operation feedback. The capability does also show up in e.g., the `/master/slaves` endpoint. I could imagine frameworks being interested in looking at this capability to follow different logic for offer operation feedback. Still, resource providers would not be directly visible to them. From an implementation point of view, we will add code to the master providing differing protocols for offer operations branching on the presence of this capability. We again mostly would not care about resource providers in that case either. - Benjamin Bannier On Oct. 11, 2017, 5:24 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62877/ > ----------------------------------------------------------- > > (Updated Oct. 11, 2017, 5:24 a.m.) > > > Review request for mesos, Benjamin Bannier, Joseph Wu, and Jan Schlicht. > > > Bugs: MESOS-8071 > https://issues.apache.org/jira/browse/MESOS-8071 > > > Repository: mesos > > > Description > ------- > > Added RESOURCE_PROVIDER agent capability. > > > Diffs > ----- > > include/mesos/mesos.proto 830985a3265b7c104d8fdc50749c395d98f5f3c8 > include/mesos/v1/mesos.proto a6d662fb26aa4f78ef20ffe6e013f7a45f7f8c21 > src/common/protobuf_utils.hpp ff0fd01b7a20f597ec6cd916f0bd9c41baa5fd51 > src/common/protobuf_utils.cpp 04f44f6f63e431c17ec67e234c8da58e7945294d > > > Diff: https://reviews.apache.org/r/62877/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >