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




ambari-server/src/main/resources/stacks/stack_advisor.py
Line 2873 (original), 2872 (patched)
<https://reviews.apache.org/r/59459/#comment249175>

    Need to declare this,
    in the try block after klass_set = ...
    klass_list = [("'" + e + "'") for e in klass_set]
    
    In the exception handler,
    klass_list = [("'" + e + "'") for e in list_classes]


- Alejandro Fernandez


On May 23, 2017, 8:32 p.m., Sriharsha Chintalapani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59459/
> -----------------------------------------------------------
> 
> (Updated May 23, 2017, 8:32 p.m.)
> 
> 
> Review request for Ambari and Alejandro Fernandez.
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> AMBARI-21045. Enable Storm's AutoTGT configs in secure mode.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog251.java
>  9255daf76b 
>   
> ambari-server/src/main/resources/common-services/STORM/1.1.0/configuration/storm-env.xml
>  PRE-CREATION 
>   
> ambari-server/src/main/resources/common-services/STORM/1.1.0/configuration/storm-site.xml
>  1a5dde9c16 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/config-upgrade.xml 
> 8b5c07dc90 
>   
> ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.6.xml
>  5aa08c5369 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.6.xml 
> d98bb53a2a 
>   ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/config-upgrade.xml 
> b3d19d42e1 
>   
> ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.6.xml
>  4a2a502a20 
>   ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.6.xml 
> 1eb9836abb 
>   ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/config-upgrade.xml 
> a29f74b668 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.6.xml
>  8c659ee469 
>   ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/upgrade-2.6.xml 
> 3054ca30c1 
>   ambari-server/src/main/resources/stacks/HDP/2.6/services/stack_advisor.py 
> 488562bf11 
>   ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-upgrade.xml 
> 1610bb5580 
>   
> ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/nonrolling-upgrade-2.6.xml
>  1cdd184529 
>   ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/upgrade-2.6.xml 
> 3e7e3d7f14 
>   ambari-server/src/main/resources/stacks/stack_advisor.py 4a81dc6c63 
> 
> 
> Diff: https://reviews.apache.org/r/59459/diff/7/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sriharsha Chintalapani
> 
>

Reply via email to