-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50409/
-----------------------------------------------------------
Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.
Bugs: AMBARI-17894
https://issues.apache.org/jira/browse/AMBARI-17894
Repository: ambari
Description
-------
STR:
- Install Ambari 2.2.2 with Hive on HDP 2.4.2.0 and Kerberize
- Upgrade to Ambari 2.4.0
- Upgrade to HDP 2.5
- Distribute Keytabs
- Add a new service
At this point, the UI flags Hive as having configurations which need attention.
The follow are all blank and are marked as required:
{code}
hive-site/hive.server2.authentication.ldap.url
hiveserver2-site/hive.conf.restricted.list
hiveserver2-site/hive.security.authenticator.manager
hiveserver2-site/hive.security.authorization.manager
{code}
There are actually two problems here:
- The Kerberos wizard interprets the stack advisor "delete" attribute and
improperly sets config properties to blank instead of actually removing them
- The upgrade logic is adding properties back when it should not.
The Kerberos issue aside, we can't be adding properties back during upgrade
stack merging when those properties were specifically removed by the stack
advisor prior.
For example (Ambari 2.2.2 installed with HDP 2.x)
- Ambari 2.2.2 does not have {{foo-site/property}} for HDP 2.x
- Ambari 2.4.0 adds {{foo-site/property}} for HDP 2.x, but instructs the
upgrade not to add it
- An upgrade to HDP 2.y sees that {{foo-site/property}} doesn't exist and
thinks it's brand new and needs to be merged.
The upgrade logic should check to see if {{foo-site/property}} is part of both
2.x and 2.y default configurations. If it's part of both of them and is not
currently set, then upgrade should NOT set it.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
55e9dbd
ambari-server/src/main/resources/stacks/HDP/2.2/services/HIVE/configuration/hiveserver2-site.xml
ad37114
Diff: https://reviews.apache.org/r/50409/diff/
Testing
-------
PENDING UNIT TESTS
Thanks,
Jonathan Hurley
