Skip to content

Conversation

@DimCitus
Copy link
Collaborator

@DimCitus DimCitus commented Apr 8, 2021

If the upstream server isn't ready for a replication connection being made,
then both pg_rewind and then pg_basebackup will fail. The current code would
attribute the error to pg_rewind itself though, and continue with a full
base backup, where it might be possible to connect and rewind later.

@DimCitus DimCitus added enhancement New feature or request Size: S Effort Estimate: Small labels Apr 8, 2021
@DimCitus DimCitus added this to the Sprint 2021 W13 W14 milestone Apr 8, 2021
@DimCitus DimCitus requested review from JelteF and thanodnl April 8, 2021 15:26
@DimCitus DimCitus self-assigned this Apr 8, 2021
If the upstream server isn't ready for a replication connection being made,
then both pg_rewind and then pg_basebackup will fail. The current code would
attribute the error to pg_rewind itself though, and continue with a full
base backup, where it might be possible to connect and rewind later.
@DimCitus DimCitus force-pushed the fix/check-replication-connection-before-pg_rewind branch from eb2156b to b13c933 Compare April 19, 2021 15:05
@DimCitus DimCitus merged commit 83f7f0c into master Apr 19, 2021
@DimCitus DimCitus deleted the fix/check-replication-connection-before-pg_rewind branch April 19, 2021 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Size: S Effort Estimate: Small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants