> On Sept. 21, 2017, 3:02 a.m., Attila Magyar wrote: > > ambari-server/src/main/resources/stacks/PERF/1.0/services/GRUMPY/kerberos.json > > Lines 69 (patched) > > <https://reviews.apache.org/r/62449/diff/1/?file=1831702#file1831702line69> > > > > is it ok to not have backslash here?
It is technically ok to not have a backslash there. However, in this case, the referance would not resolve to anything and thus it is a bug. Nice catch. - Robert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/#review185846 ----------------------------------------------------------- On Sept. 21, 2017, 7:56 a.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62449/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2017, 7:56 a.m.) > > > Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene > Chekanskiy, Jonathan Hurley, Laszlo Puskas, Nate Cole, and Sebastian Toader. > > > Bugs: AMBARI-21919 > https://issues.apache.org/jira/browse/AMBARI-21919 > > > Repository: ambari > > > Description > ------- > > Kerberos identity references should use the "reference" attribute rather than > rely on the "name" attribute to indicate the identity descriptor references > some other identity descriptor. > > Either method should work on the backend, however the UI appears to not fully > handle the "named" reference properly. > > The solution is to change > ``` > { > "name": "/HDFS/NAMENODE/namenode_nn", > "principal": { > "configuration": > "ranger-hdfs-audit/xasecure.audit.jaas.Client.option.principal" > }, > "keytab": { > "configuration": > "ranger-hdfs-audit/xasecure.audit.jaas.Client.option.keyTab" > } > } > ``` > > by changing the "name" attribute to "reference" and adding a new "name" > reference with a unique name relative to the scope of the identity > descriptor. For example: > > ``` > { > "name":"hdfs_namenode_namenode_nn" > "reference": "/HDFS/NAMENODE/namenode_nn", > "principal": { > "configuration": > "ranger-hdfs-audit/xasecure.audit.jaas.Client.option.principal" > }, > "keytab": { > "configuration": > "ranger-hdfs-audit/xasecure.audit.jaas.Client.option.keyTab" > } > } > ``` > > # NOTE: > My apologies for the large patch. However, most of the changes are to > `kerberos.json` files where there are one or more changes like: > > ``` > "name": "/HDFS/NAMENODE/namenode_nn", > ``` > to > ``` > "name": "hdfs_namenode_namenode_nn" > "reference": "/HDFS/NAMENODE/namenode_nn", > ``` > > There are also a few other file changes, related to upgrading to Ambari > 2.6.0, that may be more interesting to review: > - > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/AbstractKerberosDescriptorContainer.java > > - > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/KerberosDescriptor.java > - > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > - ambari-server/src/main/resources/kerberos_descriptor_schema.json > - > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog260Test.java > > > Diffs > ----- > > > ambari-funtest/src/test/resources/stacks/HDP/2.0.8/services/HDFS/kerberos.json > a555bb5b90 > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/AbstractKerberosDescriptorContainer.java > 85d4f69ed8 > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/KerberosDescriptor.java > a1d373bad6 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > c92b3dc4d3 > > ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/kerberos.json > 7e3f449b74 > > ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/kerberos.json > 94b1b14f8c > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/kerberos.json > a6afda5fbe > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/kerberos.json > 4fe4d3272f > > ambari-server/src/main/resources/common-services/ATLAS/0.7.0.2.5/kerberos.json > e136bcf8e6 > > ambari-server/src/main/resources/common-services/ATLAS/0.7.0.3.0/kerberos.json > 7d10ccc667 > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/kerberos.json > 08af720165 > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/kerberos.json > 4ed6b7871d > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/kerberos.json > 63e929883c > > ambari-server/src/main/resources/common-services/HBASE/2.0.0.3.0/kerberos.json > 011921b192 > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/kerberos.json > 1cf1603d47 > > ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/kerberos.json > b5acf92f3f > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json > c34026ae81 > > ambari-server/src/main/resources/common-services/HIVE/2.1.0.3.0/kerberos.json > af1130dbd1 > > ambari-server/src/main/resources/common-services/KAFKA/0.10.0.3.0/kerberos.json > b4d0018e8e > ambari-server/src/main/resources/common-services/KAFKA/0.10.0/kerberos.json > b4d0018e8e > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerberos.json > 247a602c71 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/kerberos.json > 6ab76106b6 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-30/kerberos.json > 6ab76106b6 > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.3.0/kerberos.json > 2d8aa0dc52 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/kerberos.json > 9e0f12d48e > > ambari-server/src/main/resources/common-services/MAHOUT/1.0.0.2.3/kerberos.json > 1f87a6c81f > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/kerberos.json > b3f932baf6 > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.3/kerberos.json > d2e2ab8a10 > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.3.0/kerberos.json > f1092f552b > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/kerberos.json > 22dd6cb59f > > ambari-server/src/main/resources/common-services/PIG/0.16.1.3.0/kerberos.json > 22dd6cb59f > ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json > 1fc8acf9c6 > > ambari-server/src/main/resources/common-services/RANGER/1.0.0.3.0/kerberos.json > 1fc8acf9c6 > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/kerberos.json > 208a04ddeb > > ambari-server/src/main/resources/common-services/RANGER_KMS/1.0.0.3.0/kerberos.json > 8bf4cd832e > > ambari-server/src/main/resources/common-services/SLIDER/0.60.0.2.2/kerberos.json > 0ebeb7db6b > > ambari-server/src/main/resources/common-services/SLIDER/0.91.0.3.0/kerberos.json > 0ebeb7db6b > ambari-server/src/main/resources/common-services/SPARK/1.2.1/kerberos.json > 96df126939 > ambari-server/src/main/resources/common-services/SPARK/1.4.1/kerberos.json > fb436578ba > ambari-server/src/main/resources/common-services/SPARK/2.2.0/kerberos.json > 9912af101d > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/kerberos.json > a787d3320d > ambari-server/src/main/resources/common-services/STORM/0.9.1/kerberos.json > 692b557713 > > ambari-server/src/main/resources/common-services/STORM/1.0.1.3.0/kerberos.json > a034411517 > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json > a034411517 > ambari-server/src/main/resources/common-services/STORM/1.1.0/kerberos.json > a034411517 > > ambari-server/src/main/resources/common-services/TEZ/0.4.0.2.1/kerberos.json > 65dcd3366f > > ambari-server/src/main/resources/common-services/TEZ/0.9.0.3.0/kerberos.json > 00c8ac5882 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/kerberos.json > 7315c093a0 > > ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/kerberos.json > b1501b80d8 > > ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0/kerberos.json > b605c9de12 > > ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/kerberos.json > b605c9de12 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/kerberos.json > 0a64ea526e > ambari-server/src/main/resources/kerberos_descriptor_schema.json 34cdd497bb > ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/kerberos.json > 8618804e0b > > ambari-server/src/main/resources/stacks/HDP/2.3.ECS/services/ECS/kerberos.json > 3e8451d467 > > ambari-server/src/main/resources/stacks/HDP/2.3.ECS/services/HBASE/kerberos.json > 5dfcb5e30a > > ambari-server/src/main/resources/stacks/HDP/2.3.ECS/services/YARN/kerberos.json > 4c5bcdb8cb > > ambari-server/src/main/resources/stacks/HDP/2.3.GlusterFS/services/ACCUMULO/kerberos.json > 04ef946ffb > > ambari-server/src/main/resources/stacks/HDP/2.3/services/ACCUMULO/kerberos.json > cbfac42a29 > ambari-server/src/main/resources/stacks/HDP/2.3/services/TEZ/kerberos.json > ee7c7cff49 > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/kerberos.json > 0e7a5de80d > > ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/kerberos.json > 0c4507d7ab > > ambari-server/src/main/resources/stacks/HDP/2.5/services/HBASE/kerberos.json > 011921b192 > ambari-server/src/main/resources/stacks/HDP/2.5/services/HDFS/kerberos.json > 766a0143dc > ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/kerberos.json > 044bd653b7 > ambari-server/src/main/resources/stacks/HDP/2.5/services/KNOX/kerberos.json > 2d8aa0dc52 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/RANGER_KMS/kerberos.json > 8bf4cd832e > > ambari-server/src/main/resources/stacks/HDP/2.5/services/SPARK/kerberos.json > 54ddf027c6 > ambari-server/src/main/resources/stacks/HDP/2.5/services/YARN/kerberos.json > 29cc00a3e7 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/ZEPPELIN/kerberos.json > b605c9de12 > > ambari-server/src/main/resources/stacks/HDP/2.6/services/ATLAS/kerberos.json > 7d10ccc667 > > ambari-server/src/main/resources/stacks/HDP/2.6/services/DRUID/kerberos.json > 5b6b5737a7 > ambari-server/src/main/resources/stacks/HDP/2.6/services/HDFS/kerberos.json > b5acf92f3f > > ambari-server/src/main/resources/stacks/HDP/2.6/services/OOZIE/kerberos.json > f1092f552b > > ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK/kerberos.json > c68d627201 > > ambari-server/src/main/resources/stacks/HDP/2.6/services/SPARK2/kerberos.json > 8b6b41d678 > > ambari-server/src/main/resources/stacks/HDP/2.6/services/SUPERSET/kerberos.json > b79bac9805 > ambari-server/src/main/resources/stacks/HDP/2.6/services/YARN/kerberos.json > 4b9472b8ab > > ambari-server/src/main/resources/stacks/HDP/2.6/services/ZEPPELIN/kerberos.json > b605c9de12 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/FAKEHBASE/kerberos.json > f1026f1a02 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/FAKEHDFS/kerberos.json > 412950610a > > ambari-server/src/main/resources/stacks/PERF/1.0/services/FAKEYARN/kerberos.json > b960dce6eb > > ambari-server/src/main/resources/stacks/PERF/1.0/services/FAKEZOOKEEPER/kerberos.json > 0a64ea526e > > ambari-server/src/main/resources/stacks/PERF/1.0/services/GRUMPY/kerberos.json > 335d9f3440 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/HAPPY/kerberos.json > 55389b0894 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/KERBEROS/kerberos.json > 6ab76106b6 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/SLEEPY/kerberos.json > f22274f5c8 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/SNOW/kerberos.json > 7be70f7c67 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog260Test.java > c03b400c70 > > ambari-server/src/test/resources/kerberos/test_kerberos_descriptor_ranger_kms.json > d7e048fb79 > > ambari-server/src/test/resources/stacks/HDP/2.0.8/services/HDFS/kerberos.json > a555bb5b90 > > contrib/management-packs/hdf-ambari-mpack/src/main/resources/common-services/NIFI/1.0.0/kerberos.json > 74d84e6305 > > contrib/management-packs/hdf-ambari-mpack/src/main/resources/stacks/HDF/2.0/services/KAFKA/kerberos.json > aa351d1fdc > > contrib/management-packs/microsoft-r_mpack/src/main/resources/common-services/MICROSOFT_R_SERVER/8.0.5/kerberos.json > 6e4aa4caa0 > > contrib/management-packs/odpi-ambari-mpack/src/main/resources/stacks/ODPi/2.0/services/HIVE/kerberos.json > 4b31f7ab15 > > contrib/management-packs/odpi-ambari-mpack/src/main/resources/stacks/ODPi/2.0/services/YARN/kerberos.json > 4093431ecb > > > Diff: https://reviews.apache.org/r/62449/diff/2/ > > > Testing > ------- > > Tested new Ambari 2.6.0 cluster. > Test upgrade from Ambari 2.5.1 to Ambari 2.6.0 > > # Local test results: > ``` > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 31:25 min > [INFO] Finished at: 2017-09-20T13:11:03-04:00 > [INFO] Final Memory: 89M/2007M > [INFO] > ------------------------------------------------------------------------ > ``` > > # Jenkins test results: PENDING > > > Thanks, > > Robert Levas > >