Skip to content

Conversation

@hana-akamai
Copy link
Contributor

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

Description 📝

  • Display general Service URI for all databases under Connection details in the summary tab
  • If applicable, display PgBouncer connection details Service URI for postgres clusters with at least one connection pool

Changes 🔄

List any change(s) relevant to the reviewer.

  • Refactor Database CA Certificate into a separate file
  • Refactor Service URI to include general support
  • Add general and PgBouncer service URIs to Database summary tab

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

Preview 📷

Mysql Postgres
image image

How to test 🧪

Prerequisites

(How to setup test environment)

  • Ensure the Database PgBouncer feature flag is enabled and turn on the legacy MSW

Verification steps

(How to verify changes)

  • Go to a mysql Database cluster's summary page and confirm that a general mysql service URI displays under Connection Details. There should not be a PgBouncer section.
  • Go to a postgres Database cluster's summary page and confirm that a general postgres service URI displays under Connection Details. There should be a PgBouncer section
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@hana-akamai hana-akamai self-assigned this Jan 9, 2026
@hana-akamai hana-akamai added the DBaaS Relates to Database as a Service label Jan 9, 2026
@hana-akamai hana-akamai marked this pull request as ready for review January 12, 2026 22:24
@hana-akamai hana-akamai requested a review from a team as a code owner January 12, 2026 22:24
Copy link
Contributor

@mpolotsk-akamai mpolotsk-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 verification steps:

  • MySQL Database Summary page shows the service URI under Connection Details.
  • PostgreSQL Database Summary page shows the service URI under Connection Details and the PgBouncer section.

@bnussman-akamai bnussman-akamai self-requested a review January 13, 2026 14:16
@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jan 13, 2026
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #6 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing854 Passing11 Skipped44m 39s

Details

Failing Tests
SpecTest
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts"

</Typography>
<ServiceURI database={database} />
</Grid>
)}
Copy link
Contributor

@smans-akamai smans-akamai Jan 14, 2026

Choose a reason for hiding this comment

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

It may not be necessary, but I'm wondering if it would make sense to add some kind of error state for this field in the event that the connection-pools GET call fails. Currently it just hides the PgBouncer Connection Details field as a whole, but they won't actually see any error messaging until they navigate to the Networking tab.

Copy link
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.

The changes look great and all my feedback has been addressed! I left a follow up comment on the error handling behavior in the summary tab, as something we might want to consider.

But this looks good to me so I've approved!

@hana-akamai hana-akamai merged commit 6dbabd7 into linode:develop Jan 14, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DBaaS Relates to Database as a Service

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

5 participants