[Security Solution] Using Chrome on CI for Cypress #183515
Conversation
|
/ci |
MadameSheema
added
Team: SecuritySolution
MadameSheema
requested review from
a team,
patrykkopycinski,
oatkiller,
maximpn and
banderror
as code owners
|
Pinging @elastic/security-solution (Team: SecuritySolution) |
|
/ci |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
## 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)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
|
/ci |
kibanamachine
added
the
backport missing
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
8 similar comments
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Summary
Fixes: #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.