-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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][Endpoint][Cypress] Update endpoint cypress config to include videos #170499
[Security Solution][Endpoint][Cypress] Update endpoint cypress config to include videos #170499
Conversation
This is to help debugging test failures and flakiness
Pinging @elastic/security-defend-workflows (Team:Defend Workflows) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this Ash 👍
Just few hints from my side (unless you just wanted to enable this locally) :)
- would be nice to also include these videos on CI, therefore we need to upload artifacts
- limit the compression so we do not give too hard time to machine's CPU
- filter out the videos of successful specs, therefore just save the failed ones
review changed @tomsonpl
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested it only locally, but works great, thank you for adding this! 🙇
now let's hope for a flaky test in the PR to see it on the CI 🤞
💚 Build Succeeded
Metrics [docs]Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @ashokaditya |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, thanks for doing this :) hopefully will help us debug :)
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
… to include videos (elastic#170499) ## Summary Adds videos to failed cypress tests for debugging. (cherry picked from commit 1c25217) # Conflicts: # .buildkite/scripts/lifecycle/post_command.sh # x-pack/plugins/security_solution/public/management/cypress/cypress_base.config.ts
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
… config to include videos (#170499) (#170841) # Backport This will backport the following commits from `main` to `8.11`: - [[Security Solution][Endpoint][Cypress] Update endpoint cypress config to include videos (#170499)](#170499) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ash","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-11-06T16:04:42Z","message":"[Security Solution][Endpoint][Cypress] Update endpoint cypress config to include videos (#170499)\n\n## Summary\r\n\r\nAdds videos to failed cypress tests for debugging.","sha":"1c2521705a1ece1ebe87740a4c99c27b254630f9","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Defend Workflows","OLM Sprint","v8.11.0","v8.12.0"],"number":170499,"url":"https://github.com/elastic/kibana/pull/170499","mergeCommit":{"message":"[Security Solution][Endpoint][Cypress] Update endpoint cypress config to include videos (#170499)\n\n## Summary\r\n\r\nAdds videos to failed cypress tests for debugging.","sha":"1c2521705a1ece1ebe87740a4c99c27b254630f9"}},"sourceBranch":"main","suggestedTargetBranches":["8.11"],"targetPullRequestStates":[{"branch":"8.11","label":"v8.11.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/170499","number":170499,"mergeCommit":{"message":"[Security Solution][Endpoint][Cypress] Update endpoint cypress config to include videos (#170499)\n\n## Summary\r\n\r\nAdds videos to failed cypress tests for debugging.","sha":"1c2521705a1ece1ebe87740a4c99c27b254630f9"}}]}] BACKPORT--> Co-authored-by: Kibana Machine <[email protected]>
Summary
Adds videos to failed cypress tests for debugging.