-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40810/
-----------------------------------------------------------

(Updated Dec. 1, 2015, 10:53 p.m.)


Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan 
Hurley, Nate Cole, and Swapan Shridhar.


Changes
-------

Added logging and unit test.


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/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

Reply via email to