Skip to content

Unregister member + cleanup replication slots #33

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 12 commits into from
Jan 17, 2023
Merged

Unregister member + cleanup replication slots #33

merged 12 commits into from
Jan 17, 2023

Conversation

davissp14
Copy link
Contributor

@davissp14 davissp14 commented Jan 16, 2023 •
edited
Loading

Notable changes:

  • Added script that can be called from flyctl to unregister a member from repmgr/consul.
  • Reworked the standby_cleaner to compare time durations rather than unix int's.
  • Standby cleaner will now remove inactive replication slots if the slot id is not associated with a registered repmgr member.

@davissp14 davissp14 changed the title WIP: Unregister member script Unregister member Jan 16, 2023
@davissp14 davissp14 changed the title Unregister member Unregister member + cleanup replication slots Jan 16, 2023
@davissp14 davissp14 requested a review from DAlperin January 16, 2023 21:53
@davissp14 davissp14 merged commit a166ff2 into master Jan 17, 2023
@davissp14 davissp14 deleted the unregister-member-script branch February 25, 2023 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DAlperin DAlperin Awaiting requested review from DAlperin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@davissp14