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

Review request for Ambari, Balázs Bence Sári, Jonathan Hurley, and Nate Cole.


Bugs: AMBARI-22025
    https://issues.apache.org/jira/browse/AMBARI-22025


Repository: ambari


Description
-------

Execution commands embedded in status commands are created programmatically 
instead of deserializing JSON.  AMBARI-21361 centralized addition of 
`service_package_folder` and `hooks_folder`, but the logic is not executed in 
this case.

Fix: reuse the logic (extracted to the method `setVersions`) which adds 
`service_package_folder` and a few other parameters.

(patch is for branch-2.6)


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
 7b202598d5bb88d78dc8486ed2801f87efb326a8 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 96ed2613888c1badc50c21f540868444e5698b51 


Diff: https://reviews.apache.org/r/62465/diff/1/


Testing
-------

* Tested manually on local cluster per STR.
* Unit tests: `Tests run: 4909, Failures: 0, Errors: 0, Skipped: 34`


Thanks,

Attila Doroszlai

Reply via email to