Skip to content

[8.x] [Inventory] Remove inventory dependency from observability plugin (#193251)#193344

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-193251
Sep 18, 2024
Merged

[8.x] [Inventory] Remove inventory dependency from observability plugin (#193251)#193344
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-193251

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

@kibanamachine kibanamachine commented Sep 18, 2024

Backport

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

Questions ?

Please refer to the Backport tool documentation

@kpatticha
[Inventory] Remove inventory dependency from observability plugin (el…
e04f7a3 
…astic#193251)

## Summary

closes elastic#193200
- Remove inventory dependency from observability plugin
- Register inventory in different section in classic stateful sidenav

https://github.com/user-attachments/assets/6c9c28bc-7483-4deb-b95a-67585a92f89f
(cherry picked from commit be2d641)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 18, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) September 18, 2024 18:01
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. labels Sep 18, 2024
@kibanamachine kibanamachine mentioned this pull request Sep 18, 2024
Merged
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Sep 18, 2024

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

@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Sep 18, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

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
observability 698 696 -2

Async chunks

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

id before after diff
observability 463.8KB 463.6KB -119.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
inventory 14.5KB 14.9KB +325.0B
observability 102.9KB 102.6KB -237.0B
total +88.0B
Unknown metric groups

API count

id before after diff
observability 705 703 -2

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @kpatticha

@kibanamachine kibanamachine merged commit 7823f1c into elastic:8.x Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees

@kpatticha kpatticha

Labels

backport This PR is a backport of another PR ci:project-deploy-observability Create an Observability project Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kibanamachine @elasticmachine @kibana-ci @kpatticha