Skip to content

[8.x] [Observability] Split up observability-utils package (#199801)#200886

Merged
dgieselaar merged 1 commit intoelastic:8.xfrom
dgieselaar:backport/8.x/pr-199801
Nov 20, 2024
Merged

[8.x] [Observability] Split up observability-utils package (#199801)#200886
dgieselaar merged 1 commit intoelastic:8.xfrom
dgieselaar:backport/8.x/pr-199801

Conversation

@dgieselaar
Copy link
Copy Markdown
Contributor

Backport

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

Questions ?

Please refer to the Backport tool documentation

Split up observability-utils package in browser, common, server. Also
made a small change to `withSpan` to automatically log operation times
when the debug level for the logger is enabled.

---------

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

# Conflicts:
#	.github/CODEOWNERS
@dgieselaar dgieselaar added the backport This PR is a backport of another PR label Nov 20, 2024
@dgieselaar dgieselaar enabled auto-merge (squash) November 20, 2024 10:30
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. labels Nov 20, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@dgieselaar dgieselaar merged commit b7d06a2 into elastic:8.x Nov 20, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
inference 22 23 +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/apm-utils 11 12 +1
@kbn/inference-common 38 41 +3
total +4

Async chunks

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

id before after diff
inventory 236.0KB 236.1KB +82.0B
investigateApp 483.7KB 483.8KB +82.0B
total +164.0B
Unknown metric groups

API count

id before after diff
@kbn/apm-utils 11 12 +1
@kbn/inference-common 121 124 +3
total +4

ESLint disabled line counts

id before after diff
@kbn/observability-utils 2 - -2
@kbn/observability-utils-browser - 3 +3
@kbn/observability-utils-common - 1 +1
total +2

Total ESLint disabled count

id before after diff
@kbn/observability-utils 2 - -2
@kbn/observability-utils-browser - 3 +3
@kbn/observability-utils-common - 1 +1
total +2

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 ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants