Skip to content

Comments

[Security Solution] Enable accidentally skipped prebuilt rules package integration tests#226759

Merged
maximpn merged 2 commits intoelastic:mainfrom
maximpn:enable-prebuilt-rule-package-tests
Jul 10, 2025
Merged

[Security Solution] Enable accidentally skipped prebuilt rules package integration tests#226759
maximpn merged 2 commits intoelastic:mainfrom
maximpn:enable-prebuilt-rule-package-tests

Conversation

@maximpn
Copy link
Contributor

@maximpn maximpn commented Jul 7, 2025

Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

Details

The integration tests were accidentally skipped by missing loadTestFile(require.resolve('./prebuilt_rules_package')); line in #225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in #224676. It means that users may observe this error under low-tier license only by using the API directly.

@maximpn maximpn self-assigned this Jul 7, 2025
@maximpn maximpn added test release_note:skip Skip the PR/issue when compiling release notes v9.0.0 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 Feature:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area backport:version Backport to applied version labels v9.1.0 v8.19.0 labels Jul 7, 2025
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/common/configs/ess_basic_license.config.ts: 200/200 tests passed.

see run history

@maximpn maximpn force-pushed the enable-prebuilt-rule-package-tests branch from 667cf0c to 5f86d25 Compare July 9, 2025 07:11
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/common/configs/edge_cases/ess_trial_license.config.ts: 200/200 tests passed.

see run history

@maximpn maximpn force-pushed the enable-prebuilt-rule-package-tests branch from 5f86d25 to 52d10f0 Compare July 9, 2025 09:31
@maximpn maximpn added the v8.18.0 label Jul 9, 2025
@maximpn maximpn marked this pull request as ready for review July 9, 2025 12:45
@maximpn maximpn requested review from a team as code owners July 9, 2025 12:45
@maximpn maximpn requested a review from dplumlee July 9, 2025 12:45
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@banderror banderror removed the request for review from dplumlee July 10, 2025 13:05
@banderror banderror requested a review from nikitaindik July 10, 2025 13:05
Copy link
Contributor

@dplumlee dplumlee left a comment

Choose a reason for hiding this comment

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

Rule management changes lgtm

@maximpn maximpn removed the request for review from nikitaindik July 10, 2025 16:10
@maximpn maximpn enabled auto-merge (squash) July 10, 2025 16:10
@maximpn maximpn merged commit 189cfb5 into elastic:main Jul 10, 2025
12 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.18, 8.19, 9.0, 9.1

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @maximpn

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 10, 2025
…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.

(cherry picked from commit 189cfb5)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 10, 2025
…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.

(cherry picked from commit 189cfb5)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.18 Backport failed because of merge conflicts
8.19
9.0 Backport failed because of merge conflicts
9.1

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 226759

Questions ?

Please refer to the Backport tool documentation

