Skip to content

fix: [UIE-10423] - PgBouncer Connection Pool bugs#13474

Merged
hana-akamai merged 7 commits intolinode:developfrom
hana-akamai:UIE-10423-updated-connection-pool
Mar 10, 2026
Merged

fix: [UIE-10423] - PgBouncer Connection Pool bugs#13474
hana-akamai merged 7 commits intolinode:developfrom
hana-akamai:UIE-10423-updated-connection-pool

Conversation

@hana-akamai
Copy link
Copy Markdown
Contributor

@hana-akamai hana-akamai commented Mar 9, 2026

Description 📝

Fix various PgBouncer Connection Pool bugs now that we can test with a real API

Note: Username issues are API related

Changes 🔄

  • Only send a GET connection pools request for Postgres clusters
  • Move connection pool loading/error state inside the table instead of the whole section
  • Fix updated Connection Pool values not reflecting in the table or edit drawer
  • Fix Delete Connection Pool triggering loading state
  • Disable Edit Connection Pool menu action while Database cluster is provisioning

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

Next release

Preview 📷

Before After
image image
edit.before.mov
edit.after.mov
delete.before.mov
delete.after.mov
edit.provisioning.before.mov

How to test 🧪

Prerequisites

(How to setup test environment)

  • Ensure you have the DatabasePGBouncer and HostnameEndpoints feature flag on
  • Point to devcloud locally
  • Have a Postgres and Mysql Database cluster

Reproduction steps

(How to reproduce the issue, if applicable)

  • Go to remote devcloud
  • With the Network tab open, go to a Mysql cluster's details summary page, you should see a failed connection pools request
  • Go to a Postgres cluster's details page, then click on the Networking tab
  • Add a Connection Pool if you don't already have one
  • Update the Connection Pool, the updated value does not display in the table or drawer unless you refresh
  • Delete a Connection Pool, the whole section is momentarily replaced by a loading spinner
  • Have a Database cluster in provisioning state and go to it's Networking tab. The edit connection pool menu action is not disabled and trying to save changes results in an API error

Verification steps

(How to verify changes)

  • Checkout this PR locally and point to devcloud
  • With the Network tab open, go to a Mysql cluster's details summary page, you should not see any connection pools request
  • Go to a Postgres cluster's details page, then click on the Networking tab
  • Add a Connection Pool if you don't already have one
  • Update the Connection Pool, the updated value should display in the table and drawer without a refresh
  • Delete a Connection Pool, you should not see a loading spinner
  • Have a Database cluster in provisioning state and go to it's Networking tab. The edit connection pool menu action is disabled

@hana-akamai hana-akamai self-assigned this Mar 9, 2026
@hana-akamai hana-akamai added the Bug Fixes for regressions or bugs label Mar 9, 2026
@hana-akamai hana-akamai added the DBaaS Relates to Database as a Service label Mar 9, 2026
@hana-akamai hana-akamai marked this pull request as ready for review March 9, 2026 21:12
@hana-akamai hana-akamai requested a review from a team as a code owner March 9, 2026 21:12
@hana-akamai hana-akamai requested a review from smans-akamai March 9, 2026 21:12
Copy link
Copy Markdown
Contributor

@smans-akamai smans-akamai left a comment

Choose a reason for hiding this comment

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

Tested the verification steps and the fixes look great!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Mar 10, 2026
@linode-gh-bot
Copy link
Copy Markdown
Collaborator

Cloud Manager UI test results

🔺 3 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
3 Failing873 Passing11 Skipped43m 19s

Details

Failing Tests
SpecTest
object-storage-objects-multicluster.spec.tsCloud Manager Cypress Tests→Object Storage Multicluster objects » can upload, access, and delete objects
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can create and delete object storage buckets
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Parent » can switch from Proxy user back to Parent account user from Billing page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/objectStorageMulticluster/object-storage-objects-multicluster.spec.ts,cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts"

@hana-akamai hana-akamai merged commit f6b2e8d into linode:develop Mar 10, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Mar 10, 2026
@hana-akamai hana-akamai deleted the UIE-10423-updated-connection-pool branch March 10, 2026 17:58
harsh-akamai pushed a commit to harsh-akamai/manager that referenced this pull request Mar 17, 2026
Fix various PgBouncer Connection Pool bugs now that we can test with a real API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fixes for regressions or bugs DBaaS Relates to Database as a Service

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants