Sustainable Kibana Architecture: Move modules owned by @elastic/kibana-management

[gsoldevila]

Summary

This PR aims at relocating some of the Kibana modules (plugins and packages) into a new folder structure, according to the Sustainable Kibana Architecture initiative.

Important

• We kindly ask you to:
  • Manually fix the errors in the error section below (if there are any).
  • Search for the packages[\/\\] and plugins[\/\\] patterns in the source code (Babel and Eslint config files), and update them appropriately.
  • Manually review .buildkite/scripts/pipelines/pull_request/pipeline.ts to ensure that any CI pipeline customizations continue to be correctly applied after the changed path names
  • Review all of the updated files, specially the .ts and .js files listed in the sections below, as some of them contain relative paths that have been updated.
  • Think of potential impact of the move, including tooling and configuration files that can be pointing to the relocated modules. E.g.:
    • customised eslint rules
    • docs pointing to source code

Note

• This PR has been auto-generated.
• Any manual contributions will be lost if the 'relocate' script is re-run.
• Try to obtain the missing reviews / approvals before applying manual fixes, and/or keep your changes in a .patch / git stash.
• Please use #sustainable_kibana_architecture Slack channel for feedback.

Are you trying to rebase this PR to solve merge conflicts? Please follow the steps describe here.

3 plugin(s) are going to be relocated:

Id	Target folder
@kbn/index-management-plugin	x-pack/platform/plugins/shared/index_management
@kbn/ingest-pipelines-plugin	x-pack/platform/plugins/shared/ingest_pipelines
@kbn/upgrade-assistant-plugin	x-pack/platform/plugins/private/upgrade_assistant

Updated references

./docs/developer/plugin-list.asciidoc
./package.json
./packages/kbn-repo-packages/package-map.json
./packages/kbn-ts-projects/config-paths.json
./src/dev/precommit_hook/casing_check_config.js
./tsconfig.base.json
./x-pack/.i18nrc.json
./x-pack/platform/plugins/private/transform/common/types/es_ingest_pipeline.ts
./x-pack/platform/plugins/private/upgrade_assistant/README.md
./x-pack/platform/plugins/private/upgrade_assistant/jest.config.js
./x-pack/platform/plugins/shared/index_management/README.md
./x-pack/platform/plugins/shared/index_management/jest.config.js
./x-pack/platform/plugins/shared/ingest_pipelines/jest.config.js
./yarn.lock
.github/CODEOWNERS

Updated relative paths

x-pack/platform/plugins/private/upgrade_assistant/jest.config.js:10
x-pack/platform/plugins/private/upgrade_assistant/tsconfig.json:2
x-pack/platform/plugins/private/upgrade_assistant/tsconfig.json:7
x-pack/platform/plugins/shared/index_management/jest.config.js:10
x-pack/platform/plugins/shared/index_management/tsconfig.json:12
x-pack/platform/plugins/shared/index_management/tsconfig.json:2
x-pack/platform/plugins/shared/ingest_pipelines/jest.config.js:10
x-pack/platform/plugins/shared/ingest_pipelines/tsconfig.json:11
x-pack/platform/plugins/shared/ingest_pipelines/tsconfig.json:2

[github-actions]

A documentation preview will be available soon.

• 🔨 Buildkite builds
• 📚 HTML diff
• 📙 Preview page

Request a new doc build by commenting

• Rebuild this PR: run docs-build
• Rebuild this PR and all Elastic docs: run docs-build rebuild

_{run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.}

_{If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.}

[elasticmachine]

Pinging @elastic/kibana-core (Team:Core)

[elasticmachine]

Pinging @elastic/kibana-management (Team:Kibana Management)

[ryankeairns]

SCSS files only

[mattkime]

@gsoldevila At first glance this PR appears to be too disruptive. There are already a lot of merge conflicts plus this hasn't accounted for any work that's in progress. My initial impression is that a change like this would be best managed by the team to minimize disruption.

[gsoldevila]

@gsoldevila At first glance this PR appears to be too disruptive. There are already a lot of merge conflicts plus this hasn't accounted for any work that's in progress. My initial impression is that a change like this would be best managed by the team to minimise disruption.

Well, I imagine there's never a good time to perform such a big restructuring, but here we are 😆.
On my side, I have a script that did all this automatically, I can easily run it again from latest main to pickup latest changes, but ideally I would like to coordinate with @elastic/kibana-management to run it at the right time.
I have fixed a couple of bugs in the script, so I'm confident that CI will be green for this one next time I run it.

When do you think we could do this? FWIW our goal is to have this merged before 9.0, and also backported to 8.x.

[mattkime]

@gsoldevila Lets split this up, we should be able to get this through really quick -

All 17 packages

All plugins EXCEPT @kbn/index-management-plugin and @kbn/ingest-pipelines-plugin which are currently being worked on.

Then we'll follow up on those two plugins before too long.

[peteharverson]

Edit to transform file LGTM

[elasticmachine]

💔 Build Failed

• Buildkite Build
• Commit: 6657ab6

Failed CI Steps

• Jest Tests #12
• FTR Configs #66
• FTR Configs #67
• Serverless AI Assistant - Security Solution Cypress Tests #1
• AI Assistant - Security Solution Cypress Tests #1

Test Failures

• [job] [logs] Serverless AI Assistant - Security Solution Cypress Tests / AI Assistant Conversations Changing conversations Correctly creates and titles new conversations, and allows title updates Correctly creates and titles new conversations, and allows title updates
• [job] [logs] AI Assistant - Security Solution Cypress Tests / AI Assistant Conversations Changing conversations Correctly creates and titles new conversations, and allows title updates Correctly creates and titles new conversations, and allows title updates
• [job] [logs] FTR Configs #66 / Entity Manager _count API is resilient to no valid sources
• [job] [logs] FTR Configs #67 / lens serverless - group 1 - subgroup 6 lens logsdb smoke testing functions support Scenarios with changing stream type LogsDB downgraded to regular data stream scenarios LogsDB stream with no additional stream/index and no host.name field should visualize an annotation layer from a logsDB stream
• [job] [logs] Jest Tests #12 / useMonitorStatusData handles resize events and requests based on new data

Metrics [docs]

Unknown metric groups

ESLint disabled in files

id	before	after	diff
indexManagement	3	2	-1

Total ESLint disabled count

id	before	after	diff
indexManagement	19	18	-1

History

• 💔 Build #258125 failed 1ae027c3599f02edbeb0d1f8d79a3733db4a8e0a
• 💔 Build #258113 failed 678c80f09d9b3d7edd754f2e8b36625949025176
• 💔 Build #257620 failed 3da2d76bc7cfbd5918b4f6fa9dfb0120d3ad1561
• 💔 Build #257153 failed 35c50aaf5aaf037e64e42db59759ada33eef3cff

[afharo]

Core changes LGTM

[gsoldevila]

This PR has been split up in multiple relocation PRs (e.g. this one), and all of the involved modules have already been relocated.