Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-11-07 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review190299 --- I'd suggest to discard this patch in favor of the feature flag int

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-17 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review188426 --- Patch looks great! Reviews applied: [63001, 62877, 62878, 62879,

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-17 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review188421 --- PASS: Mesos patch 62880 was successfully built and tested. Review

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

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

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

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

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-13 Thread Benjamin Bannier
> On Oct. 12, 2017, 5:39 p.m., Benjamin Bannier wrote: > > I would much prefer if we'd add another patch before this that modifies the > > agent to e.g., fail over should it receive a `CheckpointResourcesMessage` > > and has this capability. Offer operation feedback and > > `CheckpointResource

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-12 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review187781 --- I would much prefer if we'd add another patch before this that mod

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-10 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review187625 --- PASS: Mesos patch 62880 was successfully built and tested. Review

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-10 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/#review187622 --- Patch looks great! Reviews applied: [62877, 62878, 62879, 62880]

Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

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

Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable.

2017-10-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62880/ --- Review request for mesos, Benjamin Bannier, Joseph Wu, and Jan Schlicht. Reposi