Skip to content

Change cases tests ownership #241134

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
darnautov merged 10 commits into from
Nov 6, 2025
Merged

Change cases tests ownership #241134

darnautov merged 10 commits into from
Nov 6, 2025

Conversation

@darnautov
Copy link
Contributor

@darnautov darnautov commented Oct 29, 2025 •
edited by kibanamachine
Loading

Summary

  • Set cases tests ownership (/x-pack/platform/test/functional/apps/cases_attachments/**) to @elastic/kibana-cases
  • Move ML and AIOps cases tests under the cases_attachments folder

Closes #202342
Closes #196307
Closes #200836

@darnautov darnautov added :ml Team:ML Team label for ML (also use :ml) t// Team:Cases Security Solution Cases team test_ui_functional labels Oct 29, 2025
@darnautov darnautov marked this pull request as ready for review October 30, 2025 10:58
@darnautov darnautov requested a review from a team as a code owner October 30, 2025 10:58
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 30, 2025

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 30, 2025

Pinging @elastic/kibana-cases (Team:Cases)

@darnautov darnautov added backport:all-open Backport to all branches that could still receive a release release_note:skip Skip the PR/issue when compiling release notes labels Oct 30, 2025
peteharverson
peteharverson reviewed Oct 31, 2025
View reviewed changes
x-pack/platform/test/functional/apps/aiops/cases/change_point_detection_cases.ts Outdated
import type { FtrProviderContext } from '../../ftr_provider_context';
import { USER } from '../../../api_integration/services/ml/security_common';
import type { FtrProviderContext } from '../../../ftr_provider_context';
import { USER } from '../../../../api_integration/services/ml/security_common';
Copy link
Contributor

@peteharverson peteharverson Oct 31, 2025

Choose a reason for hiding this comment

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

Note when backporting to 8.19, there will be a skip to remove from the test suite in this file.

x-pack/platform/test/functional/apps/aiops/log_pattern_analysis.ts
import type { FtrProviderContext } from '../../ftr_provider_context';
import { USER } from '../../../api_integration/services/ml/security_common';

export default function ({ getPageObjects, getService }: FtrProviderContext) {
Copy link
Contributor

@peteharverson peteharverson Oct 31, 2025

Choose a reason for hiding this comment

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

Note when backporting to 8.19 there will be a skip to remove in this file.

x-pack/platform/test/functional/apps/ml/anomaly_detection_result_views/anomaly_explorer.ts
import { SWIMLANE_TYPE } from '@kbn/ml-plugin/server/embeddable/schemas';
import type { AnomalySwimlaneEmbeddableStateViewBy } from '@kbn/ml-plugin/public/embeddables/anomaly_swimlane/types';
import { USER } from '../../../services/ml/security_common';
import type { FtrProviderContext } from '../../../ftr_provider_context';
Copy link
Contributor

@peteharverson peteharverson Oct 31, 2025

Choose a reason for hiding this comment

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

Note when backporting to 8.19 there will be a skip to remove from this file.

@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 5, 2025

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

peteharverson
peteharverson approved these changes Nov 6, 2025
View reviewed changes
Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@darnautov darnautov merged commit 4831400 into elastic:main Nov 6, 2025
12 checks passed
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 6, 2025

Starting backport for target branches: 8.19, 9.1, 9.2

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

@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 6, 2025

💔 All backports failed

Status Branch Result
❌ 8.19 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.19:
- [ska] relocate esArchives to platform/test/fixtures (#229750)
❌ 9.1 Backport failed because of merge conflicts
❌ 9.2 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 241134

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 7, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 7, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 241134 locally
cc: @darnautov

@darnautov darnautov mentioned this pull request Nov 7, 2025
Merged
darnautov added a commit to darnautov/kibana that referenced this pull request Nov 7, 2025
@darnautov
Change cases tests ownership (elastic#241134)
2d10fef 
## Summary

- Set cases tests ownership
(`/x-pack/platform/test/functional/apps/cases_attachments/**`) to
@elastic/kibana-cases
- Move ML and AIOps cases tests under the `cases_attachments` folder

Closes elastic#202342
Closes elastic#196307
Closes elastic#200836

(cherry picked from commit 4831400)

# Conflicts:
#	x-pack/platform/test/functional/apps/ml/anomaly_detection_result_views/anomaly_explorer.ts
@darnautov darnautov mentioned this pull request Nov 7, 2025
Merged
darnautov added a commit to darnautov/kibana that referenced this pull request Nov 7, 2025
@darnautov
Change cases tests ownership (elastic#241134)
bad2f27 
## Summary

- Set cases tests ownership
(`/x-pack/platform/test/functional/apps/cases_attachments/**`) to
@elastic/kibana-cases
- Move ML and AIOps cases tests under the `cases_attachments` folder

Closes elastic#202342
Closes elastic#196307
Closes elastic#200836

(cherry picked from commit 4831400)

# Conflicts:
#	x-pack/platform/test/functional/apps/aiops/log_pattern_analysis.ts
#	x-pack/platform/test/functional/apps/cases_attachments/aiops/change_point_detection_cases.ts
#	x-pack/platform/test/functional/apps/cases_attachments/aiops/log_rate_analysis_cases.ts
#	x-pack/platform/test/functional/apps/ml/anomaly_detection_result_views/anomaly_explorer.ts
darnautov added a commit to darnautov/kibana that referenced this pull request Nov 7, 2025
@darnautov
Change cases tests ownership (elastic#241134)
fe077b3 
## Summary

- Set cases tests ownership
(`/x-pack/platform/test/functional/apps/cases_attachments/**`) to
@elastic/kibana-cases
- Move ML and AIOps cases tests under the `cases_attachments` folder

Closes elastic#202342
Closes elastic#196307
Closes elastic#200836

(cherry picked from commit 4831400)

# Conflicts:
#	x-pack/platform/test/functional/apps/aiops/log_pattern_analysis.ts
#	x-pack/platform/test/functional/apps/cases_attachments/aiops/change_point_detection_cases.ts
#	x-pack/platform/test/functional/apps/cases_attachments/aiops/log_rate_analysis_cases.ts
#	x-pack/platform/test/functional/apps/ml/anomaly_detection_result_views/anomaly_explorer.ts
@darnautov darnautov mentioned this pull request Nov 7, 2025
Merged
@darnautov
Copy link
Contributor Author

darnautov commented Nov 7, 2025

💚 All backports created successfully

Status Branch Result
✅ 9.2
✅ 9.1
✅ 8.19

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

Questions ?

Please refer to the Backport tool documentation

darnautov added a commit that referenced this pull request Nov 7, 2025
@darnautov
[8.19] Change cases tests ownership (#241134) (#242308)
9e3c4f2 
# Backport

This will backport the following commits from `main` to `8.19`:
- [Change cases tests ownership
(#241134)](#241134)

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

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

<!--BACKPORT [{"author":{"name":"Dima
Arnautov","email":"dmitrii.arnautov@elastic.co"},"sourceCommit":{"committedDate":"2025-11-06T14:34:17Z","message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","test_ui_functional","release_note:skip","backport
missing","Team:ML","backport:all-open","Team:Cases","v9.3.0"],"title":"Change
cases tests ownership
","number":241134,"url":"https://github.com/elastic/kibana/pull/241134","mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/241134","number":241134,"mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}}]}]
BACKPORT-->
darnautov added a commit that referenced this pull request Nov 7, 2025
@darnautov
[9.2] Change cases tests ownership (#241134) (#242294)
8c0a9b4 
# Backport

This will backport the following commits from `main` to `9.2`:
- [Change cases tests ownership
(#241134)](#241134)

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

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

<!--BACKPORT [{"author":{"name":"Dima
Arnautov","email":"dmitrii.arnautov@elastic.co"},"sourceCommit":{"committedDate":"2025-11-06T14:34:17Z","message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","test_ui_functional","release_note:skip","backport
missing","Team:ML","backport:all-open","Team:Cases","v9.3.0"],"title":"Change
cases tests ownership
","number":241134,"url":"https://github.com/elastic/kibana/pull/241134","mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/241134","number":241134,"mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}}]}]
BACKPORT-->
darnautov added a commit that referenced this pull request Nov 7, 2025
@darnautov
[9.1] Change cases tests ownership (#241134) (#242297)
acba0e1 
# Backport

This will backport the following commits from `main` to `9.1`:
- [Change cases tests ownership
(#241134)](#241134)

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

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

<!--BACKPORT [{"author":{"name":"Dima
Arnautov","email":"dmitrii.arnautov@elastic.co"},"sourceCommit":{"committedDate":"2025-11-06T14:34:17Z","message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","test_ui_functional","release_note:skip","backport
missing","Team:ML","backport:all-open","Team:Cases","v9.3.0"],"title":"Change
cases tests ownership
","number":241134,"url":"https://github.com/elastic/kibana/pull/241134","mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/241134","number":241134,"mergeCommit":{"message":"Change
cases tests ownership (#241134)\n\n## Summary\n\n- Set cases tests
ownership\n(`/x-pack/platform/test/functional/apps/cases_attachments/**`)
to\n@elastic/kibana-cases\n- Move ML and AIOps cases tests under the
`cases_attachments` folder\n\n\nCloses
https://github.com/elastic/kibana/issues/202342\nCloses
https://github.com/elastic/kibana/issues/196307\nCloses
https://github.com/elastic/kibana/issues/200836","sha":"48314006fbc030864d89cd70d8610fda90e97888"}}]}]
BACKPORT-->
@kibanamachine kibanamachine added v9.1.7 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Nov 7, 2025
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
@darnautov @eokoneyo
Change cases tests ownership (elastic#241134)
591d2ac 
## Summary

- Set cases tests ownership
(`/x-pack/platform/test/functional/apps/cases_attachments/**`) to
@elastic/kibana-cases
- Move ML and AIOps cases tests under the `cases_attachments` folder


Closes elastic#202342
Closes elastic#196307
Closes elastic#200836
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@peteharverson peteharverson peteharverson approved these changes

Assignees

No one assigned

Labels

backport:all-open Backport to all branches that could still receive a release :ml release_note:skip Skip the PR/issue when compiling release notes Team:Cases Security Solution Cases team Team:ML Team label for ML (also use :ml) t// test_ui_functional v8.19.7 v9.1.8 v9.2.2 v9.3.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Failing test: Chrome X-Pack UI Functional Tests - aiops.x-pack/test/functional/apps/aiops/change_point_detection_cases·ts - aiops change point detection in cases attaches change point charts to a case Failing test: Chrome X-Pack UI Functional Tests - aiops.x-pack/test/functional/apps/aiops/log_pattern_analysis·ts - aiops log pattern analysis attaches log pattern analysis table to a case Failing test: Chrome X-Pack UI Functional Tests - ML anomaly_detection_result_views.x-pack/test/functional/apps/ml/anomaly_detection_result_views/anomaly_explorer·ts - machine learning - anomaly detection anomaly explorer with influencers with farequote based multi metric job Anomaly Charts as embeddable attaches an embeddable to a case

5 participants

@darnautov @elasticmachine @kibanamachine @peteharverson @mistic