Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-24 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62877/#review189073 --- I noticed that we have an agent capabilities test here:

Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-24 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62877/#review189070 --- Fix it, then Ship it! include/mesos/mesos.proto Lines 972-974

Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62877/ --- (Updated Oct. 17, 2017, 11:24 p.m.) Review request for mesos, Benjamin

Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62877/ --- (Updated Oct. 17, 2017, 6:59 p.m.) Review request for mesos, Benjamin Bannier,

Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-14 Thread Jie Yu
> On Oct. 12, 2017, 3:04 p.m., Benjamin Bannier wrote: > > include/mesos/mesos.proto > > Lines 958-966 (patched) > > > > > > I do not feel very strongly about this, but it still feels to me that > > this capability

Re: Review Request 62877: Added RESOURCE_PROVIDER agent capability.

2017-10-12 Thread Benjamin Bannier
--- 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