@maximpn maximpn deleted the enable-prebuilt-rule-package-tests branch July 10, 2025 19:03
kibanamachine added a commit that referenced this pull request Jul 10, 2025
…package integration tests (#226759) (#227506)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[Security Solution] Enable accidentally skipped prebuilt rules
package integration tests
(#226759)](#226759)

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

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

<!--BACKPORT [{"author":{"name":"Maxim
Palenov","email":"maxim.palenov@elastic.co"},"sourceCommit":{"committedDate":"2025-07-10T17:32:29Z","message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","backport:version","v9.1.0","v8.19.0","v9.2.0","v8.18.4","v9.0.4"],"title":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests","number":226759,"url":"https://github.com/elastic/kibana/pull/226759","mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","8.18","9.0"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226759","number":226759,"mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},{"branch":"8.18","label":"v8.18.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Maxim Palenov <maxim.palenov@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 10, 2025
… package integration tests (#226759) (#227505)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Security Solution] Enable accidentally skipped prebuilt rules
package integration tests
(#226759)](#226759)

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

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

<!--BACKPORT [{"author":{"name":"Maxim
Palenov","email":"maxim.palenov@elastic.co"},"sourceCommit":{"committedDate":"2025-07-10T17:32:29Z","message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","backport:version","v9.1.0","v8.19.0","v9.2.0","v8.18.4","v9.0.4"],"title":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests","number":226759,"url":"https://github.com/elastic/kibana/pull/226759","mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19","8.18","9.0"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226759","number":226759,"mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},{"branch":"8.18","label":"v8.18.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Maxim Palenov <maxim.palenov@elastic.co>
maximpn added a commit to maximpn/kibana that referenced this pull request Jul 11, 2025
…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.

(cherry picked from commit 189cfb5)

# Conflicts:
#	x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/common/index.ts
maximpn added a commit to maximpn/kibana that referenced this pull request Jul 11, 2025
…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.

(cherry picked from commit 189cfb5)

# Conflicts:
#	x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/common/index.ts
@maximpn
Copy link
Contributor Author

maximpn commented Jul 11, 2025

💚 All backports created successfully

Status Branch Result
9.0
8.18

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

Questions ?

Please refer to the Backport tool documentation

maximpn added a commit that referenced this pull request Jul 11, 2025
…package integration tests (#226759) (#227565)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[Security Solution] Enable accidentally skipped prebuilt rules
package integration tests
(#226759)](#226759)

<!--- Backport version: 10.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Maxim
Palenov","email":"maxim.palenov@elastic.co"},"sourceCommit":{"committedDate":"2025-07-10T17:32:29Z","message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","backport:version","v9.1.0","v8.19.0","v9.2.0","v8.18.4","v9.0.4"],"title":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests","number":226759,"url":"https://github.com/elastic/kibana/pull/226759","mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","9.0"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227506","number":227506,"state":"MERGED","mergeCommit":{"sha":"93f21228e7cf49d7f93ea857c2a815203083181d","message":"[9.1]
[Security Solution] Enable accidentally skipped prebuilt rules package
integration tests (#226759) (#227506)\n\n# Backport\n\nThis will
backport the following commits from `main` to `9.1`:\n- [[Security
Solution] Enable accidentally skipped prebuilt rules\npackage
integration
tests\n(#226759)](https://github.com/elastic/kibana/pull/226759)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Maxim Palenov
<maxim.palenov@elastic.co>"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227505","number":227505,"state":"MERGED","mergeCommit":{"sha":"4c9c5dfadc63a489fbdba79f18f94134cb08c80b","message":"[8.19]
[Security Solution] Enable accidentally skipped prebuilt rules package
integration tests (#226759) (#227505)\n\n# Backport\n\nThis will
backport the following commits from `main` to `8.19`:\n- [[Security
Solution] Enable accidentally skipped prebuilt rules\npackage
integration
tests\n(#226759)](https://github.com/elastic/kibana/pull/226759)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Maxim Palenov
<maxim.palenov@elastic.co>"}},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226759","number":226759,"mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},{"branch":"8.18","label":"v8.18.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
maximpn added a commit that referenced this pull request Jul 12, 2025
… package integration tests (#226759) (#227566)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[Security Solution] Enable accidentally skipped prebuilt rules
package integration tests
(#226759)](#226759)

<!--- Backport version: 10.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Maxim
Palenov","email":"maxim.palenov@elastic.co"},"sourceCommit":{"committedDate":"2025-07-10T17:32:29Z","message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","backport:version","v9.1.0","v8.19.0","v9.2.0","v8.18.4","v9.0.4"],"title":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests","number":226759,"url":"https://github.com/elastic/kibana/pull/226759","mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","9.0"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227506","number":227506,"state":"MERGED","mergeCommit":{"sha":"93f21228e7cf49d7f93ea857c2a815203083181d","message":"[9.1]
[Security Solution] Enable accidentally skipped prebuilt rules package
integration tests (#226759) (#227506)\n\n# Backport\n\nThis will
backport the following commits from `main` to `9.1`:\n- [[Security
Solution] Enable accidentally skipped prebuilt rules\npackage
integration
tests\n(#226759)](https://github.com/elastic/kibana/pull/226759)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Maxim Palenov
<maxim.palenov@elastic.co>"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227505","number":227505,"state":"MERGED","mergeCommit":{"sha":"4c9c5dfadc63a489fbdba79f18f94134cb08c80b","message":"[8.19]
[Security Solution] Enable accidentally skipped prebuilt rules package
integration tests (#226759) (#227505)\n\n# Backport\n\nThis will
backport the following commits from `main` to `8.19`:\n- [[Security
Solution] Enable accidentally skipped prebuilt rules\npackage
integration
tests\n(#226759)](https://github.com/elastic/kibana/pull/226759)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Maxim Palenov
<maxim.palenov@elastic.co>"}},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226759","number":226759,"mergeCommit":{"message":"[Security
Solution] Enable accidentally skipped prebuilt rules package integration
tests (#226759)\n\n## Summary\n\nThis PR enables accidentally skipped
prebuilt rules package integration tests.\n\n## Details\n\nThe
integration tests were accidentally skipped by missing
`loadTestFile(require.resolve('./prebuilt_rules_package'));` line in
#225495 as the result of merge
conflicts.\n\nEnabling those integration tests revealed an issue with
the installing all prebuilt rules integration tests. Since the prebuilt
rules package contains ML rules all rules installation fails under the
low-tier license as ML rules require high-tier license. So this single
test was changed to run under high-tier license.\n\nJust for the record,
ML related UI issues has been addressed in
#224676. It means that users may
observe this error under low-tier license only by using the API
directly.","sha":"189cfb5bce4e490e73e1c16faf8bddbd203d8b2a"}},{"branch":"8.18","label":"v8.18.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.
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:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area 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. test v8.18.4 v8.19.0 v9.0.4 v9.1.0 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants