-
Notifications
You must be signed in to change notification settings - Fork 8.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
Fix flaky test in alert details page #64572
Fix flaky test in alert details page #64572
Conversation
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
💚 Build SucceededTo update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Note: Holding off merging into 7.7 until the branch is upgraded to 7.7.1 |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Co-authored-by: Elastic Machine <[email protected]>
Fixes #62676
The tests were flaky in the scenario it took task manager longer than usual to run the newly created alert. In this PR, I'm moving the wait for initial state before loading the UI to ensure all the data necessary is in place.
Flaky runner: https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/407/