----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54378/#review158015 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/shared_initialization.py (line 116) <https://reviews.apache.org/r/54378/#comment228671> how about adding a space after : for readability? - Dmitro Lisnichenko On Dec. 5, 2016, 5:45 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54378/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2016, 5:45 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-19087 > https://issues.apache.org/jira/browse/AMBARI-19087 > > > Repository: ambari > > > Description > ------- > > dfs.cluster.administrators handles values of the following format: > > > > CL for the admins, this configuration is used to control who can access > the default servlets in the namenode, etc. The value should be a comma > separated list of users and groups. The user list comes first and is > separated by a space followed by the group list, e.g. "user1,user2 > group1,group2". Both users and groups are optional, so "user1", " group1", > "", "user1 group1", "user1,user2 group1,group2" are all valid (note the > leading space in " group1"). '*' grants access to all users and groups, e.g. > '*', '* ' and ' *' are all valid. > > > Ambari cannot handle * value correctly it will try to create a group or user > named '*'. > Also Ambari cannot correctly create groups if there are multiple spaces > instead of one. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/shared_initialization.py > 320872e > > Diff: https://reviews.apache.org/r/54378/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
