----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56727/#review166569 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java (line 432) <https://reviews.apache.org/r/56727/#comment238583> Nit! Rename to USER_GROUPS = "user_groups" ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java (line 420) <https://reviews.apache.org/r/56727/#comment238584> Nit! Rename to userGroups ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java (line 2411) <https://reviews.apache.org/r/56727/#comment238585> Nit! Rename to userGroups ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java (line 403) <https://reviews.apache.org/r/56727/#comment238586> Nit! Rename to userGroups ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 181) <https://reviews.apache.org/r/56727/#comment238588> Diamond notation is better coding style especially when you have declarations like these :) ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 547) <https://reviews.apache.org/r/56727/#comment238547> Add Java docs ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 565) <https://reviews.apache.org/r/56727/#comment238548> Add Java docs ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 605) <https://reviews.apache.org/r/56727/#comment238589> Log both userGroupInfo.getType() and userGroupInfo.getName() Example: "User group mapping property {hadoop-env/hdfs_group} is missing for user property {hadoop-env/hdfs_user} (username = hdfs)" ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 617) <https://reviews.apache.org/r/56727/#comment238549> Add Java docs ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 634) <https://reviews.apache.org/r/56727/#comment238550> Add Java docs ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 1055) <https://reviews.apache.org/r/56727/#comment238551> doesnt look like this change is required to createConfig(). We are removing java doc here. Probably a bad merge? ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java (line 1284) <https://reviews.apache.org/r/56727/#comment238582> You don't need to change from new HashMap<>() to new HashMap<String, String> Explicit type arguments is redundant and can be replaced with diamond type <> in JDK 1.7+. Ambari is supported with JDK 1.7+ http://docs.hortonworks.com/HDPDocuments/Ambari-2.4.2.0/bk_ambari-installation/content/jdk_requirements.html https://docs.oracle.com/javase/tutorial/java/generics/types.html - Jayush Luniya On Feb. 23, 2017, 7:39 p.m., Madhuvanthi Radhakrishnan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56727/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2017, 7:39 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit > Mohanty. > > > Bugs: AMBARI-20034 > https://issues.apache.org/jira/browse/AMBARI-20034 > > > Repository: ambari > > > Description > ------- > > USER to GROUP mapping (hdfs_user -> hadoop_group) should be stack driven > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java > 5c4f08e > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > b601893 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 3b88ac9 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java > 8a35c98 > > ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java > 978ecd7 > > ambari-server/src/main/java/org/apache/ambari/server/state/PropertyInfo.java > 34c2941 > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > 56fcd74 > > ambari-server/src/main/java/org/apache/ambari/server/state/UserGroupInfo.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/ValueAttributesInfo.java > 30a1533 > ambari-server/src/main/resources/configuration-schema.xsd 1019dd9 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py > a748b33 > > ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java > 19df802 > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java > 63a9e9c > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java > c40ff64 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java > 8fad94e > > ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java > 87a1fc7 > ambari-server/src/test/python/stacks/2.0.6/configs/default.json 2a27eca > > ambari-server/src/test/resources/stacks/HDP/0.1/services/HDFS/configuration/hadoop-env.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/56727/diff/ > > > Testing > ------- > > Unit Tests > =========================== > ---------------------------------------------------------------------- > Total run:1174 > Total errors:0 > Total failures:0 > OK > [INFO] > [INFO] --- maven-checkstyle-plugin:2.17:check (checkstyle) @ ambari-server --- > [INFO] Starting audit... > Audit done. > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 21:35 min > [INFO] Finished at: 2017-02-23T09:50:38-08:00 > [INFO] Final Memory: 64M/634M > [INFO] > ------------------------------------------------------------------------ > > > Manual Testing for Zeppelin. > =========================== > Steps: > 1. Make changes to > /resources/common-services/ZEPPELIN/0.6.0.2.5/configuration/zeppelin-env.xml, > /resources\/stacks/HDP/2.6/services/ZEPPELIN/configuration/zeppelin-env.xml > ```xml > <property> > <name>zeppelin_user</name> > <value>zeppelin</value> > <property-type>USER</property-type> > <description>User zeppelin daemon runs as</description> > <value-attributes> > <type>user</type> > <overridable>false</overridable> > <user-group> > <property> > <type>zeppelin-env</type> > <name>zeppelin_group</name> > </property> > <property> > <type>hadoop-env</type> > <name>proxyuser_group</name> > </property> > </user-group> > </value-attributes> > <on-ambari-upgrade add="true"/> > </property> > ``` > > 2. Deploy a cluster with Zeppelin > > 3. Check commands-json for addition of user_group in hostLevelParams for > following scenarios > a. Start Zeppelin - AmbariManagementControllerImpl code path > b. Restart Zeppelin - AmbariCustomCommandExecutionHelper code path > c. ClientConfigResourceProviderTest - ClientConfigResourceProvider code > path > > [root@mradha22-1 data]# grep user_group command-552.json > "user_group": "{\"zeppelin\":[\"zeppelin\",\"users\"]}", > > 4. Verify if the user_group map is getting appended to the existing > user_group_dict > Comment the hard-coded logic for zeppelin user and group in > stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py > #if has_zeppelin_master: > #user_to_groups_dict[zeppelin_user] = [zeppelin_group, user_group] > #zeppelin_user = > config['configurations']['zeppelin-env']["zeppelin_user"] > #zeppelin_group = > config['configurations']['zeppelin-env']["zeppelin_group"] > > [root@mradha22-1 data]# grep Madhu output-*.txt > output-502.txt\:Madhu defaultdict(<function <lambda> at 0x13880c8>, > {'ambari-qa': ['users'], 'zeppelin': ['zeppelin', 'users']}) > 2017-02-15 22:21:48,575 - User['hdfs'] {'gid': 'hadoop', > 'fetch_nonlocal_groups': True, 'groups': ['hadoop']} > 2017-02-15 22:21:48,576 - User['zeppelin'] {'gid': 'hadoop', > 'fetch_nonlocal_groups': True, 'groups': ['zeppelin', 'users']} > > > Thanks, > > Madhuvanthi Radhakrishnan > >
