Skip to content

RANGER-4910: update Polaris service-def to remove resources principal-role and catalog-role#866

Merged
mneethiraj merged 1 commit intoapache:masterfrom
mneethiraj:RANGER-4910
Mar 3, 2026
Merged

RANGER-4910: update Polaris service-def to remove resources principal-role and catalog-role#866
mneethiraj merged 1 commit intoapache:masterfrom
mneethiraj:RANGER-4910

Conversation

@mneethiraj
Copy link
Contributor

What changes were proposed in this pull request?

  • removed resources principal-role and catalog-role, and all permissions associated with them

How was this patch tested?

  • verified that the updated service-def is loaded successfully in Ranger

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the embedded Apache Ranger service definition for the Polaris service type to remove the principal-role and catalog-role resource types (and associated permissions), simplifying the authorization surface for Polaris within Ranger.

Changes:

  • Removed principal-role and catalog-role resources and their related access types.
  • Removed several grant-management related access types and catalog-access-manage.
  • Reorganized/renumbered itemId values for remaining resources/access types.
Comments suppressed due to low confidence (1)

agents-common/src/main/resources/service-defs/ranger-servicedef-polaris.json:90

  • The PR renumbers existing resources[*].itemId values (e.g., table is now itemId 5). Ranger service-def updates match existing resource defs by itemId (not name) during update; changing itemIds can cause resources to be treated as deleted/recreated or make updates fail if policies reference the old defs. Consider keeping all existing itemIds stable and only removing the principal-role/catalog-role entries (leave remaining itemIds unchanged). See security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java:3308-3355 where resource defs are matched/removed by itemId.
    {
      "itemId":      5,
      "name":        "table",
      "label":       "Table",
      "description": "Table",
      "parent":      "namespace",

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mneethiraj mneethiraj merged commit a7746d7 into apache:master Mar 3, 2026
8 checks passed
@mneethiraj mneethiraj deleted the RANGER-4910 branch March 3, 2026 22:58
mneethiraj added a commit that referenced this pull request Mar 4, 2026
…-role and catalog-role (#866)

(cherry picked from commit a7746d7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants