Skip to content

Fix postgres non superuser#56

Merged
shreddedbacon merged 6 commits into
mainfrom
fix-postgres-non-superuser
Apr 21, 2026
Merged

Fix postgres non superuser#56
shreddedbacon merged 6 commits into
mainfrom
fix-postgres-non-superuser

Conversation

@rocketeerbkw
Copy link
Copy Markdown
Member

@rocketeerbkw rocketeerbkw commented Apr 9, 2026

When dbaas-controller manages a postgres database using a non-superuser, it's not able to provision or delete databases.

Errors provisioning:

postgres db creation failed: postgres db operation create failed: create postgres database error in change owner of database default_9gbWtiPMdjNrIpk0VsSqzodSOGpKV6MIsu26pro1W9LaPiCi1VE4ArQ: pq: must be member of role "9rJDde2oDrZkjZaR" (42501)

Errors deleting:

unable drop database default_9gbWtiPMdjNrIpk0VsSqzodSOGpKV6MIsu26pro1W9LaPiCi1VE4ArQ: pq: must be owner of database default_9gbWtiPMdjNrIpk0VsSqzodSOGpKV6MIsu26pro1W9LaPiCi1VE4ArQ

In order to continue letting provisioned users own the provisioned database, the fix is to grant access to the provisioned user for the dbaas-controller user. That way, it has permission to set owner and delete the database it's not directly an owner of.

Follow up to #34

@rocketeerbkw
Copy link
Copy Markdown
Member Author

rocketeerbkw commented Apr 9, 2026

Ran tests at commit refactor: run postgres tests with non-superuser (before the fix). They failed confirming the bug https://github.com/uselagoon/dbaas-controller/actions/runs/24211480340/job/70681118754?pr=56.

@rocketeerbkw
Copy link
Copy Markdown
Member Author

@rocketeerbkw
Copy link
Copy Markdown
Member Author

Added test case for postgres superuser, as suggested in amazeeio/dbaas-operator#74 (review)

@shreddedbacon shreddedbacon merged commit fc01026 into main Apr 21, 2026
11 checks passed
@shreddedbacon shreddedbacon deleted the fix-postgres-non-superuser branch April 21, 2026 02:57
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