-
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
[Security Solution] Explore, fix flakey Cypress tests #161874
Comments
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Original issue: #161874 ## Summary Flaky entity analytics cypress test Based on my analysis, it seems likely that the issue was caused by some tests failing to close the filter after opening it. This could have had an impact on the subsequent tests. After the change, I could run the test successfully more than 60 times. Here is a print from some of these runs: <img width="587" alt="Screenshot 2023-08-04 at 13 58 55" src="https://github.com/elastic/kibana/assets/1490444/02367810-3616-4fa2-a82e-48c19e6ac4b6">
Original issue: elastic#161874 ## Summary Flaky entity analytics cypress test Based on my analysis, it seems likely that the issue was caused by some tests failing to close the filter after opening it. This could have had an impact on the subsequent tests. After the change, I could run the test successfully more than 60 times. Here is a print from some of these runs: <img width="587" alt="Screenshot 2023-08-04 at 13 58 55" src="https://github.com/elastic/kibana/assets/1490444/02367810-3616-4fa2-a82e-48c19e6ac4b6">
Tried for a few days on this and still cannot pass CI: #163989 I can only get the failures on CI, not locally. I can throttle CPU and/or network and get different failures… but I cannot reproduce the CI failure locally. I’m stuck, going to put this back on the shelf for now. |
Resolved by #166492 |
Is this still an issue? |
latest flakey test run: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4418 |
still valid: |
## Summary part of: #161874 Fixes hover actions flaky cypress test. Flaky test runner execution after changes: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5293 --------- Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Angela Chuang <[email protected]>
## Summary part of: elastic#161874 Fixes hover actions flaky cypress test. Flaky test runner execution after changes: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5293 --------- Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Angela Chuang <[email protected]> (cherry picked from commit cd6bc5e)
## Summary part of: elastic#161874 Fixes hover actions flaky cypress test. Flaky test runner execution after changes: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5293 --------- Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Angela Chuang <[email protected]>
## Summary part of: elastic#161874 Fixes hover actions flaky cypress test. Flaky test runner execution after changes: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5293 --------- Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Angela Chuang <[email protected]>
## Summary closes: #165692 part of: #161874 Hover action flakiness was fixed here #177577 This PR unskips the test and enables it for serverless execution. Flaky test runner execution after changes: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5401
Update: The noted tests are skipped. Unskip when fixed: #161898
I've noticed an increase in flakiness in our cypress tests. Looking at the #kibana-unsupported-ftrs-alerts slack channel along with the flakey test runner to compile the list below:
The text was updated successfully, but these errors were encountered: