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

Review request for Ambari and Vitalyi Brodetskyi.


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


Repository: ambari


Description
-------

When storm security turned on using Ambari its able to write storm security
specific configs to storm.yaml and can be verified by looking at
/etc/storm/conf/storm.yaml but these values are not visible or editable via
Ambari UI. This prevents users from adding additional users to "nimbus.admins"
or change any other config.


Diffs
-----

  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/storm-site.xml
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/metainfo.xml
 ac954e6 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/configuration/storm-site.xml
 bfe1d26 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/kerberos.json 
50b92ff 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/params.py
 0e432ba 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py
 7a2488d 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/yaml_utils.py
 8187666 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/templates/storm.yaml.j2
 cc3e631 
  
ambari-server/src/main/resources/stacks/HDP/2.2/services/STORM/configuration/storm-site.xml
 befb5f8 
  
ambari-server/src/main/resources/stacks/HDPWIN/2.2/services/STORM/configuration/storm-site.xml
 c0528dd 
  ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py 2b475d1 
  ambari-server/src/test/python/stacks/2.1/configs/default-storm-start.json 
900eba4 
  ambari-server/src/test/python/stacks/2.1/configs/secured-storm-start.json 
6b8f69a 

Diff: https://reviews.apache.org/r/32480/diff/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk

Reply via email to