paras200 commented on code in PR #986: URL: https://github.com/apache/ranger/pull/986#discussion_r3448308640
########## security-admin/scripts/install.properties: ########## @@ -101,6 +101,15 @@ audit_elasticsearch_password= audit_elasticsearch_index= audit_elasticsearch_bootstrap_enabled=true +# * OpenSearch audit store properties (when audit_store=opensearch) Review Comment: Yes — install.properties is the operator-facing input; setup.sh reads audit_opensearch_* from here and substitutes them into ranger-admin-site.xml at install time (setup.sh:105–111 → setup.sh:877–904, gated on audit_store=opensearch). This mirrors how audit_elasticsearch_* / audit_solr_* already work — the entries in ranger-admin-site.xml are defaults that get overwritten. Dropping them here would remove the ability to configure the store at install. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
