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

Review request for Ambari, Miklos Gergely and Robert Nettleton.


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


Repository: ambari


Description
-------

add 2 new commands for solr-client:
1. secure znode: set 'cdrwa' acl for specific sasl users, and set 'r' acl for 
world.
2. secure solr znode: set 'cdrwa' acl for specific sasl users (solr), but skip 
znode/configs and znode/collections znodes (set those to 'cr' acl)

the patch does not contain the usage of these commands. also i removed some 
unused commands.

- the second command will be used inside infra-solr ambari integration code, 
during start
- in logsearch, acls will be setup inside the logsearch code instead of use it 
in ambari integration code.
- for atlas/ranger it will be used inside ambari integration code, later it 
would be better to use inside atlas/ranger code (same as for logsearch)


Diffs
-----

  
ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py
 521c9c6 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudCLI.java
 6035c86 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudClient.java
 20096fe 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/AmbariSolrCloudClientBuilder.java
 bad622b 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/commands/CopyZnodeZkCommand.java
 a202930 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/commands/CreateSaslUsersZkCommand.java
 1c25b9f 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/commands/SecureZNodeZkCommand.java
 cdc3ebf 
  
ambari-logsearch/ambari-logsearch-solr-client/src/main/java/org/apache/ambari/logsearch/solr/util/AclUtils.java
 23e5fb3 

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


Testing
-------

testing done. (manually)


Thanks,

Oliver Szabo

Reply via email to