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

[3.5] etcdserver: return membership.ErrIDNotFound when the memberID not found #15096

Merged

Conversation

ahrtr
Copy link
Member

@ahrtr ahrtr commented Jan 13, 2023

Backport etcd-io#15095.

When promoting a learner, we need to wait until the leader's applied ID
catches up to the commitId. Afterwards, check whether the learner ID
exist or not, and return `membership.ErrIDNotFound` directly in the API
if the member ID not found, to avoid the request being unnecessarily
delivered to raft.

Signed-off-by: Benjamin Wang <wachao@vmware.com>
@ahrtr ahrtr force-pushed the 3.5_promote_non_exist_id_20230113 branch from af2f248 to 53300ec Compare January 16, 2023 22:24
@ahrtr ahrtr requested a review from ptabor January 16, 2023 22:53
@serathius serathius merged commit 638c6f1 into etcd-io:release-3.5 Jan 17, 2023
tjungblu pushed a commit to tjungblu/etcd that referenced this pull request Jul 26, 2023
…20230113

[3.5] etcdserver: return membership.ErrIDNotFound when the memberID not found
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants