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

go.mod: upgrade etcd-io/raft to 515b1428 #104483

Merged
merged 1 commit into from Jun 8, 2023
Merged

Conversation

pav-kv
Copy link
Collaborator

@pav-kv pav-kv commented Jun 7, 2023

This includes etcd-io/raft#32 which fixes an out-of-memory scenario during election with large unapplied Raft logs.

Part of #104402
Epic: none
Release note (bug fix): Fixed a bug in upstream etcd-io/raft which could result in pulling unlimited amount of log into memory, and lead to out-of-memory situations. With the fix, the log scan has a limited memory footprint.

@pav-kv pav-kv requested a review from tbg June 7, 2023 12:19
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@pav-kv
Copy link
Collaborator Author

pav-kv commented Jun 7, 2023

bor r=tbg

@pav-kv
Copy link
Collaborator Author

pav-kv commented Jun 7, 2023

bors cancel

This includes etcd-io/raft#32 which fixes an
out-of-memory scenario during election with large unapplied Raft logs.

Part of cockroachdb#104402
Epic: none
Release note (bug fix): Fixed a bug in upstream etcd-io/raft which could result
in pulling unlimited amount of log into memory, and lead to out-of-memory
situations. With the fix, the log scan has a limited memory footprint.
@pav-kv pav-kv requested a review from a team as a code owner June 7, 2023 13:08
@pav-kv
Copy link
Collaborator Author

pav-kv commented Jun 7, 2023

bors r=tbg

@pav-kv
Copy link
Collaborator Author

pav-kv commented Jun 8, 2023

bors retry

@craig
Copy link
Contributor

craig bot commented Jun 8, 2023

Build succeeded:

@craig craig bot merged commit 13faab9 into cockroachdb:master Jun 8, 2023
7 checks passed
@pav-kv pav-kv deleted the upgrade-raft branch June 8, 2023 10:30
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.

None yet

3 participants