-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40810/
-----------------------------------------------------------
(Updated Dec. 2, 2015, 12:17 a.m.)
Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan
Hurley, Nate Cole, and Swapan Shridhar.
Bugs: AMBARI-14121
https://issues.apache.org/jira/browse/AMBARI-14121
Repository: ambari
Description
-------
When performing an Express Upgrade from HDP 2.2 to 2.3, Ambari authors the
configs as "_anonymous" instead of the authenticate user.
This is because the server side action UpdateDesiredStackAction.java calls
upgradeResourceProvider.applyStackAndProcessConfigurations but doesn't pass it
the username.
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
748dbbe
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/UpdateDesiredStackAction.java
44b13de
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
3bf6cad
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
93360bd
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
38370c7
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/config-upgrade.xml
6a2986e
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_nonrolling_new_stack.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/40810/diff/
Testing
-------
Waiting for unit test results.
Ran manually for UpgradeResourceProviderTest.java and it passed.
Verified during an Express Upgrade from HDP 2.2 to 2.3
```
ambari=> select service_name, version, user_name from serviceconfig;
service_name | version | user_name
--------------+---------+-----------
HDFS | 1 | admin
ZOOKEEPER | 1 | admin
MAPREDUCE2 | 1 | admin
YARN | 1 | admin
ZOOKEEPER | 2 | admin
MAPREDUCE2 | 2 | admin
HDFS | 2 | admin
YARN | 2 | admin
```
Thanks,
Alejandro Fernandez
