> On Feb. 6, 2016, 2:55 a.m., Guangya Liu wrote:
> > docs/roles.md, line 22
> > <https://reviews.apache.org/r/43200/diff/3/?file=1236969#file1236969line22>
> >
> >     I saw that we actually already have `role` reference in line 16 and 
> > line 19 in this file.
> >     
> >     So is it possible to also hightlihgt `role` in line 22? e.g. `a group 
> > of frameworks in one role`?

Good point; let me know what you think of the current text :-)


- Greg


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43200/#review118132
-----------------------------------------------------------


On Feb. 6, 2016, 3:47 a.m., Greg Mann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43200/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2016, 3:47 a.m.)
> 
> 
> Review request for mesos, Neil Conway and Vinod Kone.
> 
> 
> Bugs: MESOS-4452
>     https://issues.apache.org/jira/browse/MESOS-4452
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Updated role documentation.
> 
> Added information on the distinction between roles and principals.
> 
> 
> Diffs
> -----
> 
>   docs/roles.md c84a483259922be01d1686befd01b7b1c4005bbd 
> 
> Diff: https://reviews.apache.org/r/43200/diff/
> 
> 
> Testing
> -------
> 
> Viewed in the mesos website container: 
> https://github.com/mesosphere/mesos-website-container
> 
> 
> Thanks,
> 
> Greg Mann
> 
>

Reply via email to