-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
rolling-upgrade:v8.12.0#oneThirdUpgradedTest IllegalStateException: failed to obtain node locks #101231
Comments
Pinging @elastic/es-delivery (Team:Delivery) |
@breskeby this sounds like it might be related to #101069. Looking at the cluster logs it looks like we're attempting to start an already started cluster, which would explain the error above. Perhaps the updated logic is losing track of clusters that are used across multiple tasks as is the case for many BWC tests. My guess is some state is getting confused when we upgrade nodes in a cluster. |
@mark-vieira From a brief look at the logic we changed and the project in question I couldn't see how that change affected this and wasn't able to reproduce. I'll have another fresh look tomorrow. as indeed it seems related that we see this failure after making the change we did in #101069 |
Another failure today: https://gradle-enterprise.elastic.co/s/izhi63q6ustnw |
And another: https://gradle-enterprise.elastic.co/s/6ey6xm4uylriy Note that this one was a failure of |
I ran into this failure in a pr: https://gradle-enterprise.elastic.co/s/lwkluhs5zpwf6/console-log?page=3#L2846 |
Another one today: https://gradle-enterprise.elastic.co/s/coocr6hsiw7ny |
We had one in the intake build on 17 March: https://gradle-enterprise.elastic.co/s/a4567iuaplgju/ |
Here is another intake build failure due to this: https://gradle-enterprise.elastic.co/s/h4gi5trbgx5rk All three nodes crashed due to failing to obtain locks on their data paths. |
I think we want to move this pull request forward. The downside is it'll probably make the test execution a bit slower but I think the improvement in stability is probably worth it. I'll pick this back up. |
https://gradle-enterprise.elastic.co/s/l34azxsevqole looks like another instance of this |
Another one: https://gradle-enterprise.elastic.co/s/q3tppdm2xbp3m |
Same error for |
CI Link
https://gradle-enterprise.elastic.co/s/augsybdqwff3i
Repro line
:x-pack:plugin:shutdown:qa:rolling-upgrade:v8.12.0-1
Does it reproduce?
Didn't try
Applicable branches
main
Failure history
No response
Failure excerpt
The text was updated successfully, but these errors were encountered: