----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29474/#review66386 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_master.py <https://reviews.apache.org/r/29474/#comment109854> We should use named imports whenever possible instead of * ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params.py <https://reviews.apache.org/r/29474/#comment109855> Should perhaps initialize this to False despite the userInput value. ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/ranger_admin.py <https://reviews.apache.org/r/29474/#comment109865> This file is copied several times for different services and would be hard to maintain consistent. Is there an alternative so that it only exists once? ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/ranger_admin.py <https://reviews.apache.org/r/29474/#comment109859> For function names, we typically follow the PEP 8 guide, e.g., get_repository_by_name_pycurl ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/ranger_admin.py <https://reviews.apache.org/r/29474/#comment109856> Should the username and password be hardcoded? ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/ranger_admin.py <https://reviews.apache.org/r/29474/#comment109857> Ideally, should get the return code instead of searching for text in the response. ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/setup_ranger_hbase.py <https://reviews.apache.org/r/29474/#comment109860> Instead of using hdpVersion, should use the pointer in /usr/hdp/current ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode.py <https://reviews.apache.org/r/29474/#comment109863> Prefer named imports over * - Alejandro Fernandez On Dec. 30, 2014, 5:17 a.m., Gautam Borad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29474/ > ----------------------------------------------------------- > > (Updated Dec. 30, 2014, 5:17 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin > Jetly, and Yusaku Sako. > > > Bugs: AMBARI-8949 > https://issues.apache.org/jira/browse/AMBARI-8949 > > > Repository: ambari > > > Description > ------- > > Add Ranger support to Ambari for Installation/Management/Monitoring. > > Supports of the following Ranger components: > 1] Ranger Admin > 2] Ranger User Sync > 3] Ranger Plugins ( currently HDFS, Hive, HBase, Knox, Storm) > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/configuration/ranger-hbase-plugin-properties.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/configuration/ranger-hbase-plugin.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/metainfo.xml > 18572ef > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_master.py > c0e84b4 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params.py > 23c2f6c > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/ranger_admin.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/setup_ranger_hbase.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/ranger-hdfs-plugin-properties.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/ranger-hdfs-plugin.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/metainfo.xml > ce0ab29 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode.py > e8dfe16 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params.py > 3a9d074 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/ranger_admin.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/setup_ranger_hdfs.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/ranger-hive-plugin-properties.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/ranger-hive-plugin.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml > db48936 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server.py > 735d59d > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params.py > 8e6eacb > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/ranger_admin.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_ranger_hive.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/configuration/ranger-knox-plugin-properties.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/configuration/ranger-knox-plugin.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/metainfo.xml > ce4448f > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox_gateway.py > f578926 > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params.py > a2b122a > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/ranger_admin.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/setup_ranger_knox.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/configuration/admin-properties.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/configuration/ranger-env.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/configuration/usersync-properties.xml > PRE-CREATION > ambari-server/src/main/resources/common-services/RANGER/0.4.0/metainfo.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_usersync.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/setup_ranger.py > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.2/role_command_order.json > e8bbe32 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/RANGER/metainfo.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/29474/diff/ > > > Testing > ------- > > Performed the following steps to integrate Ranger : > > Ranger Admin & Ranger User Sync > ------------------------------- > 1] Add Ranger related files under > /var/lib/ambari-server/resources/common-services/RANGER/0.4.0/ > 2] Add the right path in file > /var/lib/ambari-server/resources/stacks/HDP/2.2/services/RANGER/metainfo.xml > 3] Add the same path in file > /var/lib/ambari-agent/cache/stacks/HDP/2.2/services/RANGER/metainfo.xml > 4] Restart Ambari Server > 5] Ranger service was available on Ambari UI. > > > Ranger Plugins > -------------- > Same set of steps did not work for Ranger Plugins. To resolve the same, we > had to add the changes in stacks of each component to ambari-server and > ambari-agents cache folders [This part is not included in this attached > patch]. > > > Thanks, > > Gautam Borad > >
