-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
raft/rafttest: TestRestart failed #127413
Comments
This is the same flake that was discussed in etcd-io/raft#181. It was later fixed upstream by etcd-io/raft#188. We are in the process of picking this fix up over in #122133. @pav-kv do we want to revive that PR and get it merged so that we can close this? If not, I'll import just that one commit. |
raft/rafttest.TestRestart failed with artifacts on master @ 4ae4a52772b4b6b2bf2c340e06919c090894dffb:
|
raft/rafttest.TestRestart failed with artifacts on master @ 167a9af39d0c1fadb9de8ee3c40e616e6c98af78:
|
raft/rafttest.TestRestart failed with artifacts on master @ bb2613e5348ea864c6a9cc348af73117d701f453:
|
raft/rafttest.TestRestart failed with artifacts on master @ 1993fc04b5116f20f4814d637c7ce87b003687e4:
|
raft/rafttest.TestRestart failed with artifacts on master @ 972ccd5d17c54f2cef84e38d14290e52b01bd70a:
|
raft/rafttest.TestRestart failed with artifacts on master @ 8551145a0c99c4c95a28ec470e699d0c20ca97ab:
|
raft/rafttest.TestRestart failed with artifacts on master @ 344fb32842be378bc82c2f3ed8ba12d6f3225be7:
|
Fixes cockroachdb#127413. This commit bypasses the larger rebase in cockroachdb#122133 to pick up the test flake fix in etcd-io/raft#188. There was some discussion in etcd-io/raft#181 about alternatives for fixing this test. For now, we stick with a direct cherry-pick. Release note: None
raft/rafttest.TestRestart failed with artifacts on master @ 77606cbe1c98b248ea658105d7109479439a037f:
|
130084: raft: fix flaky leader index in waitLeader function r=pav-kv a=nvanbenschoten Fixes #127413. This commit bypasses the larger rebase in #122133 to pick up the test flake fix in etcd-io/raft#188. There was some discussion in etcd-io/raft#181 about alternatives for fixing this test. For now, we stick with a direct cherry-pick. Release note: None Co-authored-by: Nathan VanBenschoten <[email protected]>
Based on the specified backports for linked PR #130084, I applied the following new label(s) to this issue: branch-release-24.1, branch-release-24.2. Please adjust the labels as needed to match the branches actually affected by this issue, including adding any known older branches. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
Fixes cockroachdb#127413. This commit bypasses the larger rebase in cockroachdb#122133 to pick up the test flake fix in etcd-io/raft#188. There was some discussion in etcd-io/raft#181 about alternatives for fixing this test. For now, we stick with a direct cherry-pick. Release note: None
Fixes cockroachdb#127413. This commit bypasses the larger rebase in cockroachdb#122133 to pick up the test flake fix in etcd-io/raft#188. There was some discussion in etcd-io/raft#181 about alternatives for fixing this test. For now, we stick with a direct cherry-pick. Release note: None
raft/rafttest.TestRestart failed with artifacts on master @ 46854026bc11b3870c8bbefc976f5550b18dd82c:
Help
See also: How To Investigate a Go Test Failure (internal)
This test on roachdash | Improve this report!
Jira issue: CRDB-40428
The text was updated successfully, but these errors were encountered: