-
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
[TSVB] fixes flaky test on switching index patterns #70811
[TSVB] fixes flaky test on switching index patterns #70811
Conversation
Pinging @elastic/kibana-app (Team:KibanaApp) |
Good catch! Thank you for the fix! I started flaky-test-runner to see how stable is it now https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/563/ |
Thank you @dmlemeshko ! I have already done it for 42 attempts. Hasn't failed but let's test it again!! https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/561/ |
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
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / Kibana Embedded in iframe with X-Pack Security.x-pack/test/functional_embedded/tests/iframe_embedded·ts.Kibana embedded in iframe should open Kibana for logged-in userStandard Out
Stack Trace
Build metrics
History
To update your PR or re-run it, just comment with: |
Co-authored-by: Elastic Machine <[email protected]>
* master: fix flaky test on tsvb switch index patterns (elastic#70811) skip flaky suite (elastic#70757) Fix Data Streams and Rollups Jobs deep-link bugs (elastic#70903)
…rbac * alerting/consumer-based-rbac: fix flaky test on tsvb switch index patterns (elastic#70811) skip flaky suite (elastic#70757) Fix Data Streams and Rollups Jobs deep-link bugs (elastic#70903)
Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Summary
Fixes #63274. Moved the security role on the initialization of the test as it was causing a page reload and the metric value could not be found.
Checklist