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

backport-2.1: Bump etcd/raft to pick up uncommitted log size fix #33228

Merged
merged 1 commit into from
Dec 18, 2018

Conversation

tbg
Copy link
Member

@tbg tbg commented Dec 18, 2018

Picks up the bug fix etcd-io/etcd#10199.
It also picks up these further small fixes (which is intentional,
I authored most of them):

5c209d6 raft: ensure leader is in ProgressStateReplicate
1569f48 raft: print RejectHint of zero on MsgAppResp
9668536 raft: add a test case in TestStorageAppend
e4af2be raft: separate MaxCommittedSizePerReady config from MaxSizePerMsg
aa4313a *: fix github links
4aa72ca raft: Explain ReportSnapshot and Propose behavior
b7ed416 raft: fix godoc in tests
10255cf raft: Fix comment on TestLeaderBcastBeat
c561f83 OWNERS: experiment
ad49c8f raft: fix bug in unbounded log growth prevention mechanism
de47099 raft: fix description in UT
73c20cc raft: Fix comment on sendHeartbeat
7be7ac5 raft: Fix spelling in doc.go

Fixes #32895.
Fixes #32900.
Fixes #32898.

Release note (bug fix): Prevent import/restore operations from getting stuck indefinitely.

Picks up the bug fix etcd-io/etcd#10199.
It also picks up these further small fixes (which is intentional,
I authored most of them):

5c209d6 raft: ensure leader is in ProgressStateReplicate
1569f48 raft: print RejectHint of zero on MsgAppResp
9668536 raft: add a test case in TestStorageAppend
e4af2be raft: separate MaxCommittedSizePerReady config from MaxSizePerMsg
aa4313a *: fix github links
4aa72ca raft: Explain ReportSnapshot and Propose behavior
b7ed416 raft: fix godoc in tests
10255cf raft: Fix comment on TestLeaderBcastBeat
c561f83 OWNERS: experiment
ad49c8f raft: fix bug in unbounded log growth prevention mechanism
de47099 raft: fix description in UT
73c20cc raft: Fix comment on sendHeartbeat
7be7ac5 raft: Fix spelling in doc.go

Release note: None
@tbg tbg requested a review from a team December 18, 2018 12:20
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@tbg tbg requested a review from petermattis December 18, 2018 12:20
Copy link
Collaborator

@petermattis petermattis left a comment

Choose a reason for hiding this comment

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

Do you want to also backport the PR that makes use of MaxCommittedSizePerReady?

@tbg
Copy link
Member Author

tbg commented Dec 18, 2018

Sure, let me do that.

TFTR!

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.

3 participants