Skip to content

[Raft] Add etcd raft dependencies#1460

Merged
mickmis merged 4 commits into
interuss:masterfrom
Orbitalize:add_etcd_raft_dependency
May 13, 2026
Merged

[Raft] Add etcd raft dependencies#1460
mickmis merged 4 commits into
interuss:masterfrom
Orbitalize:add_etcd_raft_dependency

Conversation

@MariemBaccari
Copy link
Copy Markdown
Contributor

@MariemBaccari MariemBaccari commented May 8, 2026

Adds Raft related dependencies (library, wal, snap and rafthttp) under pkg/raftstore/consensus.

One of the imports requires the version 0.5.0 of github.com/jonboulle/clockwork, leading to an upgrade. This has caused a few unit tests to break, more specifically due to this change.

@MariemBaccari MariemBaccari marked this pull request as draft May 8, 2026 12:28
@MariemBaccari MariemBaccari force-pushed the add_etcd_raft_dependency branch from a4a3157 to df293a0 Compare May 8, 2026 12:56
@MariemBaccari MariemBaccari marked this pull request as ready for review May 8, 2026 12:57
@MariemBaccari MariemBaccari marked this pull request as draft May 11, 2026 10:26
@MariemBaccari MariemBaccari force-pushed the add_etcd_raft_dependency branch from f5f6492 to a0b7739 Compare May 12, 2026 14:16
add raft dependencies
@MariemBaccari MariemBaccari force-pushed the add_etcd_raft_dependency branch from a0b7739 to 25f62af Compare May 12, 2026 14:16
@MariemBaccari MariemBaccari changed the title [Raft] Add etcd raft dependency [Raft] Add etcd raft dependencies May 12, 2026
@MariemBaccari MariemBaccari marked this pull request as ready for review May 12, 2026 16:15
Copy link
Copy Markdown
Contributor

@mickmis mickmis left a comment

Choose a reason for hiding this comment

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

LGTM

@mickmis mickmis merged commit 5b78d0d into interuss:master May 13, 2026
12 checks passed
@mickmis mickmis deleted the add_etcd_raft_dependency branch May 13, 2026 07:11
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