Skip to content

Conversation

addaleax
Copy link
Collaborator

This provides the same functionality as the newly introduced legacy
shell equivalent.

Notable differences are:

  • This does some type cleanup around the replset code
  • This is more verbose, and prints:
    • A warning if the new member is already in the existing config
      with a different _id but the same hostname
    • Indicators that retrying is still going on when it is
    • The second reconfig commmand, if it failed, so that the user
      can retry manually
  • This performs exponential backoff while waiting for the second
    reconfig to succeed
  • This uses a fixed number of attempts (12) instead of a fixed
    duration (90 seconds) for performing the second reconfig

This provides the same functionality as the newly introduced legacy
shell equivalent.

Notable differences are:
- This does some type cleanup around the replset code
- This is more verbose, and prints:
  - A warning if the new member is already in the existing config
    with a different _id but the same hostname
  - Indicators that retrying is still going on when it is
  - The second reconfig commmand, if it failed, so that the user
    can retry manually
- This performs exponential backoff while waiting for the second
  reconfig to succeed
- This uses a fixed number of attempts (12) instead of a fixed
  duration (90 seconds) for performing the second reconfig
Copy link
Contributor

@rose-m rose-m left a comment

Choose a reason for hiding this comment

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

Nice, really understandable implementation 👍 😊

@addaleax addaleax merged commit e3a8cb0 into main May 26, 2021
@addaleax addaleax deleted the 702-dev branch May 26, 2021 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants