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

Internalise Raft lib #292

Merged
merged 17 commits into from
Jun 4, 2024
Merged

Conversation

coufalja
Copy link
Contributor

@coufalja coufalja commented May 23, 2024

Description

The Raft replacement targets next major 0.6.x release, but for now will live out-of-main branch in feat/raft-replacement due to need of extensive testing.

The PR is a

  • Fork of a lib
  • Removal of some unused features
  • Rename of a module
  • Upgrade of a pebble dependency
  • Some cleanup of tests so they could run as part of general regatta suite

Related Issue

Related to https://github.com/jamf/regatta/blob/master/docs/proposals/003-replace-raft-lib.md

Motivation and Context

This PR forks the dragonboat lib into the Regatta codebase. The goal of the change is to get rid of hacks in Regatta codebase as well as ensure the ongoing development. Dragonboat is now in sort of a development limbo as well as being encumbered by historic burden. The fork should give us more freedom in designing and maintaining regatta as well as potentially ease up replacement with a different library.

How Has This Been Tested?

@github-actions github-actions bot added the code Pull requests that update Go code label May 23, 2024
@coufalja coufalja force-pushed the feature/internalise_raft branch 2 times, most recently from 5974c2a to 6136acc Compare May 23, 2024 11:21
@coufalja coufalja force-pushed the feature/internalise_raft branch 2 times, most recently from 0cdbeca to 6f3d884 Compare May 31, 2024 14:39
@coufalja coufalja marked this pull request as ready for review June 2, 2024 14:18
@coufalja coufalja requested review from a team as code owners June 2, 2024 14:18
@coufalja coufalja changed the base branch from master to feat/raft-replacement June 2, 2024 14:20
@coufalja coufalja changed the title WIP - Internalise Raft lib Internalise Raft lib Jun 2, 2024
raft/AUTHORS Outdated Show resolved Hide resolved
@coufalja coufalja requested a review from jsfpdn June 3, 2024 08:11
@coufalja coufalja merged commit 820311f into feat/raft-replacement Jun 4, 2024
9 checks passed
@coufalja coufalja deleted the feature/internalise_raft branch June 4, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants