-
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
Unskip replace flyout test #57093
Unskip replace flyout test #57093
Conversation
Pinging @elastic/kibana-app-arch (Team:AppArch) |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / Chrome X-Pack UI Functional Tests.x-pack/test/functional/apps/discover/feature_controls/discover_security·ts.discover feature controls security global discover all privileges allow saving via the saved query management component popover with no query loadedStandard Out
Stack Trace
To update your PR or re-run it, just comment with: |
retest |
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
💚 Build SucceededHistory
To 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.
I ran it on flaky-test-runner CI job and got 20 times passed. Thank you for the fix and unskipping the test!
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / Chrome X-Pack UI Functional Tests.x-pack/test/functional/apps/visualize/feature_controls/visualize_spaces·ts.Visualize visualize space with no features disabled can view existing VisualizationStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
Close #55992
This regressed in #54105 and should be implicitly fixed in #55158.
Just for info:
Cannot reproduce locally
I was able to reproduce locally, by forcing state to come from URL. e.g
And that error is in console