Skip to content

[8.x] [Index Management + ML] Remove unused code from index management and ml (#211266)#212591

Merged
Samiul-TheSoccerFan merged 2 commits intoelastic:8.xfrom
Samiul-TheSoccerFan:backport/8.x/pr-211266
Feb 27, 2025
Merged

[8.x] [Index Management + ML] Remove unused code from index management and ml (#211266)#212591
Samiul-TheSoccerFan merged 2 commits intoelastic:8.xfrom
Samiul-TheSoccerFan:backport/8.x/pr-211266

Conversation

@Samiul-TheSoccerFan
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…ml (elastic#211266)

## Summary

This PR includes
- Remove unused package from kibana
- Remove references from index_management and ml plugins
- Update translations file
- Delete skipped tests

Closes elastic#204507

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [X] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 5270cc6)

# Conflicts:
#	.github/CODEOWNERS
#	x-pack/platform/packages/private/ml/inference_integration_flyout/components/flyout_layout.tsx
#	x-pack/platform/plugins/private/translations/translations/zh-CN.json
#	x-pack/platform/plugins/shared/ml/public/application/services/ml_api_service/inference_models.ts
#	x-pack/test_serverless/functional/test_suites/search/inference_management.ts
@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 26, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
indexManagement 679 678 -1
ml 2405 2393 -12
total -13

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
indexManagement 698.2KB 698.0KB -154.0B
ml 5.2MB 5.2MB +80.0B
total -74.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
ml 77.7KB 77.5KB -138.0B

History

@Samiul-TheSoccerFan Samiul-TheSoccerFan merged commit bcfb4f4 into elastic:8.x Feb 27, 2025
8 checks passed
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Mar 4, 2025
…t and ml (elastic#211266) (elastic#212591)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Index Management + ML] Remove unused code from index management and
ml (elastic#211266)](elastic#211266)

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

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

<!--BACKPORT [{"author":{"name":"Samiul
Monir","email":"150824886+Samiul-TheSoccerFan@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-02-26T21:09:44Z","message":"[Index
Management + ML] Remove unused code from index management and ml
(elastic#211266)\n\n## Summary\n\nThis PR includes\n- Remove unused package
from kibana\n- Remove references from index_management and ml plugins\n-
Update translations file\n- Delete skipped tests\n\nCloses
https://github.com/elastic/kibana/issues/204507\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [X] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Flaky
Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\nused on any tests changed\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5270cc621f18f8104338db046ff4c275e7c68769","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Kibana
Management","release_note:skip","Team:ML","ci:project-deploy-elasticsearch","backport:version","v9.1.0","v8.19.0"],"title":"[Index
Management + ML] Remove unused code from index management and
ml","number":211266,"url":"https://github.com/elastic/kibana/pull/211266","mergeCommit":{"message":"[Index
Management + ML] Remove unused code from index management and ml
(elastic#211266)\n\n## Summary\n\nThis PR includes\n- Remove unused package
from kibana\n- Remove references from index_management and ml plugins\n-
Update translations file\n- Delete skipped tests\n\nCloses
https://github.com/elastic/kibana/issues/204507\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [X] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Flaky
Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\nused on any tests changed\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5270cc621f18f8104338db046ff4c275e7c68769"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/211266","number":211266,"mergeCommit":{"message":"[Index
Management + ML] Remove unused code from index management and ml
(elastic#211266)\n\n## Summary\n\nThis PR includes\n- Remove unused package
from kibana\n- Remove references from index_management and ml plugins\n-
Update translations file\n- Delete skipped tests\n\nCloses
https://github.com/elastic/kibana/issues/204507\n\n###
Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers
should verify this PR satisfies this list as well.\n\n- [X] Any text
added follows [EUI's
writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\nsentence case text and includes
[i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n-
[x] [Flaky
Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\nused on any tests changed\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5270cc621f18f8104338db046ff4c275e7c68769"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants