SKA: Relocate `@kbn/core` inside `src/core/packages/core` #205036

gsoldevila · 2024-12-20T10:52:34Z

Summary

We want to relocate all of packages/core/** modules inside src/core/packages, and we believe some tooling (e.g. build_api_docs) might not like the fact that we're creating a packages folder inside the @kbn/core package folder.

This PR is an attempt at moving it within packages, as the title describes. cc @afharo

github-actions · 2024-12-20T10:52:49Z

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 · 2024-12-20T10:52:51Z

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

Click to trigger kibana-pull-request for this PR!
Click to trigger kibana-deploy-project-from-pr for this PR!

afharo

LGTM! Let's see if this change has the effect we expect.

I noticed many (previously) broken links. I wouldn't update them here to reduce the PR codereview impact.

afharo · 2024-12-20T11:32:42Z

dev_docs/tutorials/saved_objects.mdx

@@ -14,7 +14,7 @@ The folder should contain a file per type, named after the snake_case name of th
 **src/plugins/my_plugin/server/saved_objects/dashboard_visualization.ts**

 ```ts
-import { SavedObjectsType } from 'src/core/server';
+import { SavedObjectsType } from 'src/core/packages/core/server';


All the imports here should actually be @kbn/core/server (or the individual saved-objects package)

afharo · 2024-12-20T11:34:37Z

legacy_rfcs/text/0015_bazel.md

@@ -76,7 +76,7 @@ A Bazel [macro](https://docs.bazel.build/versions/master/skylark/macros.html) wi

 A Bazel [macro](https://docs.bazel.build/versions/master/skylark/macros.html) will be created to centralize the usage of Webpack. The macro will, at minimum, accept a configuration file and supply a base `webpack.config.js` file. Currently, all plugins share the same Webpack configuration. Allowing a plugin to provide additional configuration will allow plugins the ability to add loaders without affecting the performance of others.

-While running Kibana from source in development, the proxy server will ensure that client-side code for plugins is compiled and available. This is currently handled by the [basePathProxy](https://github.com/elastic/kibana/blob/main/src/core/server/http/base_path_proxy_server.ts), where server restarts and optimizer builds are observed and cause the proxy to pause requests. With Bazel, we will utilize [iBazel](https://github.com/bazelbuild/bazel-watche) to watch for file changes and re-build the plugin targets when necessary. The watcher will emit [events](https://github.com/bazelbuild/bazel-watcher#remote-events) that we will use to block requests and provide feedback to the logs.
+While running Kibana from source in development, the proxy server will ensure that client-side code for plugins is compiled and available. This is currently handled by the [basePathProxy](https://github.com/elastic/kibana/blob/main/src/core/packages/core/server/http/base_path_proxy_server.ts), where server restarts and optimizer builds are observed and cause the proxy to pause requests. With Bazel, we will utilize [iBazel](https://github.com/bazelbuild/bazel-watche) to watch for file changes and re-build the plugin targets when necessary. The watcher will emit [events](https://github.com/bazelbuild/bazel-watcher#remote-events) that we will use to block requests and provide feedback to the logs.


this path is no-longer working (it didn't work before this PR anyways) 🫠

I'll exclude all changes in legacy_rfcs

afharo · 2024-12-20T11:35:06Z

packages/core/application/core-application-browser/src/app_mount.ts

@@ -67,7 +67,7 @@ export interface AppMountParameters<HistoryLocationState = unknown> {
   * import ReactDOM from 'react-dom';
   * import { Router, Route } from 'react-router-dom';
   *
-   * import { CoreStart, AppMountParameters } from 'src/core/public';
+   * import { CoreStart, AppMountParameters } from 'src/core/packages/core/public';


@kbn/core/public

afharo · 2024-12-20T11:35:44Z

packages/core/deprecations/core-deprecations-common/src/types.ts

@@ -20,12 +20,12 @@ export interface DeprecationDetailsMessage {
 export interface BaseDeprecationDetails {
  /**
   * The title of the deprecation.
-   * Check the README for writing deprecations in `src/core/server/deprecations/README.mdx`
+   * Check the README for writing deprecations in `src/core/packages/core/server/deprecations/README.mdx`


I doubt this link worked 🫠

afharo · 2024-12-20T11:39:28Z

test/functional/page_objects/home_page.ts

@@ -58,7 +58,7 @@ export class HomePageObject extends FtrService {
  }

  async isWelcomeInterstitialDisplayed() {
-    // This element inherits style defined {@link https://github.com/elastic/kibana/blob/v8.14.3/src/core/public/styles/core_app/_mixins.scss#L72|here}
+    // This element inherits style defined {@link https://github.com/elastic/kibana/blob/v8.14.3/src/core/packages/core/public/styles/core_app/_mixins.scss#L72|here}


we are not backporting to v8.14.3 😬

afharo · 2024-12-20T11:40:34Z

x-pack/plugins/monitoring/dev_docs/reference/data_collection_modes.md

+  click Collectors "https://github.com/elastic/kibana/tree/main/src/core/packages/core/server/metrics/collectors"
+  click OpsMetricsObservable "https://github.com/elastic/kibana/blob/92a8636f0ff63ab072527574e96e6616327b2ea4/src/core/packages/core/server/metrics/metrics_service.ts#L32"


Suggested change

click Collectors "https://github.com/elastic/kibana/tree/main/src/core/packages/core/server/metrics/collectors"

click OpsMetricsObservable "https://github.com/elastic/kibana/blob/92a8636f0ff63ab072527574e96e6616327b2ea4/src/core/packages/core/server/metrics/metrics_service.ts#L32"

click Collectors "https://github.com/elastic/kibana/tree/main/src/core/server/metrics/collectors"

click OpsMetricsObservable "https://github.com/elastic/kibana/blob/92a8636f0ff63ab072527574e96e6616327b2ea4/src/core/server/metrics/metrics_service.ts#L32"

I'm actually moving the server folder, so I think the 1st line is correct? I'll revert changes in the second.

My point is that src/core/metrics don't exist since we moved the service to a package. But we forgot to update this reference.

I'd prefer to update these already broken URLs in a separate PR to avoid making too much noise in this important PR.

afharo · 2024-12-20T11:41:54Z

x-pack/plugins/monitoring/server/deprecations.ts

@@ -85,6 +85,6 @@ export const deprecations = ({
    // vice versa.
    // ^ These deprecations should only be shown if they are explicitly configured for monitoring -- we should not show Monitoring
    // deprecations for these settings if they are inherited from the Core elasticsearch settings.
-    // See the Core implementation: src/core/server/elasticsearch/elasticsearch_config.ts
+    // See the Core implementation: src/core/packages/core/server/elasticsearch/elasticsearch_config.ts


Old path was still broken... let's minimize noise.

Suggested change

// See the Core implementation: src/core/packages/core/server/elasticsearch/elasticsearch_config.ts

// See the Core implementation: src/core/server/elasticsearch/elasticsearch_config.ts

afharo · 2024-12-20T11:42:14Z

x-pack/plugins/upgrade_assistant/README.md

@@ -29,7 +29,7 @@ These surface runtime deprecations, e.g. a Painless script that uses a deprecate
 request to a deprecated API. These are also generally surfaced as deprecation headers within the
 response. Even if the cluster state is good, app maintainers need to watch the logs in case
 deprecations are discovered as data is migrated. Starting in 7.x, deprecation logs can be written to a file or a data stream ([#58924](https://github.com/elastic/elasticsearch/pull/58924)). When the data stream exists, the Upgrade Assistant provides a way to analyze the logs through Observability or Discover ([#106521](https://github.com/elastic/kibana/pull/106521)).
-* [**Kibana deprecations API.**](https://github.com/elastic/kibana/blob/main/src/core/server/docs/kib_core_deprecations_service.mdx) This is information about deprecated features and configs in Kibana. These deprecations are only communicated to the user if the deployment is using these features. Kibana engineers are responsible for adding deprecations to the deprecations API for their respective team.
+* [**Kibana deprecations API.**](https://github.com/elastic/kibana/blob/main/src/core/packages/core/server/docs/kib_core_deprecations_service.mdx) This is information about deprecated features and configs in Kibana. These deprecations are only communicated to the user if the deployment is using these features. Kibana engineers are responsible for adding deprecations to the deprecations API for their respective team.


elasticmachine · 2024-12-20T15:15:49Z

💔 Build Failed

Buildkite Build
Commit: a11d70f

Failed CI Steps

Build Kibana Distribution

History

afharo approved these changes Dec 20, 2024

View reviewed changes

gsoldevila force-pushed the move-kbn-core-to-src-core-packages branch 2 times, most recently from 840c67e to 8541576 Compare December 20, 2024 13:37

Relocate @kbn/core inside src/core/packages/core

656cae1

gsoldevila force-pushed the move-kbn-core-to-src-core-packages branch from 8541576 to 656cae1 Compare December 20, 2024 13:56

gsoldevila added 3 commits December 20, 2024 15:29

Remove unexisting paths

0f395e8

Add test-helpers as dev code in ESLint exceptions

1ea7c2a

Update outdated scss files

a11d70f

gsoldevila force-pushed the move-kbn-core-to-src-core-packages branch from 2a25ad1 to a11d70f Compare December 20, 2024 14:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

SKA: Relocate `@kbn/core` inside `src/core/packages/core` #205036

SKA: Relocate `@kbn/core` inside `src/core/packages/core` #205036

gsoldevila commented Dec 20, 2024 •

edited

Loading

github-actions bot commented Dec 20, 2024

elasticmachine commented Dec 20, 2024 •

edited

Loading

afharo left a comment

afharo Dec 20, 2024

afharo Dec 20, 2024

gsoldevila Dec 20, 2024

afharo Dec 20, 2024

afharo Dec 20, 2024

afharo Dec 20, 2024

afharo Dec 20, 2024

gsoldevila Dec 20, 2024

afharo Dec 20, 2024

afharo Dec 20, 2024

afharo Dec 20, 2024

elasticmachine commented Dec 20, 2024 •

edited

Loading

		click Collectors "https://github.com/elastic/kibana/tree/main/src/core/packages/core/server/metrics/collectors"
		click OpsMetricsObservable "https://github.com/elastic/kibana/blob/92a8636f0ff63ab072527574e96e6616327b2ea4/src/core/packages/core/server/metrics/metrics_service.ts#L32"

	// See the Core implementation: src/core/packages/core/server/elasticsearch/elasticsearch_config.ts
	// See the Core implementation: src/core/server/elasticsearch/elasticsearch_config.ts

SKA: Relocate @kbn/core inside src/core/packages/core #205036

Are you sure you want to change the base?

SKA: Relocate @kbn/core inside src/core/packages/core #205036

Conversation

gsoldevila commented Dec 20, 2024 • edited Loading

Summary

github-actions bot commented Dec 20, 2024

elasticmachine commented Dec 20, 2024 • edited Loading

afharo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

elasticmachine commented Dec 20, 2024 • edited Loading

💔 Build Failed

Failed CI Steps

History

SKA: Relocate `@kbn/core` inside `src/core/packages/core` #205036

SKA: Relocate `@kbn/core` inside `src/core/packages/core` #205036

gsoldevila commented Dec 20, 2024 •

edited

Loading

elasticmachine commented Dec 20, 2024 •

edited

Loading

elasticmachine commented Dec 20, 2024 •

edited

Loading