Release/auditor 10.9 revise Oracle Database configuration and requirements#1000
Draft
pavelshabanov2025 wants to merge 10 commits into
Draft
Release/auditor 10.9 revise Oracle Database configuration and requirements#1000pavelshabanov2025 wants to merge 10 commits into
pavelshabanov2025 wants to merge 10 commits into
Conversation
719ece8 to
4417966
Compare
b97915f to
61b6ae9
Compare
Two new pages covering State-in-Time report capabilities and the configuration steps for enabling State-in-Time data collection in Azure Files monitoring plans.
2cec009 to
bc57fc1
Compare
…0.9 [US 439213] (#1083) * docs(auditor): add Group.Read.All API permission to AFA Azure Files 10.9 [US 439213] Port Group.Read.All permission requirement from Auditor 10.8 (commit 2656597) to the 10.9 documentation. Required by the AFA collector to resolve groups and search by SID from DACLs, verified by live SIT testing. Generated with AI Co-Authored-By: Claude Code <ai@netwrix.com> * docs(auditor): apply 4 style fixes from PR #1047 review comments - Remove "used in the points above" from opening Note (wordy phrase) - Fix passive voice + missing article in Prerequisites ("is configured for data storage account" → "configured for the data storage account") - Remove passive "You should" from Azure App Registration intro, add trailing period - Remove redundant "find and" from IAM Roles Step 1 search instruction Generated with AI Co-Authored-By: Claude Code <ai@netwrix.com> --------- Co-authored-by: Claude Code <ai@netwrix.com>
Contributor
Code ReviewReviewed the non-content changes ( Findings:
Non-blocking note (not a code issue): The PR title/description describe a focused Oracle-configuration revision, but the diff actually introduces the entire new 10.9 product version (~950 files, +37k lines). Worth confirming this matches the intended scope before merge, since reviewers reading just the description may not realize a full version is being added. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
permissions.md: restructured into Prerequisites, numbered steps, DBA Role section, and Required Object Privileges table; removed "Step N" format and 12c referencesdatabase.mdStep 3: restored HTML table formatting for Unified Auditing commandsoverview.md: removed automatic configuration option, removed duplicate CAUTION block, removed 11g/12c remnantsverifysettings.md: split Oracle 19c row to reflect column rename in 19c (ENTITY_NAME/ENABLED_OPTIONvsUSER_NAME/ENABLED_OPTfor 12.2/18c); removed Oracle 11g rowunified.md: irrelevant for 10.9 since minimum supported Oracle version is 12.2, which has Unified Auditing nativelysoftware.mdandsupporteddatasources.md: removed Oracle 11g and 12c references, replaced with 12.2Generated with AI
Co-Authored-By: Claude Code ai@netwrix.com