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

[ci] Move cypress burn tests out of base pull request pipeline #169326

Merged
merged 2 commits into from
Oct 23, 2023

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Oct 18, 2023

Currently operations does not have a good mechanism for communicating soft failures, leading to confusion on pull requests. We're tracking support for soft failures at #169244.

For example, in #169309, it doesn't makes sense for the author to be responsible for fixing flakiness here, so we need to make it clear it can be ignored or find a way to skip this step in similar situations.

This moves the cypress burn out of the base pull request pipeline, behind the ci:all-cypress-suites and ci:cypress-burn labels. I'm open to all improvements here, not sure this quite finds the right balance between stability and noise.

@jbudz jbudz added Team:Operations Team label for Operations Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. ci:all-cypress-suites labels Oct 18, 2023
@jbudz jbudz marked this pull request as ready for review October 18, 2023 23:41
@jbudz jbudz requested a review from a team as a code owner October 18, 2023 23:41
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@elasticmachine
Copy link
Contributor

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

@jbudz jbudz added the release_note:skip Skip the PR/issue when compiling release notes label Oct 18, 2023
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Osquery Cypress Tests #1 / ALL - Add Integration Add and upgrade integration "after each" hook for "should add the old integration and be able to upgrade it" "after each" hook for "should add the old integration and be able to upgrade it"
  • [job] [logs] Serverless Osquery Cypress Tests #1 / ALL - Add Integration Add and upgrade integration should add the old integration and be able to upgrade it should add the old integration and be able to upgrade it

Metrics [docs]

✅ unchanged

History

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

@jbudz jbudz merged commit ccf07de into elastic:main Oct 23, 2023
29 checks passed
@kibanamachine kibanamachine added v8.12.0 backport:skip This commit does not require backporting labels Oct 23, 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:Operations Team label for Operations Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants