Skip to content

upcoming: [UIE-10174] - Update Database types for Hostname changes#13386

Merged
hana-akamai merged 4 commits intolinode:developfrom
hana-akamai:UIE-10174-update-database-types-hostname
Feb 10, 2026
Merged

upcoming: [UIE-10174] - Update Database types for Hostname changes#13386
hana-akamai merged 4 commits intolinode:developfrom
hana-akamai:UIE-10174-update-database-types-hostname

Conversation

@hana-akamai
Copy link
Copy Markdown
Contributor

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

Description 📝

Deprecate connection_pool_port property and add endpoints array to DatabaseHosts to support backend hostname changes along with factory/mock updates

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 📷

No visual changes

Verification steps

(How to verify changes)

  • Changes should match the API spec linked in ticket
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 Feb 9, 2026
@hana-akamai hana-akamai added the DBaaS Relates to Database as a Service label Feb 9, 2026
@hana-akamai hana-akamai marked this pull request as ready for review February 10, 2026 17:41
@hana-akamai hana-akamai requested a review from a team as a code owner February 10, 2026 17:41
Comment thread packages/api-v4/src/databases/types.ts Outdated
Copy link
Copy Markdown
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

Approved pending addition of standby-connection-pool as a role

Comment thread packages/api-v4/src/databases/types.ts Outdated
private_access: true,
port: 12345,
},
],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me, but as a future change, I'd suggest making the 'endpoints' array content dynamic based on the VPC configuration. We could have it render different endpoints based on the settings, similar to how it behaves in the backend.

That way, it'd be easier to test the display behavior for the fields that appear in the UI and we could even add a comment to give insight into the backend behavior.

But since we're still confirming the behavior, we could consider it one of the upcoming tickets for this instead.

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.

This looks good and my feedback has been addressed! I left an additional comment on some behavior we could include in one of the upcoming pull requests around this feature.

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

Cloud Manager UI test results

🔺 1 failing test on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing865 Passing11 Skipped43m 6s

Details

Failing Tests
SpecTest
quotas-storage.spec.tsCloud Manager Cypress Tests→Quota workflow tests→Quota storage table » Quotas and quota usages display properly

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/account/quotas-storage.spec.ts"

@hana-akamai hana-akamai merged commit 83e410d into linode:develop Feb 10, 2026
34 of 35 checks passed
@hana-akamai hana-akamai deleted the UIE-10174-update-database-types-hostname branch February 10, 2026 22:36
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Feb 10, 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

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants