Skip to content

feat(redshift_default_privileges): add role support#34

Open
Intuinewin wants to merge 1 commit intodbsystel:masterfrom
Intuinewin:alter-default-role
Open

feat(redshift_default_privileges): add role support#34
Intuinewin wants to merge 1 commit intodbsystel:masterfrom
Intuinewin:alter-default-role

Conversation

@Intuinewin
Copy link

This PR adds role support to the resource redshift_default_privileges. It also updates the code to use the table svv_default_privileges which contains the default privleges instead of the pg_default_acl table

Docs: https://docs.aws.amazon.com/redshift/latest/dg/r_SVV_DEFAULT_PRIVILEGES.html

@mmichaelb mmichaelb self-assigned this Feb 8, 2026
@nico34638
Copy link

Hi @mmichaelb, do you have any news about this PR ?

@mmichaelb
Copy link

Thank you for your contribution! I am currently reviewing this and coming across the previous pg_default_acl table usage, it again makes clear that this provider originates from the postgres provider - good catch on adapting the new table as it seems.

/cc @nico34638

@mmichaelb
Copy link

The changes look good to me so far. In the next days, I will do some regression tests and after completing these tests, this feature can be merged. Due to the changes only adding a further option instead of changes an existing one, I would consider this a non breaking change.

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