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

Add note about pgbouncer incompatibility #37338

Merged
merged 6 commits into from
Feb 23, 2024
Merged

Conversation

programmerq
Copy link
Contributor

@programmerq programmerq commented Jan 26, 2024

Trying to use the pgbouncer backend won't work and can produce an error similar to this:

2024-01-23T22:47:51Z INFO [PGBK]      Setting up backend. pgbk/pgbk.go:181
2024-01-23T22:47:51Z WARN [PGBK]      Failed to connect to the "postgres" database. error:[
ERROR REPORT:
Original Error: *pgconn.connectError failed to connect to `host=dbserver.domain.com user=serviceaccount database=postgres`: server error (FATAL: unsupported startup parameter: default_transaction_isolation (SQLSTATE 08P01))

I'm including this error message here because there were zero hits when searching "unsupported startup parameter" "default_transaction_isolation" "08P01" Google or DuckDuckGo.

Copy link

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@programmerq programmerq added the no-changelog Indicates that a PR does not require a changelog entry label Jan 26, 2024
Copy link

🤖 Vercel preview here: https://docs-fnlyxcevy-goteleport.vercel.app/docs/ver/preview

Copy link
Contributor

@ptgott ptgott left a comment

Choose a reason for hiding this comment

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

Approved, assuming we merge Zac's suggestion

Co-authored-by: Zac Bergquist <[email protected]>
Copy link

🤖 Vercel preview here: https://docs-i3vkzxrnb-goteleport.vercel.app/docs/ver/preview

@zmb3
Copy link
Collaborator

zmb3 commented Jan 30, 2024

@programmerq we just need to add pgbouncer to cspell.json so it's not treated as a misspelling.

add pgbouncer to cspell.json
Copy link

🤖 Vercel preview here: https://docs-1i9cna8p2-goteleport.vercel.app/docs/ver/preview

Copy link

🤖 Vercel preview here: https://docs-9q3k4kdd5-goteleport.vercel.app/docs/ver/preview

Copy link

🤖 Vercel preview here: https://docs-ijhc90de6-goteleport.vercel.app/docs/ver/preview

Merged via the queue into master with commit 861dbba Feb 23, 2024
35 checks passed
@programmerq programmerq deleted the programmerq-patch-1 branch February 23, 2024 15:22
@public-teleport-github-review-bot

@programmerq See the table below for backport results.

Branch Result
branch/v13 Create PR
branch/v14 Create PR
branch/v15 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants