Skip to content

Unskip eql tests, only skip one#231934

Merged
nkhristinin merged 3 commits intoelastic:mainfrom
nkhristinin:unskip-some-eql
Aug 15, 2025
Merged

Unskip eql tests, only skip one#231934
nkhristinin merged 3 commits intoelastic:mainfrom
nkhristinin:unskip-some-eql

Conversation

@nkhristinin
Copy link
Copy Markdown
Contributor

@nkhristinin nkhristinin commented Aug 15, 2025

We do have 1 test, which is failing constantly (also locally).

We can unskip the whole file, and only keep this failing test skipped

@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/solutions/security/test/security_solution_api_integration/test_suites/detections_response/detection_engine/rule_execution_logic/eql/trial_license_complete_tier/configs/ess.config.ts: 100/100 tests passed.
[✅] x-pack/solutions/security/test/security_solution_api_integration/test_suites/detections_response/detection_engine/rule_execution_logic/eql/trial_license_complete_tier/configs/serverless.config.ts: 100/100 tests passed.

see run history

@nkhristinin nkhristinin changed the title - Unskip esql tests, only skip one Aug 15, 2025
@nkhristinin nkhristinin added release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.2.0 v9.1.3 v8.19.3 labels Aug 15, 2025
@nkhristinin nkhristinin marked this pull request as ready for review August 15, 2025 13:28
@nkhristinin nkhristinin requested a review from a team as a code owner August 15, 2025 13:29
@nkhristinin nkhristinin requested a review from rylnd August 15, 2025 13:29
Copy link
Copy Markdown
Contributor

@vitaliidm vitaliidm left a comment

Choose a reason for hiding this comment

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

NOTE: it's EQL test, not ES|QL how PR title states

@nkhristinin nkhristinin changed the title Unskip esql tests, only skip one Unskip eql tests, only skip one Aug 15, 2025
@nkhristinin nkhristinin enabled auto-merge (squash) August 15, 2025 13:42
…on/test_suites/detections_response/detection_engine/rule_execution_logic/eql/trial_license_complete_tier/eql_alert_suppression.ts

Co-authored-by: Ryland Herrick <ryalnd@gmail.com>
@nkhristinin nkhristinin requested a review from rylnd August 15, 2025 14:41
Copy link
Copy Markdown
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

Thanks! :shipit:

@nkhristinin nkhristinin merged commit d237d27 into elastic:main Aug 15, 2025
12 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19, 9.1

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

@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 15, 2025
We do have 1 test, which is failing constantly (also locally).

We can unskip the whole file, and only keep this failing test skipped

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

💔 Some backports could not be created

Status Branch Result
8.19
9.1 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.1:
- [ska] relocation security_solution_* FTR tests (#231416)
- [Detection Engine] Unskip some flaky tests, add better failure messages (#230318)

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 231934

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Aug 15, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [Unskip eql tests, only skip one
(#231934)](#231934)

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

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

<!--BACKPORT [{"author":{"name":"Khristinin
Nikita","email":"nikita.khristinin@elastic.co"},"sourceCommit":{"committedDate":"2025-08-15T18:04:18Z","message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.2.0","v9.1.3","v8.19.3"],"title":"Unskip
eql tests, only skip
one","number":231934,"url":"https://github.com/elastic/kibana/pull/231934","mergeCommit":{"message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/231934","number":231934,"mergeCommit":{"message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b"}},{"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"}]}]
BACKPORT-->

Co-authored-by: Khristinin Nikita <nikita.khristinin@elastic.co>
This was referenced Aug 18, 2025
This was referenced Aug 18, 2025
@nkhristinin
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
9.1

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

Questions ?

Please refer to the Backport tool documentation

nkhristinin added a commit to nkhristinin/kibana that referenced this pull request Aug 18, 2025
We do have 1 test, which is failing constantly (also locally).

We can unskip the whole file, and only keep this failing test skipped

(cherry picked from commit d237d27)
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Aug 18, 2025
We do have 1 test, which is failing constantly (also locally).

We can unskip the whole file, and only keep this failing test skipped
nkhristinin added a commit that referenced this pull request Aug 18, 2025
# Backport

This will backport the following commits from `main` to `9.1`:
- [Unskip eql tests, only skip one
(#231934)](#231934)

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

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

<!--BACKPORT [{"author":{"name":"Khristinin
Nikita","email":"nikita.khristinin@elastic.co"},"sourceCommit":{"committedDate":"2025-08-15T18:04:18Z","message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.2.0","v9.1.3","v8.19.3"],"title":"Unskip
eql tests, only skip
one","number":231934,"url":"https://github.com/elastic/kibana/pull/231934","mergeCommit":{"message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b"}},"sourceBranch":"main","suggestedTargetBranches":["9.1"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/231934","number":231934,"mergeCommit":{"message":"Unskip
eql tests, only skip one (#231934)\n\nWe do have 1 test, which is
failing constantly (also locally).\n\nWe can unskip the whole file, and
only keep this failing test
skipped","sha":"d237d27b15967c514d5c112390606dcb5126339b"}},{"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,"url":"https://github.com/elastic/kibana/pull/231982","number":231982,"state":"MERGED","mergeCommit":{"sha":"2433d18232c760c59a7076280f7e9b41bd4f52b9","message":"[8.19]
Unskip eql tests, only skip one (#231934) (#231982)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.19`:\n- [Unskip eql tests, only skip
one\n(#231934)](https://github.com/elastic/kibana/pull/231934)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Khristinin Nikita <nikita.khristinin@elastic.co>"}}]}] BACKPORT-->
qn895 pushed a commit to qn895/kibana that referenced this pull request Aug 26, 2025
We do have 1 test, which is failing constantly (also locally).

We can unskip the whole file, and only keep this failing test skipped
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 release_note:skip Skip the PR/issue when compiling release notes v8.19.3 v9.1.3 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants