Skip to content

[v14] Add PostgreSQL auto-user deletion#33570

Merged
gabrielcorado merged 1 commit intobranch/v14from
gabrielcorado/v14/pg-auto-user-deletion
Oct 19, 2023
Merged

[v14] Add PostgreSQL auto-user deletion#33570
gabrielcorado merged 1 commit intobranch/v14from
gabrielcorado/v14/pg-auto-user-deletion

Conversation

@gabrielcorado
Copy link
Copy Markdown
Contributor

Backport #32792 to branch/v14

* feat: add auto-user deletion postgres

* refactor: change to IsEnabled func to check auto-user

* test: fix linting and test

* refactor(db): code review suggestions

* refactor: rename option to best effort drop

* refactor(api): rename createa database user mode property

* refactor(services): review suggestions

* feat(postgres): add log for user deletion result

* refactor(integrations): regenerate crd manifests

* feat(examples): update operator role spec

* refactor(db): use common sql state codes
@github-actions github-actions Bot added backport database-access Database access related issues and PRs helm size/md labels Oct 17, 2023
Copy link
Copy Markdown
Contributor

@greedy52 greedy52 left a comment

Choose a reason for hiding this comment

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

do we have a doc update for this new mode?

@gabrielcorado
Copy link
Copy Markdown
Contributor Author

do we have a doc update for this new flag?

I'll open the PR for the docs update soon.

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from espadolini October 19, 2023 11:16
@gabrielcorado gabrielcorado added this pull request to the merge queue Oct 19, 2023
Merged via the queue into branch/v14 with commit 5892f59 Oct 19, 2023
@gabrielcorado gabrielcorado deleted the gabrielcorado/v14/pg-auto-user-deletion branch October 19, 2023 12:50
@fheinecke fheinecke mentioned this pull request Oct 23, 2023
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 helm size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants