[Security Solution] Unskip Cypress tests for rule execution log#232023
Merged
banderror merged 2 commits intoelastic:mainfrom Aug 18, 2025
Merged
Conversation
banderror
added
test
release_note:skip
73 tasks
Contributor
|
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Contributor
|
Pinging @elastic/security-solution (Team: SecuritySolution) |
Contributor
|
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
Contributor
💚 Build Succeeded
Metrics [docs]
History
cc @banderror |
Contributor
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#9162[✅] Security Solution Rule Management - Cypress: 100/100 tests passed. |
xcrzx
reviewed
Aug 18, 2025
Comment on lines
+51
to
+60
| cy.waitUntil( | ||
| () => { | ||
| cy.log('Waiting for execution logs to appear in execution log table'); | ||
| refreshRuleExecutionTable(); | ||
| return getExecutionLogTableRow().then((rows) => { | ||
| return rows.length > 0; | ||
| }); | ||
| }, | ||
| { interval: 5000, timeout: 20000 } | ||
| ); |
Contributor
There was a problem hiding this comment.
Why do we need to wait for a rule execution before scheduling a rule run? These actions don’t seem related. Would be nice to add a comment here explaining why this step is necessary.
Contributor
Author
There was a problem hiding this comment.
@xcrzx This block of code is a replacement for the removed waitForAlertsToPopulate() which was a weird logic for waiting for a rule execution to finish, and wasn't working anyways.
My goal was to just fix and unskip the test while keeping its original logic and structure introduced by @nkhristinin. Refactoring it or changing what we test here would be out of the scope of this PR. That said, I agree that "These actions don’t seem related" and we need to rework the whole test. I'd also argue that calling the manual rule run API should be owned by the DE team, so even more consideration and rework might be needed here.
I can create a ticket for follow-up work. Is there anything you want me to do in this PR?
Contributor
There was a problem hiding this comment.
Okay, I thought you might know the reason, but thanks for the explanation anyway.
This test kind of sits between the responsibilities of two teams, so it’s not that easy to say who should own it. But if it relies on intricate knowledge of how manual runs work under the hood, it would be better to transfer it to DE.
Contributor
Author
There was a problem hiding this comment.
This test kind of sits between the responsibilities of two teams, so it’s not that easy to say who should own it. But if it relies on intricate knowledge of how manual runs work under the hood, it would be better to transfer it to DE.
Yeah, I agree. Ok, I'll get back to this in the coming days and open a follow-up ticket.
xcrzx
approved these changes
Aug 18, 2025
Contributor
|
Starting backport for target branches: 8.18, 8.19, 9.0, 9.1 https://github.com/elastic/kibana/actions/runs/17047103087 |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 18, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 18, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 18, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 18, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
Contributor
💚 All backports created successfully
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
Aug 18, 2025
…#232023) (#232134) # Backport This will backport the following commits from `main` to `9.1`: - [[Security Solution] Unskip Cypress tests for rule execution log (#232023)](#232023) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Georgii Gorbachev","email":"georgii.gorbachev@elastic.co"},"sourceCommit":{"committedDate":"2025-08-18T16:57:40Z","message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections and Resp","Team: SecuritySolution","Feature:Rule Monitoring","Team:Detection Rule Management","Feature:Rule Details","backport:version","v9.2.0","v9.1.3","v8.19.3","v9.0.6","v8.18.6"],"title":"[Security Solution] Unskip Cypress tests for rule execution log","number":232023,"url":"https://github.com/elastic/kibana/pull/232023","mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","9.0","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/232023","number":232023,"mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},{"branch":"9.1","label":"v9.1.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Georgii Gorbachev <georgii.gorbachev@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Aug 18, 2025
#232023) (#232132) # Backport This will backport the following commits from `main` to `8.19`: - [[Security Solution] Unskip Cypress tests for rule execution log (#232023)](#232023) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Georgii Gorbachev","email":"georgii.gorbachev@elastic.co"},"sourceCommit":{"committedDate":"2025-08-18T16:57:40Z","message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections and Resp","Team: SecuritySolution","Feature:Rule Monitoring","Team:Detection Rule Management","Feature:Rule Details","backport:version","v9.2.0","v9.1.3","v8.19.3","v9.0.6","v8.18.6"],"title":"[Security Solution] Unskip Cypress tests for rule execution log","number":232023,"url":"https://github.com/elastic/kibana/pull/232023","mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","9.0","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/232023","number":232023,"mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},{"branch":"9.1","label":"v9.1.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Georgii Gorbachev <georgii.gorbachev@elastic.co>
viduni94
pushed a commit
to viduni94/kibana
that referenced
this pull request
Aug 18, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels.
banderror
added a commit
to banderror/kibana
that referenced
this pull request
Aug 19, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
banderror
added a commit
to kibanamachine/kibana
that referenced
this pull request
Aug 19, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels. (cherry picked from commit c09e281)
kibanamachine
added a commit
that referenced
this pull request
Aug 19, 2025
…#232023) (#232133) # Backport This will backport the following commits from `main` to `9.0`: - [[Security Solution] Unskip Cypress tests for rule execution log (#232023)](#232023) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Georgii Gorbachev","email":"georgii.gorbachev@elastic.co"},"sourceCommit":{"committedDate":"2025-08-18T16:57:40Z","message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections and Resp","Team: SecuritySolution","Feature:Rule Monitoring","Team:Detection Rule Management","Feature:Rule Details","backport:version","v9.2.0","v9.1.3","v8.19.3","v9.0.6","v8.18.6"],"title":"[Security Solution] Unskip Cypress tests for rule execution log","number":232023,"url":"https://github.com/elastic/kibana/pull/232023","mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","9.0","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/232023","number":232023,"mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},{"branch":"9.1","label":"v9.1.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> --------- Co-authored-by: Georgii Gorbachev <georgii.gorbachev@elastic.co>
kibanamachine
added a commit
that referenced
this pull request
Aug 19, 2025
#232023) (#232131) # Backport This will backport the following commits from `main` to `8.18`: - [[Security Solution] Unskip Cypress tests for rule execution log (#232023)](#232023) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Georgii Gorbachev","email":"georgii.gorbachev@elastic.co"},"sourceCommit":{"committedDate":"2025-08-18T16:57:40Z","message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections and Resp","Team: SecuritySolution","Feature:Rule Monitoring","Team:Detection Rule Management","Feature:Rule Details","backport:version","v9.2.0","v9.1.3","v8.19.3","v9.0.6","v8.18.6"],"title":"[Security Solution] Unskip Cypress tests for rule execution log","number":232023,"url":"https://github.com/elastic/kibana/pull/232023","mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","9.0","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/232023","number":232023,"mergeCommit":{"message":"[Security Solution] Unskip Cypress tests for rule execution log (#232023)\n\n**Partially addresses: https://github.com/elastic/kibana/issues/229688**\n**Resolves: https://github.com/elastic/kibana/issues/184360**\n\n## Summary\n\nThis PR fixes and unskips the following tests:\n\n-\n`x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts`\n\n## Flaky test runs\n\n- [100x ESS + 100x\nServerless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162)\n🟢\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"c09e281c9ff08a96773b9d1fe59697a8fde7cbe7"}},{"branch":"9.1","label":"v9.1.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> --------- Co-authored-by: Georgii Gorbachev <georgii.gorbachev@elastic.co>
qn895
pushed a commit
to qn895/kibana
that referenced
this pull request
Aug 26, 2025
…tic#232023) **Partially addresses: elastic#229688 **Resolves: elastic#184360 ## Summary This PR fixes and unskips the following tests: - `x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.ts` ## Flaky test runs - [100x ESS + 100x Serverless](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9162) 🟢 ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Partially addresses: #229688
Resolves: #184360
Summary
This PR fixes and unskips the following tests:
x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/rule_details/execution_log.cy.tsFlaky test runs
Checklist
release_note:*label is applied per the guidelinesbackport:*labels.