-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55986/
-----------------------------------------------------------
(Updated Jan. 27, 2017, 3:32 p.m.)
Review request for Ambari, Miklos Gergely and Robert Nettleton.
Changes
-------
- turn on the feature
- use file location instead of string content (with that we persists the
security.json)
Bugs: AMBARI-19727
https://issues.apache.org/jira/browse/AMBARI-19727
Repository: ambari
Description
-------
Authorization is missing from ambari-infra whcih can cause issues with other
services. (like if ranger uses solr, if a user could kinit, the user can
CREATE/UPDATE/DELETE docs in collections)
I think here we cannot use ranger solr plugin, its like a chicken egg problem
because we need a running solr to create collection with ranger and start
ranger admin, but if solr uses ranger plugin for authorization, it needs to go
to ranger. (which is not running yet)
at least what we can do: we can make sercurity.json editable, in that case it
can be modified with authorization data if needed.
Diffs (updated)
-----
ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py
3e076b8
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudCLI.java
94e43e4
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudClient.java
6bb5a77
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudClientBuilder.java
590485f
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/commands/EnableKerberosPluginSolrZkCommand.java
fe97994
ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/configuration/infra-solr-security-json.xml
PRE-CREATION
ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/metainfo.xml
b94812b
ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/package/scripts/params.py
c2b8a44
ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/package/scripts/setup_infra_solr.py
46535cd
ambari-server/src/main/resources/common-services/AMBARI_INFRA/0.1.0/properties/infra-solr-security.json.j2
PRE-CREATION
ambari-server/src/test/python/stacks/2.4/AMBARI_INFRA/test_infra_solr.py
6af463c
ambari-server/src/test/python/stacks/2.4/configs/default.json dd8a096
Diff: https://reviews.apache.org/r/55986/diff/
Testing
-------
in progress...
Thanks,
Oliver Szabo