Skip to content

[v14] Database Automatic User Provisioning support for Redshift#34126

Merged
greedy52 merged 2 commits intobranch/v14from
STeve/backport/v14/33307
Nov 6, 2023
Merged

[v14] Database Automatic User Provisioning support for Redshift#34126
greedy52 merged 2 commits intobranch/v14from
STeve/backport/v14/33307

Conversation

@greedy52
Copy link
Copy Markdown
Contributor

@greedy52 greedy52 commented Nov 1, 2023

backport of #33307 and #34006 to v14

only minor merge conflict as the change to GetAdminUser from string to struct is not backported yet. I will adjust accordingly when backporting MariaDB support.

changelog: Added Database Automatic User Provisioning support for Redshift

TODOs

greedy52 and others added 2 commits November 1, 2023 15:24
* Database Automatic User Provisioning support for Redshift

* capitalize sql script

* DeleteUser to fallback to deactivate.

* add TPxxx code to RAISE messages

* fix missing space in scripts
@greedy52 greedy52 self-assigned this Nov 1, 2023
@github-actions github-actions Bot requested a review from Tener November 1, 2023 19:38
@github-actions github-actions Bot added backport database-access Database access related issues and PRs size/md labels Nov 1, 2023
Copy link
Copy Markdown
Contributor

@gabrielcorado gabrielcorado left a comment

Choose a reason for hiding this comment

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

Thanks for back porting #34006

@greedy52 greedy52 added this pull request to the merge queue Nov 6, 2023
Merged via the queue into branch/v14 with commit 38cc706 Nov 6, 2023
@greedy52 greedy52 deleted the STeve/backport/v14/33307 branch November 6, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport database-access Database access related issues and PRs size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants