Skip to content

Proper Configuration of Single Instance DBs after Deletion of Sync Standbies #577

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 31 commits into from
Aug 27, 2024
Merged

Proper Configuration of Single Instance DBs after Deletion of Sync Standbies #577

merged 31 commits into from
Aug 27, 2024

Conversation

eberlep
Copy link
Collaborator

@eberlep eberlep commented Jul 26, 2024 •
edited
Loading

Based on #575 , which should be merged first.

eberlep added 24 commits July 12, 2024 15:56
@eberlep
Double check config
ea3d308
@eberlep
Logging
59c5021
@eberlep
Especially double check when bootstrapping
f1ca8db
@eberlep
Check for new value
4ecb5f5
@eberlep
Revert "Especially double check when bootstrapping"
054ba69 
This reverts commit f1ca8db.
@eberlep
Especially double check when bootstrapping
5d3ca54
@eberlep
Revert "Check for new value"
7b86f4c 
This reverts commit 4ecb5f5.
@eberlep
Fetch and use the actual postgres cluster name as application_name fo…
fd79457 
…r. This resolves some issues when using sync replication.
@eberlep
Always fetch application_name when needed
2c215f1
@eberlep
make linter happy
194b466
@eberlep
Simplify
8b76805
@eberlep
logging
d9920c1
@eberlep
Fix typos
827fe9f
@eberlep
Merge branch 'sync_mode_reloaded_part_2' into sync_mode_reloaded_part_3
715a61c
@eberlep
Merge remote-tracking branch 'origin' into sync_mode_reloaded_part_3
6575b87
@eberlep
Logging
030c0d8
@eberlep
Also patch patroni in single instance mode
4b4816e 
(to ensure proper cleanup after a sync standby has been deleted)
@eberlep
Debug logging
e5aff3d
@eberlep
More precise nil
77864cb
@eberlep
On a related note, do not requeue
5f9402d
@eberlep
Some more debug logging
97bf6dd
@eberlep
Additional check
b74e827
@eberlep
Logging
d3ea714
@eberlep
Back to default nil
5143709
@eberlep eberlep marked this pull request as ready for review July 26, 2024 11:38
@eberlep eberlep requested a review from ulrichSchreiner July 26, 2024 11:38
ulrichSchreiner
ulrichSchreiner reviewed Jul 29, 2024
View reviewed changes
ulrichSchreiner
ulrichSchreiner reviewed Jul 29, 2024
View reviewed changes
@eberlep eberlep merged commit de4dd9d into main Aug 27, 2024
1 check passed
@eberlep eberlep deleted the sync_mode_reloaded_part_4 branch August 27, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ulrichSchreiner ulrichSchreiner ulrichSchreiner left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eberlep @ulrichSchreiner