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

Reproduce issue 12535 #5

Closed
wants to merge 2 commits into from
Closed

Reproduce issue 12535 #5

wants to merge 2 commits into from

Conversation

chaochn47
Copy link
Owner

@chaochn47 chaochn47 commented Feb 8, 2022

Reproduce issue described in etcd-io#12535

=== RUN   TestIssue12535
    v3_lease_test.go:199: expect revision 17, but got 16
    v3_lease_test.go:202: lease removed but key remains
--- FAIL: TestIssue12535 (1.15s)
FAIL

Process finished with exit code 1
{
    "level": "warn",
    "ts": "2022-02-08T23:10:35.433Z",
    "caller": "etcdserver/util.go:163",
    "msg": "apply request took too long",
    "took": "20.64µs",
    "expected-duration": "0s",
    "prefix": "",
    "request": "header:<ID:10441453742734613263 > lease_revoke:<id:-000000000000001>",
    "response": "size:27",
    "error": "lease not found"
}
{
    "level": "warn",
    "ts": "2022-02-08T23:10:35.433Z",
    "caller": "etcdserver/util.go:121",
    "msg": "failed to apply request",
    "took": "55.007µs",
    "request": "header:<ID:10441453742734613263 > lease_revoke:<id:-000000000000001>",
    "response": "size:27",
    "error": "lease not found"
}{
    "level": "warn",
    "ts": "2022-02-08T23:10:35.433Z",
    "caller": "etcdserver/util.go:163",
    "msg": "apply request took too long",
    "took": "98.02µs",
    "expected-duration": "0s",
    "prefix": "",
    "request": "header:<ID:10441453742734613263 > lease_revoke:<id:-000000000000001>",
    "response": "size:27"
}

The full test log is captured in test.log

@serathius
Copy link

Hey @chaochn47 can you start fixing the issue from main branch. Maybe not all branches are affected, however we should add tests to all of them.

@chaochn47
Copy link
Owner Author

chaochn47 commented Feb 9, 2022

@serathius ack. That sounds good to me. I will open a PR against main branch.

@chaochn47
Copy link
Owner Author

Update - Here is the reproduce on the main branch #6 @serathius

@chaochn47 chaochn47 closed this Oct 4, 2022
@chaochn47 chaochn47 deleted the reproduce_12535 branch October 4, 2022 04:06
chaochn47 added a commit that referenced this pull request Feb 23, 2023
Signed-off-by: Chao Chen <chaochn@amazon.com>
chaochn47 added a commit that referenced this pull request Feb 28, 2023
Signed-off-by: Chao Chen <chaochn@amazon.com>
chaochn47 added a commit that referenced this pull request Mar 3, 2023
Signed-off-by: Chao Chen <chaochn@amazon.com>
chaochn47 pushed a commit that referenced this pull request Mar 17, 2023
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