> On March 2, 2017, 1:21 a.m., Qian Zhang wrote: > > src/common/roles.cpp > > Line 71 (original), 71 (patched) > > <https://reviews.apache.org/r/57166/diff/1/?file=1652003#file1652003line72> > > > > Why can't role start with a slash? According to the design doc, it > > seems the role like `/eng/frontend` is valid.
Qian, this aspect of the hierarchical role design has changed since the design doc was initially proposed. The design doc has been partially updated -- I'll finish updating it shortly. The basic idea is that the syntax for hierarchical roles will just be an extension to the current syntax -- i.e., hierarchical role names will look like `eng/dev` and `eng/prod`, not `/eng/dev`. - Neil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57166/#review167623 ----------------------------------------------------------- On Feb. 28, 2017, 8:23 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57166/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2017, 8:23 p.m.) > > > Review request for mesos, Benjamin Bannier and Michael Park. > > > Repository: mesos > > > Description > ------- > > Role names can now contain forward slashes. Each component in a role > name must now be validated separately. > > > Diffs > ----- > > src/common/roles.cpp 31774a9b8f99f5efeed35b1c3e3486e05ca00f6a > src/tests/role_tests.cpp 77f3d46a544a51ba71476e2f0735bb32758dd9e1 > > > Diff: https://reviews.apache.org/r/57166/diff/1/ > > > Testing > ------- > > `make check` > > > Thanks, > > Neil Conway > >
