Skip to content
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.

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

Sustainable Kibana Architecture: Move content management related packages #205593

Merged
merged 22 commits into from
Jan 7, 2025

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Jan 6, 2025

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.

15 packages(s) are going to be relocated:

Id Target folder
@kbn/content-management-content-editor src/platform/packages/shared/content-management/content_editor
@kbn/content-management-content-insights-public src/platform/packages/shared/content-management/content_insights/content_insights_public
@kbn/content-management-content-insights-server src/platform/packages/shared/content-management/content_insights/content_insights_server
@kbn/content-management-favorites-common src/platform/packages/shared/content-management/favorites/favorites_common
@kbn/content-management-favorites-public src/platform/packages/shared/content-management/favorites/favorites_public
@kbn/content-management-favorites-server src/platform/packages/shared/content-management/favorites/favorites_server
@kbn/content-management-tabbed-table-list-view src/platform/packages/shared/content-management/tabbed_table_list_view
@kbn/content-management-table-list-view src/platform/packages/shared/content-management/table_list_view
@kbn/content-management-table-list-view-common src/platform/packages/shared/content-management/table_list_view_common
@kbn/content-management-table-list-view-table src/platform/packages/shared/content-management/table_list_view_table
@kbn/content-management-user-profiles src/platform/packages/shared/content-management/user_profiles
@kbn/object-versioning src/platform/packages/shared/kbn-object-versioning
@kbn/object-versioning-utils src/platform/packages/shared/kbn-object-versioning-utils
@kbn/recently-accessed src/platform/packages/shared/kbn-recently-accessed
@kbn/ui-actions-browser src/platform/packages/shared/kbn-ui-actions-browser
Updated relative paths
src/platform/packages/shared/content-management/content_editor/jest.config.js:12
src/platform/packages/shared/content-management/content_editor/tsconfig.json:2
src/platform/packages/shared/content-management/content_insights/content_insights_public/jest.config.js:12
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.json:2
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:27
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:30
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:33
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:36
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:39
src/platform/packages/shared/content-management/content_insights/content_insights_public/tsconfig.type_check.json:45
src/platform/packages/shared/content-management/content_insights/content_insights_server/jest.config.js:12
src/platform/packages/shared/content-management/content_insights/content_insights_server/tsconfig.json:2
src/platform/packages/shared/content-management/content_insights/content_insights_server/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/content_insights/content_insights_server/tsconfig.type_check.json:20
src/platform/packages/shared/content-management/content_insights/content_insights_server/tsconfig.type_check.json:23
src/platform/packages/shared/content-management/content_insights/content_insights_server/tsconfig.type_check.json:26
src/platform/packages/shared/content-management/favorites/favorites_common/jest.config.js:12
src/platform/packages/shared/content-management/favorites/favorites_common/tsconfig.json:2
src/platform/packages/shared/content-management/favorites/favorites_public/jest.config.js:12
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.json:2
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.type_check.json:24
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.type_check.json:27
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.type_check.json:33
src/platform/packages/shared/content-management/favorites/favorites_public/tsconfig.type_check.json:36
src/platform/packages/shared/content-management/favorites/favorites_server/jest.config.js:12
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.json:2
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:20
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:23
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:26
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:29
src/platform/packages/shared/content-management/favorites/favorites_server/tsconfig.type_check.json:32
src/platform/packages/shared/content-management/tabbed_table_list_view/jest.config.js:12
src/platform/packages/shared/content-management/tabbed_table_list_view/tsconfig.json:2
src/platform/packages/shared/content-management/tabbed_table_list_view/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/tabbed_table_list_view/tsconfig.type_check.json:28
src/platform/packages/shared/content-management/table_list_view/jest.config.js:12
src/platform/packages/shared/content-management/table_list_view/tsconfig.json:2
src/platform/packages/shared/content-management/table_list_view/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/table_list_view/tsconfig.type_check.json:25
src/platform/packages/shared/content-management/table_list_view_common/tsconfig.json:2
src/platform/packages/shared/content-management/table_list_view_common/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/table_list_view_table/jest.config.js:12
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.json:2
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:27
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:30
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:36
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:39
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:42
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:45
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:48
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:51
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:54
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:57
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:60
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:66
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:69
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:72
src/platform/packages/shared/content-management/table_list_view_table/tsconfig.type_check.json:78
src/platform/packages/shared/content-management/user_profiles/jest.config.js:12
src/platform/packages/shared/content-management/user_profiles/tsconfig.json:2
src/platform/packages/shared/content-management/user_profiles/tsconfig.type_check.json:2
src/platform/packages/shared/content-management/user_profiles/tsconfig.type_check.json:22
src/platform/packages/shared/content-management/user_profiles/tsconfig.type_check.json:25
src/platform/packages/shared/content-management/user_profiles/tsconfig.type_check.json:28
src/platform/packages/shared/content-management/user_profiles/tsconfig.type_check.json:31
src/platform/packages/shared/kbn-object-versioning-utils/jest.config.js:12
src/platform/packages/shared/kbn-object-versioning-utils/tsconfig.json:2
src/platform/packages/shared/kbn-object-versioning-utils/tsconfig.type_check.json:2
src/platform/packages/shared/kbn-object-versioning/jest.config.js:12
src/platform/packages/shared/kbn-object-versioning/tsconfig.json:2
src/platform/packages/shared/kbn-object-versioning/tsconfig.type_check.json:2
src/platform/packages/shared/kbn-recently-accessed/jest.config.js:12
src/platform/packages/shared/kbn-recently-accessed/tsconfig.json:2
src/platform/packages/shared/kbn-recently-accessed/tsconfig.type_check.json:2
src/platform/packages/shared/kbn-ui-actions-browser/jest.config.js:12
src/platform/packages/shared/kbn-ui-actions-browser/tsconfig.json:2
src/platform/packages/shared/kbn-ui-actions-browser/tsconfig.type_check.json:2
src/platform/packages/shared/kbn-ui-actions-browser/tsconfig.type_check.json:25

