RANGER-5482:Create Ranger Audit Server with SOLR and HDFS as audit consumer#847
RANGER-5482:Create Ranger Audit Server with SOLR and HDFS as audit consumer#847
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a comprehensive Ranger Audit Server architecture that decouples audit collection from Ranger Admin by implementing:
- A centralized Audit Server that receives audits from plugins via REST API and produces them to Kafka
- Separate consumer services for SOLR and HDFS that consume from Kafka and write to their respective destinations
- Shared common infrastructure for Kafka consumer management with support for consumer groups and rebalancing
Changes:
- New audit server infrastructure with 3 microservices (audit-server, consumer-solr, consumer-hdfs) and shared common module
- Integration with existing Ranger plugins (HDFS, Hive) to support HTTP-based audit destination
- Docker support for testing the complete audit pipeline with Kerberos authentication
- Thread-safe audit handling with concurrent counter management
Reviewed changes
Copilot reviewed 110 out of 111 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| ranger-audit-server/scripts/* | Shell scripts for starting/stopping audit services |
| ranger-audit-server/ranger-audit-server-service/* | Core audit server service with REST API and Kafka producer |
| ranger-audit-server/ranger-audit-consumer-solr/* | SOLR consumer service with Kafka-to-SOLR pipeline |
| ranger-audit-server/ranger-audit-consumer-hdfs/* | HDFS consumer service with Kafka-to-HDFS pipeline |
| ranger-audit-server/ranger-audit-common/* | Shared utilities, base classes, and consumer registry |
| agents-audit/dest-auditserver/* | New audit destination for plugins to send to audit server |
| agents-common/src/main/java/org/apache/ranger/plugin/audit/* | Enhanced audit handler with service type in additional info |
| agents-audit/core/src/main/java/org/apache/ranger/audit/provider/* | Thread-safe counters using AtomicLong |
| dev-support/ranger-docker/* | Docker compose and Dockerfiles for audit server services |
| pom.xml, distro/* | Build configuration and assembly descriptors |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…nsumer - fix failing testing
…nsumer - fix pmd issue
…onsumer - Fix audit commit failure propagation and recovery in the consumers
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 110 out of 111 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…nsumer - audit server partition management enhancement
mneethiraj
left a comment
There was a problem hiding this comment.
@rameeshm - sending comments of the portions I reviewed so far. Will continue reviewing.
...itserver/src/main/java/org/apache/ranger/audit/destination/RangerAuditServerDestination.java
Outdated
Show resolved
Hide resolved
...itserver/src/main/java/org/apache/ranger/audit/destination/RangerAuditServerDestination.java
Outdated
Show resolved
Hide resolved
...itserver/src/main/java/org/apache/ranger/audit/destination/RangerAuditServerDestination.java
Outdated
Show resolved
Hide resolved
...itserver/src/main/java/org/apache/ranger/audit/destination/RangerAuditServerDestination.java
Outdated
Show resolved
Hide resolved
agents-audit/core/src/main/java/org/apache/ranger/audit/provider/BaseAuditHandler.java
Outdated
Show resolved
Hide resolved
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
…nsumer - Fix review comments
…nsumer - PojoMappingFeature for AuditEvent Object for serialization
…onsumer - Fix review comments set #2
…nsumer - Audit Batch processing and failure reprocessing improvement
...server/ranger-audit-server-service/src/main/java/org/apache/ranger/audit/rest/AuditREST.java
Outdated
Show resolved
Hide resolved
…onsumer - Fix duplicate dependency error in the pom for sl4j
…nsumer - Fix ubuntu audit ranger module war file creation failure
| return ret; | ||
| } | ||
|
|
||
| if (!serviceName.equals(authenticatedUser)) { |
There was a problem hiding this comment.
Requiring serviceName same as the authenticatedUser doesn't seem right. I guess the intention is that the caller is the identity of the service account for the specified serviceName. This identity should be obtained from the service configuration retrieved from Ranger admin - for example configurations service.admin.users. If necessary, we can introduce a new configuration service.account.user for this purpose.
What changes were proposed in this pull request?
Audit Server should have service which has kafka as a queue mechanism and have Kafka Producer to relay and store audits in Kafka Topic / Partition.
Audit Server should have Solr consumer running as a separate service to write audit into SOLR Index as a destination
Audit Server should have HDFS consumer running as a separate service to write audit into HDFS.
All the above mentioned service should be available and test in docker container.
How was this patch tested?