-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43640/
-----------------------------------------------------------
(Updated Feb. 17, 2016, 7:25 p.m.)
Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan
Hurley, and Nate Cole.
Changes
-------
Addressed comments.
Bugs: AMBARI-15064
https://issues.apache.org/jira/browse/AMBARI-15064
Repository: ambari
Description
-------
RU/EU tries to resolve the hostnames for NameNode, HBASE Master, and
ResourceManager by using MasterHostResolver.java, which queries configs and
then JMX.
Ambari DB stores the hostnames in all lowercase, so if the configs or JMX store
the hostnames in a different case, then Ambari will throw an exception that the
host could not be found.
To fix this, convert all hostnames to lowercase.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariActionExecutionHelper.java
795dfa7
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
70440fc
ambari-server/src/main/java/org/apache/ambari/server/stack/MasterHostResolver.java
561350b
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
eb5bf62
Diff: https://reviews.apache.org/r/43640/diff/
Testing
-------
Ran unit tests in UpgradeHelperTest.java
Verified that was able to start Express Upgrade.
Waiting for full set of unit tests to run.
Thanks,
Alejandro Fernandez