@Dosant Dosant changed the title <title> Sustainable Kibana Architecture: Move content management related packages Jan 6, 2025
@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) plugin groups Relocate all plugins and packages under their respective platform/solution groups labels Jan 6, 2025
@Dosant Dosant marked this pull request as ready for review January 6, 2025 10:56
@Dosant Dosant requested a review from a team as a code owner January 6, 2025 10:56
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@elastic-vault-github-plugin-prod elastic-vault-github-plugin-prod bot requested a review from a team as a code owner January 6, 2025 11:02
@@ -9927,6 +9927,38 @@
}
}
},
"favorites": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@elastic/kibana-core If this is added here, then should this be also deleted from somewhere? 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

There is a bug in the telemetry_check script that didn't let it update completely removed properties.

I opened #205613 to resolve this. Please wait until it is merged and then merge upstream into this PR. Otherwise, there is a chance of getting a broken main.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Awesome, thanks! will wait for that PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dokmic, looks like nothing has changed after the merge. Should I remove it manually?

Copy link
Contributor

@eokoneyo eokoneyo left a comment

Choose a reason for hiding this comment

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

Code changes LGTM, searched through the codebase didn't find any reference to the old directory paths for migrated packages

@dokmic
Copy link
Contributor

dokmic commented Jan 6, 2025

@elasticmachine merge upstream

@dokmic
Copy link
Contributor

dokmic commented Jan 7, 2025

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

@Dosant Dosant merged commit 72d3f7b into elastic:main Jan 7, 2025
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- [Streams] Dashboard linking (#204309)
- [Stack Connectors][Microsoft Defender] Adds new connector for Microsoft Defender for Endpoint (#203183)
- [Synonyms UI] Synonyms UI base plugin (#203284)
- [ResponseOps][Rules] Move synthetic rule types' params to @kbn/response-ops-rule-params (#204582)

Manual backport

To create the backport manually run:

node scripts/backport --pr 205593

Questions ?

Please refer to the Backport tool documentation

@Dosant
Copy link
Contributor Author

Dosant commented Jan 8, 2025

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

Dosant added a commit to Dosant/kibana that referenced this pull request Jan 8, 2025
…ages (elastic#205593)

(cherry picked from commit 72d3f7b)

# Conflicts:
#	.github/CODEOWNERS
Dosant added a commit that referenced this pull request Jan 8, 2025
…d packages (#205593) (#205855)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Sustainable Kibana Architecture: Move content management related
packages (#205593)](#205593)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Anton
Dosov","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-07T21:41:31Z","message":"Sustainable
Kibana Architecture: Move content management related packages
(#205593)","sha":"72d3f7b89c46231e70e8c992df133f653d9154a1","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","plugin
groups"],"number":205593,"url":"https://github.com/elastic/kibana/pull/205593","mergeCommit":{"message":"Sustainable
Kibana Architecture: Move content management related packages
(#205593)","sha":"72d3f7b89c46231e70e8c992df133f653d9154a1"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205593","number":205593,"mergeCommit":{"message":"Sustainable
Kibana Architecture: Move content management related packages
(#205593)","sha":"72d3f7b89c46231e70e8c992df133f653d9154a1"}}]}]
BACKPORT-->
crespocarlos pushed a commit to crespocarlos/kibana that referenced this pull request Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) plugin groups Relocate all plugins and packages under their respective platform/solution groups release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants