| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Add patch 078 and optimized schema updates for all DB backends: seed rangerauditserver with ROLE_ADMIN_AUDITOR and register RangerAuditPartitionPlan global state name for RANGER-5655 admin-managed partition plans.
There was a problem hiding this comment.
Adds DB patch 078 across all supported Ranger databases for audit partition-plan state and machine-user setup.
Changes:
Copilot reviewed 10 out of 10 changed files in this pull request and generated 12 comments.
Show a summary per file| File | Description |
|---|---|
| security-admin/db/sqlserver/patches/078-audit-partition-plan-global-state.sql | Updated as part of this pull request. |
| security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql | Updated as part of this pull request. |
| security-admin/db/sqlanywhere/patches/078-audit-partition-plan-global-state.sql | Updated as part of this pull request. |
| security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql | Updated as part of this pull request. |
| security-admin/db/postgres/patches/078-audit-partition-plan-global-state.sql | Updated as part of this pull request. |
| security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql | Updated as part of this pull request. |
| security-admin/db/oracle/patches/078-audit-partition-plan-global-state.sql | Updated as part of this pull request. |
| security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql | Updated as part of this pull request. |
| security-admin/db/mysql/patches/078-audit-partition-plan-global-state.sql | Updated as part of this pull request. |
| security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql | Updated as part of this pull request. |
security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql:600
`app_data` TEXT NULL DEFAULT NULL,
security-admin/db/mysql/patches/078-audit-partition-plan-global-state.sql:44
ALTER TABLE x_ranger_global_state MODIFY app_data TEXT DEFAULT NULL;
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
Disable JDBC login for rangerauditserver (status=0, empty password) for SPIFFE-only Admin access; fix Oracle app_data ALTER guard; use SQL Anywhere ALTER COLUMN syntax.
… 078. Oracle stores empty string as NULL on NOT NULL PASSWORD; use a space placeholder for rangerauditserver. Remove stale x_policy_ref_user_FK_upd_by CHECK from SQL Server optimized schema (patch 077 regression).
Use LONGTEXT for app_data on MySQL so the partition plan can grow beyond TEXT limits. On Oracle upgrade, migrate VARCHAR2 app_data via add/copy/drop/ rename instead of MODIFY to avoid ORA-22858. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
@ramackri - in addition to topic partition details, it might be necessary to store additional details, such as audit ingestor endpoint, allowed users per service. How about introducing a new table:
x_audit_config(cfg_name string cfg_value string, version long);
insert into x_audit_config(cfg_name, cfg_value) values ("ingestor.url, "https://ranger-audit-ingestor:8765");
insert into x_audit_config(cfg_name, cfg_value) values ("service.hive.allowed.users", "hive");
insert into x_audit_config(cfg_name, cfg_value) values ("topic-partitions", 30);
Sorry, something went wrong.
Per review feedback, store ingestor URL, per-service allowed users, and topic partition count in x_audit_config (patch 078 + optimized schema on all DB backends). Seed defaults idempotently on fresh install and upgrade. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep ingestor URL and per-service allow-list in x_audit_config; partition routing (topic, topicPartitionCount, plugins, buffer) stays in RangerAuditPartitionPlan JSON only. Drop topic-partitions from x_audit_config. Co-authored-by: Cursor <cursoragent@cursor.com>
This reverts commit 3b33734.
Store ingestor URL, topic, partition plan JSON, and per-service allow-list in x_audit_config. Revert patch 078 changes to x_ranger_global_state (no app_data widen, no RangerAuditPartitionPlan row). Keep rangerauditserver machine user setup in patch 078. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Sorry, something went wrong.
|
Sorry, something went wrong.
Now we moved to x_audit_config |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
RANGER-5720 — DB patch 078 for RANGER-5655. Adds the x_audit_config table for audit runtime settings, seeds default config rows, and creates the rangerauditserver machine user. DB-only (no Java); pairs with #1137.
Per review feedback, audit configuration lives in x_audit_config, not x_ranger_global_state. Patch 078 does not widen app_data and does not insert a RangerAuditPartitionPlan global-state row.
Changes
Patch file (all 5 backends): 078-add-x_audit_config.sql (renamed from 078-audit-partition-plan-global-state.sql)
Schema
Seed rows in x_audit_config
User bootstrap
Backend-specific notes
Testing
CI
Docker DB E2E (local, dev-support/ranger-docker)
Scripts under scripts/db-test/:
How we ran upgrade tests
Upgrade path (077→078) simulates production upgrade:
Results (all docker backends, Aug 2026)
What 10/10 means — each check in verify-patch-078.sh:
Tests validate database state after db_setup.py; they do not start Admin UI, ingestor, or plugins.
Harness note: SQL Server upgrade baseline uses master optimized schema with a one-line fix for the orphaned FK CHECK (master bug from patch 077); branch optimized schema includes the same fix.
Related