Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 29, 2016, 7:20 a.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
> On Feb. 28, 2016, 3:06 p.m., Klaus Ma wrote: > > docs/reservation.md, line 363 > > > > > > s/slave/agent/ The rest of this file uses "slave", so I think this is better as-is for now (until we do the rename). -

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
> On Feb. 28, 2016, 3:15 a.m., Guangya Liu wrote: > > docs/persistent-volume.md, lines 372-374 > > > > > > Only the `reserved_resources_full` was mentioned here, what about > > `used_resources_full` and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 29, 2016, 7:13 a.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 29, 2016, 7:01 a.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 29, 2016, 6:55 a.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-28 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/#review121145 --- docs/reservation.md (line 363)

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-27 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/#review121114 --- docs/persistent-volume.md (lines 372 - 374)

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/#review12 --- Patch looks great! Reviews applied: [43937, 43938, 43939, 43816,

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-27 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 28, 2016, 12:17 a.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-26 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- (Updated Feb. 26, 2016, 7:10 p.m.) Review request for mesos, Michael Park and

Re: Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/#review120845 --- Patch looks great! Reviews applied: [43816, 43822, 43823, 43817,

Review Request 44047: Added full reserved resource info to `/slaves` master endpoint.

2016-02-25 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44047/ --- Review request for mesos, Michael Park and Vinod Kone. Bugs: MESOS-4667