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] unskip endpoint alerts cypress tests #174493

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

joeypoon
Copy link
Member

@joeypoon joeypoon commented Jan 9, 2024

Summary

Unskip endpoint alerts cypress test. Various enhancements to our tests were previously made, this is just retroactively unskipping after verifying with flaky test runner.

Flaky test runs:

For maintainers

@joeypoon joeypoon force-pushed the fix/flaky-cypress-endpoint-alerts branch from fec0550 to d461535 Compare January 9, 2024 16:06
@joeypoon joeypoon marked this pull request as ready for review January 9, 2024 16:09
@joeypoon joeypoon requested a review from a team as a code owner January 9, 2024 16:09
@joeypoon joeypoon requested review from tomsonpl and parkiino January 9, 2024 16:09
@joeypoon joeypoon added release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.12.0 labels Jan 9, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-defend-workflows (Team:Defend Workflows)

@joeypoon joeypoon enabled auto-merge (squash) January 10, 2024 15:15
@joeypoon
Copy link
Member Author

/ci

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Defend Workflows Cypress Tests on Serverless #6 / User Roles for Security Complete PLI with Endpoint Complete addon for role: t1_analyst "before all" hook for "should have READ access to Endpoint list page" "before all" hook for "should have READ access to Endpoint list page"

Metrics [docs]

✅ unchanged

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

@joeypoon joeypoon merged commit f141534 into elastic:main Jan 10, 2024
39 of 40 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 10, 2024
)

## Summary

Unskip endpoint alerts cypress test. Various enhancements to our tests
were previously made, this is just retroactively unskipping after
verifying with flaky test runner.

Flaky test runs:
-
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4798
(50x ✅ )

### For maintainers

- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit f141534)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.12

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jan 10, 2024
) (#174641)

# Backport

This will backport the following commits from `main` to `8.12`:
- [[Security Solution] unskip endpoint alerts cypress tests
(#174493)](#174493)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Joey F.
Poon","email":"joey.poon@elastic.co"},"sourceCommit":{"committedDate":"2024-01-10T17:21:10Z","message":"[Security
Solution] unskip endpoint alerts cypress tests (#174493)\n\n##
Summary\r\n\r\nUnskip endpoint alerts cypress test. Various enhancements
to our tests\r\nwere previously made, this is just retroactively
unskipping after\r\nverifying with flaky test runner.\r\n\r\nFlaky test
runs:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4798\r\n(50x
✅ )\r\n\r\n\r\n### For maintainers\r\n\r\n- [x] This was checked for
breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"f1415345bac26e594ab76fa223f0b439e1920be8","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Defend
Workflows","v8.12.0","v8.13.0"],"title":"[Security Solution] unskip
endpoint alerts cypress
tests","number":174493,"url":"https://github.com/elastic/kibana/pull/174493","mergeCommit":{"message":"[Security
Solution] unskip endpoint alerts cypress tests (#174493)\n\n##
Summary\r\n\r\nUnskip endpoint alerts cypress test. Various enhancements
to our tests\r\nwere previously made, this is just retroactively
unskipping after\r\nverifying with flaky test runner.\r\n\r\nFlaky test
runs:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4798\r\n(50x
✅ )\r\n\r\n\r\n### For maintainers\r\n\r\n- [x] This was checked for
breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"f1415345bac26e594ab76fa223f0b439e1920be8"}},"sourceBranch":"main","suggestedTargetBranches":["8.12"],"targetPullRequestStates":[{"branch":"8.12","label":"v8.12.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.13.0","branchLabelMappingKey":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/174493","number":174493,"mergeCommit":{"message":"[Security
Solution] unskip endpoint alerts cypress tests (#174493)\n\n##
Summary\r\n\r\nUnskip endpoint alerts cypress test. Various enhancements
to our tests\r\nwere previously made, this is just retroactively
unskipping after\r\nverifying with flaky test runner.\r\n\r\nFlaky test
runs:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4798\r\n(50x
✅ )\r\n\r\n\r\n### For maintainers\r\n\r\n- [x] This was checked for
breaking API changes and was
[labeled\r\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"f1415345bac26e594ab76fa223f0b439e1920be8"}}]}]
BACKPORT-->

Co-authored-by: Joey F. Poon <joey.poon@elastic.co>
@joeypoon joeypoon deleted the fix/flaky-cypress-endpoint-alerts branch January 10, 2024 19:32
delanni pushed a commit to delanni/kibana that referenced this pull request Jan 11, 2024
)

## Summary

Unskip endpoint alerts cypress test. Various enhancements to our tests
were previously made, this is just retroactively unskipping after
verifying with flaky test runner.

Flaky test runs:
-
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/4798
(50x ✅ )


### For maintainers

- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.12.0 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants