Skip to content

[Security Solution] Unskip Related Integrations Cypress test#239532

Merged
nikitaindik merged 1 commit intoelastic:mainfrom
nikitaindik:unskip-related-integrations-cypress-tests
Oct 21, 2025
Merged

[Security Solution] Unskip Related Integrations Cypress test#239532
nikitaindik merged 1 commit intoelastic:mainfrom
nikitaindik:unskip-related-integrations-cypress-tests

Conversation

@nikitaindik
Copy link
Copy Markdown
Contributor

@nikitaindik nikitaindik commented Oct 17, 2025

Resolves: #239356

🟢 Flaky test runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

Summary

This PR unskips the Related Integrations Cypress tests. The tests were previously failing because one of the Fleet packages they depend on contained a YAML syntax error. That issue has now been fixed (PR), so the tests can be safely re-enabled.

The diff appears large, but the only functional change is the removal of .only. The rest comes from Prettier formatting.

@nikitaindik nikitaindik self-assigned this Oct 17, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#9560

[✅] Security Solution Rule Management - Cypress: 50/50 tests passed.
[✅] [Serverless] Security Solution Rule Management - Cypress: 50/50 tests passed.

see run history

@nikitaindik nikitaindik added v9.2.0 v8.18.7 v9.3.0 v9.1.6 v8.19.6 release_note:skip Skip the PR/issue when compiling release notes Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detection Rule Management Security Detection Rule Management Team backport:version Backport to applied version labels Feature:Related Integrations Security Solution Detection Rules Related Integrations feature and removed v8.18.7 labels Oct 20, 2025
@nikitaindik nikitaindik marked this pull request as ready for review October 20, 2025 09:20
@nikitaindik nikitaindik requested a review from a team as a code owner October 20, 2025 09:20
@nikitaindik nikitaindik requested a review from dplumlee October 20, 2025 09:20
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

@elasticmachine
Copy link
Copy Markdown
Contributor

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

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management)

@nikitaindik nikitaindik force-pushed the unskip-related-integrations-cypress-tests branch from 108cb9e to 5c984b9 Compare October 20, 2025 09:22
Copy link
Copy Markdown
Contributor

@banderror banderror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @nikitaindik!

@banderror banderror removed the request for review from dplumlee October 20, 2025 09:34
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 20, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

cc @nikitaindik

@nikitaindik nikitaindik enabled auto-merge (squash) October 21, 2025 08:27
@nikitaindik nikitaindik merged commit 1bfd3c1 into elastic:main Oct 21, 2025
12 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19, 9.1, 9.2

https://github.com/elastic/kibana/actions/runs/18678580887

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 21, 2025
…#239532)

**Resolves: elastic#239356

🟢 Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

