ramackri commented on code in PR #980:
URL: https://github.com/apache/ranger/pull/980#discussion_r3385523012
##########
distro/src/main/assembly/pdp.xml:
##########
@@ -33,6 +33,7 @@
<include>org.apache.ranger:authz-embedded</include>
<include>org.apache.ranger:ranger-audit-core</include>
<include>org.apache.ranger:ranger-audit-dest-auditserver</include>
+ <include>org.apache.ranger:ranger-audit-dest-solr</include>
Review Comment:
Thanks @mneethiraj — agreed.
The Solr packaging in pdp.xml was added to unblock CI after ranger-pdp
failed with ClassNotFoundException: SolrAuditDestination while Docker still had
ranger.authz.audit.destination.solr=true.
Updated as suggested:
distro/src/main/assembly/pdp.xml — reverted the ranger-audit-dest-solr
module and Solr-only HTTP client deps; PDP keeps only
ranger-audit-dest-auditserver (aligned with RANGER-5632 / master).
dev-support/ranger-docker/scripts/pdp/ranger-pdp-site.xml — disabled Solr
audit destination and enabled auditserver
(url=http://ranger-audit-ingestor.rangernw:7081, with batch spool dir),
matching other Docker plugins.
javax.inject in pdp/pom.xml is unchanged — that addresses the separate
Jersey startup issue.
--
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]
