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] skip failing related_integrations e2e test #154662

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

PhilippeOberti
Copy link
Contributor

Summary

The Security Solution e2e tests in the context installed integrations: Amazon CloudFront, AWS CloudTrail, System, enabled integrations: Amazon CloudFront, Aws Cloudfront, System are all failing in the before section, and more specifically within the installAwsCloudFrontWithPolicy method, at the very last line, where Cypress can't find the pop up.

It seems there an error
Screenshot 2023-04-10 at 11 14 47 AM

@PhilippeOberti PhilippeOberti requested review from a team as code owners April 10, 2023 16:27
@PhilippeOberti PhilippeOberti added Team:Threat Hunting Security Solution Threat Hunting Team Team:Threat Hunting:Investigations Security Solution Investigations Team v8.8.0 labels Apr 10, 2023
@elasticmachine
Copy link
Contributor

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

@PhilippeOberti PhilippeOberti added the release_note:skip Skip the PR/issue when compiling release notes label Apr 10, 2023
@kevinlog
Copy link
Contributor

Thanks for opening this. For some additional context, Defend Workflows is also seeing this test failure on this PR: #154638

Copy link
Member

@spong spong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the noise here @PhilippeOberti! I've created this issue (#154663) for tracking the skip on the rules side -- we'll get this flake taken care of before unskipping. 👍

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Security Solution Tests #2 / Alerts detection rules table auto-refresh should disable auto refresh when any rule selected and enable it after rules unselected

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 433 436 +3

Total ESLint disabled count

id before after diff
securitySolution 513 516 +3

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

@PhilippeOberti PhilippeOberti merged commit 813c2a3 into main Apr 10, 2023
@PhilippeOberti PhilippeOberti deleted the security-solution-skip-e2e-test branch April 10, 2023 19:18
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team Team:Threat Hunting Security Solution Threat Hunting Team v8.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants