Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v15] Use pg_temp to store PostgreSQL auto provisioning procedures #52150

Merged
merged 1 commit into from
Feb 14, 2025

Conversation

GavinFrazar
Copy link
Contributor

Backports #44255 to branch/v15.

This is to make e2e test backports easier, as the postgres tests now rely on temp tables to test with multiple db admins (each must own the teleport procedures we install).

changelog: Moved PostgreSQL auto provisioning users procedures to pg_temp schema.

@GavinFrazar GavinFrazar requested a review from greedy52 February 13, 2025 21:44
@github-actions github-actions bot added database-access Database access related issues and PRs size/md labels Feb 13, 2025
* refactor(postgres): use pg_temp to store auto user procedures

* refactor(postgres): code review suggestions

* test(postgres): replace logrus with slog

* chore(postgres): move switch statement to a map

* test(postgres): with wrong slog call

* test(postgres): use slog context functions
@GavinFrazar GavinFrazar force-pushed the gavinfrazar/v15-postgres-auto-users-pg-temp branch from 5c9f2a7 to 96bcbc2 Compare February 13, 2025 23:08
@GavinFrazar GavinFrazar added this pull request to the merge queue Feb 14, 2025
Merged via the queue into branch/v15 with commit 9025ed9 Feb 14, 2025
35 checks passed
@GavinFrazar GavinFrazar deleted the gavinfrazar/v15-postgres-auto-users-pg-temp branch February 14, 2025 19:51
@camscale camscale mentioned this pull request Feb 14, 2025
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.

3 participants