-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54552/
-----------------------------------------------------------
(Updated Dec. 12, 2016, 1:37 p.m.)
Review request for Ambari, Attila Magyar, Laszlo Puskas, Sumit Mohanty, and
Thejas Nair.
Bugs: AMBARI-19133
https://issues.apache.org/jira/browse/AMBARI-19133
Repository: ambari
Description
-------
`hadoop.proxyuser.HTTP.hosts` should not be updated when Hive is installed
unless WebHcat is installed.
This is happening because the following block in the Kerberos descriptor is at
the HIVE service level rather than the WEBHCAT_SERVER component level.
```
{
"core-site": {
"hadoop.proxyuser.HTTP.hosts":
"${clusterHostInfo/webhcat_server_host}"
}
},
```
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java
d472b79
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json
872bfac
ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/kerberos.json
34bda73
ambari-server/src/test/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelperTest.java
90f7022
Diff: https://reviews.apache.org/r/54552/diff/
Testing (updated)
-------
Manually tested
# Local test results:
```
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 31:18.138s
[INFO] Finished at: Mon Dec 12 10:36:39 EST 2016
[INFO] Final Memory: 68M/858M
[INFO] ------------------------------------------------------------------------
```
# Jenkins test results:
```
+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12842810/AMBARI-19133_trunk_02.patch
against trunk revision .
+1 @author. The patch does not contain any @author tags.
+1 tests included. The patch appears to include 1 new or modified test files.
+1 javac. The applied patch does not increase the total number of javac
compiler warnings.
+1 release audit. The applied patch does not increase the total number of
release audit warnings.
+1 core tests. The patch passed unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/9625//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/9625//console
```
Thanks,
Robert Levas
