Skip to content
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] Fix flakey Hover Actions test #173114

Closed
wants to merge 9 commits into from

Conversation

stephmilovic
Copy link
Contributor

@stephmilovic stephmilovic commented Dec 11, 2023

Summary

Fixes hover actions and overflow items tests using cypress-recurse.

Part of #161874

Flakey test runner build: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4511
There are a couple failures so far, but both seem to be something wrong with the server: Exited with status -1 (agent lost)

@stephmilovic stephmilovic added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore v8.13.0 labels Dec 11, 2023
@stephmilovic stephmilovic requested a review from a team as a code owner December 11, 2023 20:16
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@stephmilovic
Copy link
Contributor Author

@elasticmachine merge upstream

@semd
Copy link
Contributor

semd commented Jan 3, 2024

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Investigations - Security Solution Cypress Tests #6 / Correlation tab should update timeline after removing eql should update timeline after removing eql

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@stephmilovic
Copy link
Contributor Author

@semd The reason I haven't merged this is because although it passed the flakey test runner, on what would have been the final build it flaked on the hover actions test: https://buildkite.com/elastic/kibana-pull-request/builds/184115

I have not had the chance to look into this yet, may need more expertise from @MadameSheema or @lgestc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore Team:Threat Hunting Security Solution Threat Hunting Team v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants