Skip to content

RANGER-5261: Disable Log4J Audit Destination for Ozone in OM container #614

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jul 21, 2025
Merged

RANGER-5261: Disable Log4J Audit Destination for Ozone in OM container #614

merged 2 commits into from
Jul 21, 2025
+3 −2

Conversation

kumaab
Copy link
Contributor

@kumaab kumaab commented Jul 20, 2025 •
edited
Loading

What changes were proposed in this pull request?

Disable Log4J Audit destination for Ozone.

How was this patch tested?

Tested locally in docker containers:

  • plugin installation succeeds and the Audits tab report plugin download for dev_ozone service.
  • authorization for a volume succeeds and is seen in Audits in Ranger UI

@kumaab kumaab self-assigned this Jul 20, 2025
@kumaab kumaab requested a review from mneethiraj July 20, 2025 21:19
vyommani
vyommani approved these changes Jul 21, 2025
View reviewed changes
Copy link
Contributor

@vyommani vyommani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kumaab
Copy link
Contributor Author

kumaab commented Jul 21, 2025

Thanks @mneethiraj and @vyommani for the reviews.

@kumaab kumaab merged commit fbc2624 into ranger-2.7 Jul 21, 2025
3 checks passed
@kumaab kumaab deleted the ranger-5261 branch July 21, 2025 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vyommani vyommani vyommani approved these changes

@mneethiraj mneethiraj mneethiraj approved these changes

Assignees

@kumaab kumaab

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kumaab @vyommani @mneethiraj