Skip to content

Comments

[9.1] [Discover][Oblt] Move getDocViewer into the main observability root profile (#225610)#225858

Merged
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-225610
Jun 30, 2025
Merged

[9.1] [Discover][Oblt] Move getDocViewer into the main observability root profile (#225610)#225858
kibanamachine merged 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-225610

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 30, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) June 30, 2025 14:40
… profile (elastic#225610)

close elastic#225408
## Summary

Follow on
elastic#224894 (comment)

Since the Attributes tab is now enabled by default, the separate
observability-root-profile-with-attributes-tab profile is no longer
needed. Moved getDocViewer into the main observability root profile to
simplify the code.

Functionality remains unchanged; this is a code cleanup.

(cherry picked from commit 55b1ef5)
@kibanamachine kibanamachine merged commit 4e60f15 into elastic:9.1 Jun 30, 2025
10 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #120 / @ess @serverless @skipInServerlessMKI Import prebuilt rules when the package is not installed imports new prebuilt rules

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
discover 1339 1337 -2

Async chunks

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

id before after diff
discover 1.1MB 1.1MB -136.0B

cc @kpatticha

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants