Skip to content
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.

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

NET-8983: bump raft to latest version #21751

Closed
wants to merge 3 commits into from

Conversation

JadhavPoonam
Copy link
Contributor

Description

The new raft library has the latest pre-vote change with relevant bug fixes. Bumping this now for code-freeze while we are still testing it out in Consul. If the testing reveals any major bugs then we will revert it back before the release.

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@github-actions github-actions bot added the pr/dependencies PR specifically updates dependencies of project label Sep 17, 2024
@JadhavPoonam JadhavPoonam added pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry labels Sep 17, 2024
@JadhavPoonam JadhavPoonam added backport/1.20 Changes are backported to 1.20 and removed pr/no-backport labels Sep 17, 2024
@JadhavPoonam JadhavPoonam changed the title chore: bump raft to latest version NET-8983: bump raft to latest version Sep 17, 2024
Copy link
Contributor

@dhiaayachi dhiaayachi left a comment

Choose a reason for hiding this comment

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

@JadhavPoonam Thank you for adding this. I think we need a changelog for this, we can also point in the changelog that the new raft version includes pre-vote and that it's activated by default.

We will also need to map the raft config to a consul config to be able to deactivate prevote if needed.

@JadhavPoonam
Copy link
Contributor Author

@dhiaayachi Closing this since it's best we put a new PR with the new changes to the config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.20 Changes are backported to 1.20 pr/dependencies PR specifically updates dependencies of project pr/no-changelog PR does not need a corresponding .changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants