feat: use FQDN instead of pod IP for Patroni connect_address#13
Closed
marcelozani wants to merge 1 commit intoongres:mainfrom
Closed
feat: use FQDN instead of pod IP for Patroni connect_address#13marcelozani wants to merge 1 commit intoongres:mainfrom
marcelozani wants to merge 1 commit intoongres:mainfrom
Conversation
This change fixes Citus metadata sync issues where nodes register in
pg_dist_node with ephemeral pod IPs. When pods restart and get new IPs,
the stale IPs in pg_dist_node cause metadata synchronization to fail.
Changes:
- PatroniConfigMap.java: Added PATRONI_CONFIG_SERVICE_NAME and
PATRONI_POD_NAMESPACE env vars, changed PATRONI_POSTGRESQL_CONNECT_ADDRESS
to use FQDN pattern
- PatroniEnvironmentVariables.java: Changed PATRONI_RESTAPI_CONNECT_ADDRESS
to use FQDN pattern
- start-patroni.sh: Changed connect_address in restapi and postgresql
sections to use the FQDN-based env vars instead of PATRONI_KUBERNETES_POD_IP
The FQDN pattern used is:
${POD_NAME}.${PATRONI_CONFIG_SERVICE_NAME}.${PATRONI_POD_NAMESPACE}.svc.cluster.local
This ensures stable node registration that survives pod restarts.
|
This repository is just a mirror and does not accept pull requests, the primary repository is located at https://gitlab.com/ongresinc/stackgres |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change fixes Citus metadata sync issues where nodes register in pg_dist_node with ephemeral pod IPs. When pods restart and get new IPs, the stale IPs in pg_dist_node cause metadata synchronization to fail.
Changes:
The FQDN pattern used is:
${POD_NAME}.${PATRONI_CONFIG_SERVICE_NAME}.${PATRONI_POD_NAMESPACE}.svc.cluster.local
This ensures stable node registration that survives pod restarts.