Skip to content

Add special case - mdb_admin group#1763

Open
Alena0704 wants to merge 1 commit into
apache:mainfrom
Alena0704:mdb_admin-resource-group
Open

Add special case - mdb_admin group#1763
Alena0704 wants to merge 1 commit into
apache:mainfrom
Alena0704:mdb_admin-resource-group

Conversation

@Alena0704
Copy link
Copy Markdown
Contributor

Allow mdb_admin role to manage resource groups

Summary

Backport of open-gpdb commit 3ac99962ad ("Add special case - mdb_admin group")
by Andrey Borodin, which extends the resource-group permission checks so a
non-superuser member of the mdb_admin role can CREATE / ALTER / DROP
resource groups and call pg_resgroup_move_query().

Without this patch, a managed-cluster user that holds CREATEROLE / CREATEDB
but not SUPERUSER (the typical "MDB admin" persona) hits:

ERROR: must be superuser to create resource groups

even though they own the cluster from the operator's point of view.

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


(cherry picked from commit 3ac99962ad23ec95bb9ef8967a4226e35cf97cc7)
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.

2 participants