Skip to content

Conversation

@zliang-akamai
Copy link
Member

📝 Description

Update secondary host to standby host to align with API changes.

✔️ How to Test

make test-unit

Or long running integration tests:

make test-int TEST_SUITE=databasepostgresqlv2
make test-int TEST_SUITE=databasemysqlv2
make test-int TEST_SUITE=databases

@zliang-akamai zliang-akamai requested a review from a team as a code owner December 12, 2025 17:02
@zliang-akamai zliang-akamai requested review from vshanthe and yec-akamai and removed request for a team December 12, 2025 17:02
@zliang-akamai zliang-akamai added the breaking-change for breaking changes in the changelog. label Dec 12, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates database resource and data source implementations to rename "secondary host" references to "standby host" throughout the codebase, aligning with upstream API changes in linodego v1.62.0.

Key changes:

  • Updated struct field tags from host_secondary to host_standby across all database resources
  • Modified schema descriptions to use "standby" terminology instead of "secondary"
  • Updated linodego dependency from v1.61.0 to v1.62.0 and related transitive dependencies

Reviewed changes

Copilot reviewed 34 out of 35 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
go.mod Bumped linodego to v1.62.0 and updated golang.org/x dependencies
linode/databases/* Updated filter config, schema, models, and tests to use host_standby
linode/databasepostgresql/* Renamed host_secondary to host_standby in schemas, models, and tests
linode/databasepostgresqlv2/* Updated schemas and models to reference Standby field from linodego
linode/databasemysql/* Renamed host_secondary to host_standby in schemas, models, and tests
linode/databasemysqlv2/* Updated schemas and models to reference Standby field from linodego
docs/resources/* Updated documentation to reference host_standby instead of host_secondary
docs/data-sources/* Updated documentation to reference host_standby instead of host_secondary

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 34 out of 35 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zliang-akamai zliang-akamai force-pushed the zhiwei/dbaas-standby-host branch 2 times, most recently from 62d38d8 to bc6cb89 Compare December 12, 2025 22:00
@zliang-akamai zliang-akamai force-pushed the zhiwei/dbaas-standby-host branch from bc6cb89 to a1333e9 Compare December 12, 2025 22:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 34 out of 35 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

zliang-akamai and others added 4 commits December 15, 2025 13:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@zliang-akamai zliang-akamai marked this pull request as draft December 15, 2025 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change for breaking changes in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants