-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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 modules owned by @elastic/observability-ui
#202834
base: main
Are you sure you want to change the base?
Sustainable Kibana Architecture: Move modules owned by @elastic/observability-ui
#202834
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
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. |
Pinging @elastic/kibana-core (Team:Core) |
Pinging @elastic/unified-observability (Team:Unified observability) |
i find it weird that word observability is used 4 times in this dir structure |
💭 this might cause conflicts with open PRs that change |
e0fd1ac
to
21a0527
Compare
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
0ab8693
to
fe143a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Core changes LGTM
f7ef92d
to
f70c6b7
Compare
💔 Build Failed
Failed CI StepsHistory
|
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
packages[\/\\]
andplugins[\/\\]
patterns in the source code (Babel and Eslint config files), and update them appropriately..buildkite/scripts/pipelines/pull_request/pipeline.ts
to ensure that any CI pipeline customizations continue to be correctly applied after the changed path names.ts
and.js
files listed in the sections below, as some of them contain relative paths that have been updated.Note
Are you trying to rebase this PR to solve merge conflicts? Please follow the steps describe here.
1 plugin(s) are going to be relocated:
@kbn/observability-shared-plugin
x-pack/solutions/observability/plugins/observability_shared
3 packages(s) are going to be relocated:
@kbn/observability-utils-browser
x-pack/solutions/observability/packages/observability_utils/observability_utils_browser
@kbn/observability-utils-common
x-pack/solutions/observability/packages/observability_utils/observability_utils_common
@kbn/observability-utils-server
x-pack/solutions/observability/packages/observability_utils/observability_utils_server
Updated references
Updated relative paths