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

(Updated Jan. 29, 2017, 3:37 p.m.)


Review request for Ambari, Miklos Gergely and Robert Nettleton.


Changes
-------

small fixes


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

Reply via email to