-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
Flaky test: DockerSwarmSuite.TestSwarmLockUnlockCluster #34051
Comments
/cc @tophj-ibm |
We think that moby/swarmkit#2281 fixes this, but it hasn't been vendored yet. |
Opened #34061 to vendor that change and others into master. |
NOTE: this is now failing with A lot of swarm tests have been failing recently (also #34993) from what appears to be the same issue, nodes restarting and getting out of sync. From experimental: https://jenkins.dockerproject.org/job/Docker-PRs-experimental/37592/consoleFull When this node restarts, it's ahead by 1 in the term cycle so it ignores all msgvotes. It then starts it's own elections and keeps getting rejected, resulting in the "The swarm does not have a leader" error message, and eventually "swarm component could not be started before timeout was reached".
I could be wrong, but I don't think this is how this is supposed to work. |
This one is failing a lot recently |
@arkodg I assume we need to to set some flags on our test daemons to make sure they don't try to bind on the same addr here. |
@cpuguy83 there's a moby/integration-cli/check_test.go Lines 324 to 327 in 4866f51
But looks like it isn't protected; so we might need a lock; moby/integration-cli/check_test.go Line 339 in 4866f51
|
perhaps this works: #39516 |
This is still flaky. Running the PR checks (janky) against moby/moby master branch, I see this failing once out of the last 30 runs. From console logs of tests run against tree https://github.com/moby/moby/tree/7449ca3:
|
Seen failing on
Note that this test was flaky before (#28997), but recently became an issue again
ping @cyli @aaronlehmann PTAL
The text was updated successfully, but these errors were encountered: