You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2015/03/25 16:26:58 UTC
Review Request 32480: Storm secure config not visible through Ambari
storm config UI
-----------------------------------------------------------
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
Re: Review Request 32480: Storm secure config not visible through
Ambari storm config UI
Posted by Vitalyi Brodetskyi <vb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32480/#review77741
-----------------------------------------------------------
Ship it!
Ship It!
- Vitalyi Brodetskyi
On Березень 25, 2015, 3:26 після полудня, Andrew Onischuk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32480/
> -----------------------------------------------------------
>
> (Updated Березень 25, 2015, 3:26 після полудня)
>
>
> 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
>
>