> 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 > >
