-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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
[CI] HistoryActionConditionTests.testActionCondition is failing #65064
Comments
Pinging @elastic/es-core-features (:Core/Features/Watcher) |
Probably related to #65048 |
When we trigger watcher in tests we may fail if scheduler is paused. That can happen when watcher is reloading due to changes in its shards. This is transient state and retry should fix it. This change adds assertBusy around triggering code so we will try again. Closes #65183 #65176 #65117 #65091 #65089 #65086 #65064 #65063
Should be fixed by #65186 |
|
Another with the same expected 1 got 2 error. Muted on master in #65592
|
When watcher gets reloaded (due to shards routings change) during tests we sometimes get additional watcher history entry (either that the same watch is already queued or that it got throttled because it tried to execute right after previous one). That trips our assertions which expect exact number of entries. This change relaxes them to allow exactly-one-more situation and leaves all other checks in place. Closes #65831 Closes #65176 Closes #65064 Closes #65091
I got another one of these: https://gradle-enterprise.elastic.co/s/ryflwip3sc7ze/tests/:x-pack:plugin:watcher:internalClusterTest/org.elasticsearch.xpack.watcher.history.HistoryActionConditionTests/testActionCondition#1
|
Another on master: https://gradle-enterprise.elastic.co/s/rpo5nn6hpjhce
|
Another failure https://gradle-enterprise.elastic.co/s/rpo5nn6hpjhce This is still failing regularly so I've re-applied the muting I made 2 comments ago. (#68650) |
This test was unmuted a while ago via #67404 which should have closed this issue but didn't for some reason. No comments in over 6m, and no failures in the past 28 days. Closing. |
This test is still muted in |
Pinging @elastic/es-data-management (Team:Data Management) |
Build scan: https://gradle-enterprise.elastic.co/s/qhcmhs7bgvyyq
Repro line:
Reproduces locally?: No
Applicable branches:
master
Failure history:
build stats
Failed 16 times in the past month
Failure excerpt:
The text was updated successfully, but these errors were encountered: