kumaab commented on code in PR #1058:
URL: https://github.com/apache/ranger/pull/1058#discussion_r3807875003
##########
dev-support/ranger-docker/download-archives.sh:
##########
@@ -163,7 +162,7 @@ else
extractOzoneIfNeeded
elif [[ $arg == 'opensearch' ]]
Review Comment:
Let's remove this if condition, for opensearch - download-archives.sh is not
applicable.
##########
dev-support/ranger-docker/README.md:
##########
Review Comment:
opensearch should be removed here.
##########
dev-support/ranger-docker/README.md:
##########
Review Comment:
This section is not really relevant here, could you please help this move
out to a different readme in the same directory, may be something like
OZONE-ACTION-MATCHER.md, ref:
https://github.com/apache/ranger/commit/37b6e02c4d3b989790bbd669b7d5d4366f7f7644#diff-da6584afb9b693c71ad5427c1a5e1e0720e306e0fbfb664f9fb79c7ce44c29fa
##########
dev-support/ranger-docker/README.md:
##########
@@ -169,8 +172,7 @@ used by the Solr audit path.
##### Setup
-With the default `RANGER_DB_TYPE=postgres`, OpenSearch auditing is
preconfigured and runs
-out of the box — the commands below need no `install.properties` changes.
+Set `RANGER_AUDIT_STORE=opensearch` — no manual `install.properties` edits
needed.
~~~
# Prerequisites: build Ranger artifacts (admin, audit ingestor/dispatcher,
...) and download archives
Review Comment:
L178-182 can be skipped, they are already covered earlier in the setup
process.
--
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]
