----------------------------------------------------------- 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