# Summary
This PR unskips the Related Integrations Cypress tests. The tests were
previously failing because one of the Fleet packages they depend on
contained a YAML syntax error. That issue has now been fixed
([PR](elastic/integrations#15669)), so the tests
can be safely re-enabled.

The diff appears large, but the only functional change is the removal of
`.only`. The rest comes from Prettier formatting.

(cherry picked from commit 1bfd3c1)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 21, 2025
…#239532)

**Resolves: elastic#239356

🟢 Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

# Summary
This PR unskips the Related Integrations Cypress tests. The tests were
previously failing because one of the Fleet packages they depend on
contained a YAML syntax error. That issue has now been fixed
([PR](elastic/integrations#15669)), so the tests
can be safely re-enabled.

The diff appears large, but the only functional change is the removal of
`.only`. The rest comes from Prettier formatting.

(cherry picked from commit 1bfd3c1)
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 Some backports could not be created

Status Branch Result
8.19 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.19:
- [Security Solution] [SKA] Moved cypress folder (#230198)
9.1
9.2

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 239532

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 21, 2025
…239532) (#239869)

# Backport

This will backport the following commits from `main` to `9.2`:
- [[Security Solution] Unskip Related Integrations Cypress test
(#239532)](#239532)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"nikita.indik@elastic.co"},"sourceCommit":{"committedDate":"2025-10-21T09:00:12Z","message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Related
Integrations","backport:version","v9.2.0","v9.3.0","v9.1.6","v8.19.6"],"title":"[Security
Solution] Unskip Related Integrations Cypress
test","number":239532,"url":"https://github.com/elastic/kibana/pull/239532","mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},"sourceBranch":"main","suggestedTargetBranches":["9.2","9.1","8.19"],"targetPullRequestStates":[{"branch":"9.2","label":"v9.2.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239532","number":239532,"mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},{"branch":"9.1","label":"v9.1.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Nikita Indik <nikita.indik@elastic.co>
kibanamachine added a commit that referenced this pull request Oct 21, 2025
…239532) (#239868)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[Security Solution] Unskip Related Integrations Cypress test
(#239532)](#239532)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"nikita.indik@elastic.co"},"sourceCommit":{"committedDate":"2025-10-21T09:00:12Z","message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Related
Integrations","backport:version","v9.2.0","v9.3.0","v9.1.6","v8.19.6"],"title":"[Security
Solution] Unskip Related Integrations Cypress
test","number":239532,"url":"https://github.com/elastic/kibana/pull/239532","mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},"sourceBranch":"main","suggestedTargetBranches":["9.2","9.1","8.19"],"targetPullRequestStates":[{"branch":"9.2","label":"v9.2.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239532","number":239532,"mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},{"branch":"9.1","label":"v9.1.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Nikita Indik <nikita.indik@elastic.co>
@nikitaindik
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

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

Questions ?

Please refer to the Backport tool documentation

nikitaindik added a commit to nikitaindik/kibana that referenced this pull request Oct 21, 2025
…#239532)

**Resolves: elastic#239356

🟢 Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

# Summary
This PR unskips the Related Integrations Cypress tests. The tests were
previously failing because one of the Fleet packages they depend on
contained a YAML syntax error. That issue has now been fixed
([PR](elastic/integrations#15669)), so the tests
can be safely re-enabled.

The diff appears large, but the only functional change is the removal of
`.only`. The rest comes from Prettier formatting.

(cherry picked from commit 1bfd3c1)

# Conflicts:
#	x-pack/solutions/security/test/security_solution_cypress/cypress/e2e/detection_response/rule_management/related_integrations/related_integrations.cy.ts
nikitaindik added a commit that referenced this pull request Oct 22, 2025
…239532) (#239926)

ℹ️ The diff appears large, but the only functional change is the removal
of `.only`. The rest comes from Prettier formatting.

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Security Solution] Unskip Related Integrations Cypress test
(#239532)](#239532)

<!--- Backport version: 10.1.0 -->

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

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"nikita.indik@elastic.co"},"sourceCommit":{"committedDate":"2025-10-21T09:00:12Z","message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Related
Integrations","backport:version","v9.2.0","v9.3.0","v9.1.6","v8.19.6"],"title":"[Security
Solution] Unskip Related Integrations Cypress
test","number":239532,"url":"https://github.com/elastic/kibana/pull/239532","mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"9.2","label":"v9.2.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/239869","number":239869,"state":"MERGED","mergeCommit":{"sha":"36efd601450c4dbebcd441ae9639bbdfe6cf055f","message":"[9.2]
[Security Solution] Unskip Related Integrations Cypress test (#239532)
(#239869)\n\n# Backport\n\nThis will backport the following commits from
`main` to `9.2`:\n- [[Security Solution] Unskip Related Integrations
Cypress
test\n(#239532)](https://github.com/elastic/kibana/pull/239532)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Nikita Indik
<nikita.indik@elastic.co>"}},{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239532","number":239532,"mergeCommit":{"message":"[Security
Solution] Unskip Related Integrations Cypress test
(#239532)\n\n**Resolves:
https://github.com/elastic/kibana/issues/239356**\n\n🟢 Flaky test
runner:\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560\n\n#
Summary\nThis PR unskips the Related Integrations Cypress tests. The
tests were\npreviously failing because one of the Fleet packages they
depend on\ncontained a YAML syntax error. That issue has now been
fixed\n([PR](elastic/integrations#15669)), so
the tests\ncan be safely re-enabled.\n\nThe diff appears large, but the
only functional change is the removal of\n`.only`. The rest comes from
Prettier
formatting.","sha":"1bfd3c1046ae74295d77a33d68a0aa91071ce5e5"}},{"branch":"9.1","label":"v9.1.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/239868","number":239868,"state":"MERGED","mergeCommit":{"sha":"c8d163f6133593a534b53955c7e266a6408ed8e3","message":"[9.1]
[Security Solution] Unskip Related Integrations Cypress test (#239532)
(#239868)\n\n# Backport\n\nThis will backport the following commits from
`main` to `9.1`:\n- [[Security Solution] Unskip Related Integrations
Cypress
test\n(#239532)](https://github.com/elastic/kibana/pull/239532)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Nikita Indik
<nikita.indik@elastic.co>"}},{"branch":"8.19","label":"v8.19.6","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
nickpeihl pushed a commit to nickpeihl/kibana that referenced this pull request Oct 23, 2025
…#239532)

**Resolves: elastic#239356

🟢 Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

# Summary
This PR unskips the Related Integrations Cypress tests. The tests were
previously failing because one of the Fleet packages they depend on
contained a YAML syntax error. That issue has now been fixed
([PR](elastic/integrations#15669)), so the tests
can be safely re-enabled.

The diff appears large, but the only functional change is the removal of
`.only`. The rest comes from Prettier formatting.
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Oct 27, 2025
…#239532)

**Resolves: elastic#239356

🟢 Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/9560

# Summary
This PR unskips the Related Integrations Cypress tests. The tests were
previously failing because one of the Fleet packages they depend on
contained a YAML syntax error. That issue has now been fixed
([PR](elastic/integrations#15669)), so the tests
can be safely re-enabled.

The diff appears large, but the only functional change is the removal of
`.only`. The rest comes from Prettier formatting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:Related Integrations Security Solution Detection Rules Related Integrations feature release_note:skip Skip the PR/issue when compiling release notes Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.19.6 v9.1.6 v9.2.0 v9.3.0

Projects

None yet

4 participants