----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55986/#review163266 -----------------------------------------------------------
Ship it! Ship It! - Robert Nettleton On Jan. 26, 2017, 5:48 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55986/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2017, 5:48 p.m.) > > > Review request for Ambari, Miklos Gergely and Robert Nettleton. > > > 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 > ----- > > > 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/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/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/configs/default.json dd8a096 > > Diff: https://reviews.apache.org/r/55986/diff/ > > > Testing > ------- > > in progress... > > > Thanks, > > Oliver Szabo > >
