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.

Sign up for GitHub

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

Jump to bottom

[8.14] [Security Solution] Using Chrome on CI for Cypress (#183515) #183667

Open
wants to merge 4 commits into
base: 8.14
Choose a base branch
from
Open

[8.14] [Security Solution] Using Chrome on CI for Cypress (#183515) #183667

wants to merge 4 commits into from
+105 −20

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

@MadameSheema
[Security Solution] Using Chrome on CI for Cypress (elastic#183515)
02985a6 
## Summary

Fixes: elastic#183098

Lately we have experienced some issues with our Cypress tests, where we
are seeing that tests are passing but the output is marked as a failure.

@patrykkopycinski has done a great job trying to fix the issue but
lately we have seen that is still reproducible.

The issue seems to be related directly with electron, so in this PR we
reenabling chrome on CI to try to fix it.

In order to do so, some adjustments has been introduced to woraround the
chrome out of memory crashes that we faced before and was the main
reason to move to electron.

---------

Co-authored-by: Patryk Kopycinski <contact@patrykkopycinski.com>
(cherry picked from commit a72ddc0)
@kibanamachine kibanamachine enabled auto-merge (squash) May 16, 2024 17:27
@kibanamachine kibanamachine mentioned this pull request May 16, 2024
Merged
@MadameSheema
Copy link
Member

@elasticmachine merge upstream

@MadameSheema
Copy link
Member

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented May 17, 2024 •
edited

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Detection Engine - Exceptions - Security Solution Cypress Tests #1 / Manage exceptions Add, edit and delete exception item should create exception item from Shared Exception List page, linked to a Shared List and validate Edit/delete in list detail page should create exception item from Shared Exception List page, linked to a Shared List and validate Edit/delete in list detail page
  • [job] [logs] Detection Engine - Exceptions - Security Solution Cypress Tests #1 / Manage exceptions Add, edit and delete exception item should not allow to add huge text as a comment should not allow to add huge text as a comment

Metrics [docs]

✅ unchanged

History

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

cc @MadameSheema

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions approved these changes

Assignees

@MadameSheema MadameSheema

Labels
backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kibanamachine @MadameSheema @kibana-ci