Skip to content

azurerm_postgresql_flexible_server_virtual_endpoint - fix read error when in replica set in failover state.#30789

Merged
jackofallops merged 1 commit intomainfrom
b/pgsql_flex_failover
Oct 6, 2025
Merged

azurerm_postgresql_flexible_server_virtual_endpoint - fix read error when in replica set in failover state.#30789
jackofallops merged 1 commit intomainfrom
b/pgsql_flex_failover

Conversation

@jackofallops
Copy link
Member

Community Note

  • Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
  • Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

During a failover the source_server_id and replica_server_id are reversed and attempting to read the ID for the Virtual Endpoint from the non-active server can result in a 400 Bad Request, this PR accounts for this to allow the ID to be read from the other node.

Change Log

Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.

  • azurerm_postgresql_flexible_server_virtual_endpoint - fix Read when in Failover state.

This is a (please select all that apply):

  • Bug Fix
  • New Feature (ie adding a service, resource, or data source)
  • Enhancement
  • Breaking Change

Related Issue(s)

Fixes #0000

AI Assistance Disclosure

  • AI Assisted - This contribution was made by, or with the assistance of, AI/LLMs

Rollback Plan

If a change needs to be reverted, we will publish an updated version of the provider.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

Copy link
Member

@catriona-m catriona-m left a comment

Choose a reason for hiding this comment

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

Thanks @jackofallops LGTM!

@jackofallops jackofallops merged commit c480663 into main Oct 6, 2025
49 checks passed
@jackofallops jackofallops deleted the b/pgsql_flex_failover branch October 6, 2025 14:22
@github-actions github-actions bot added this to the v4.48.0 milestone Oct 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants