Re: Review Request 43199: Updated authorization documentation.

2016-02-10 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118611 --- Fix it, then Ship it! Just a few clarification questions, but

Re: Review Request 43199: Updated authorization documentation.

2016-02-10 Thread Greg Mann
> On Feb. 10, 2016, 8:06 a.m., Adam B wrote: > > docs/authorization.md, line 89 > > > > > > "operating system user" still isn't quite right to me, especially in > > light of the abstraction of a "datacenter

Re: Review Request 43199: Updated authorization documentation.

2016-02-10 Thread Greg Mann
> On Feb. 10, 2016, 8:06 a.m., Adam B wrote: > > docs/authorization.md, line 189 > > > > > > Does this mean that no other principal can register a framework at all? > > Or can they still register a framework with

Re: Review Request 43199: Updated authorization documentation.

2016-02-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 11, 2016, 1:26 a.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 11, 2016, 1:31 a.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 9, 2016, 7:35 p.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118446 --- Fix it, then Ship it! docs/authorization.md (line 76)

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 9, 2016, 11:19 p.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118499 --- docs/authorization.md (line 78)

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 10, 2016, 12:32 a.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Greg Mann
> On Feb. 9, 2016, 11:27 p.m., Neil Conway wrote: > > docs/authorization.md, line 78 > > > > > > I don't think a "framework is used to represent various individuals or > > groups". A framework is just a piece of

Re: Review Request 43199: Updated authorization documentation.

2016-02-09 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118583 --- Ship it! Ship It! - Guangya Liu On 二月 10, 2016, 12:32 a.m.,

Re: Review Request 43199: Updated authorization documentation.

2016-02-08 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 8, 2016, 6:40 p.m.) Review request for mesos, Neil Conway and

Re: Review Request 43199: Updated authorization documentation.

2016-02-08 Thread Greg Mann
> On Feb. 6, 2016, 8:02 a.m., Guangya Liu wrote: > > docs/authorization.md, line 50 > > > > > > Do u want to add an example for quota_principals? Good idea! I added examples for quota. Also, after reading through

Re: Review Request 43199: Updated authorization documentation.

2016-02-08 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118325 --- docs/authorization.md (lines 98 - 101)

Re: Review Request 43199: Updated authorization documentation.

2016-02-06 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review118140 --- docs/authorization.md (line 50)

Re: Review Request 43199: Updated authorization documentation.

2016-02-05 Thread Greg Mann
> On Feb. 4, 2016, 7:43 p.m., Neil Conway wrote: > > docs/authorization.md, line 223 > > > > > > s/framework/principal/ , I'd think. Fixed this here, and elsewhere. - Greg

Re: Review Request 43199: Updated authorization documentation.

2016-02-05 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 6, 2016, 2:40 a.m.) Review request for mesos, Neil Conway and

Review Request 43199: Updated authorization documentation.

2016-02-04 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- Review request for mesos, Neil Conway and Vinod Kone. Bugs: MESOS-4452

Re: Review Request 43199: Updated authorization documentation.

2016-02-04 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/#review117865 --- docs/authorization.md (line 78)

Re: Review Request 43199: Updated authorization documentation.

2016-02-04 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43199/ --- (Updated Feb. 4, 2016, 7:54 p.m.) Review request for mesos, Neil Conway and