Skip to content
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

Add @kbn/shared-ux-router to shared bundle #204786

Merged
merged 3 commits into from
Dec 30, 2024

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Dec 18, 2024

Summary

Part of #204761 . This package is used by every app and I thought it is worth adding it to the shared bundle. Especially since it is already part of Bazel.

@Dosant Dosant changed the title add router to shared bundle Add @kbn/shared-ux-router to shared bundle Dec 19, 2024
@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Dec 19, 2024
@Dosant Dosant marked this pull request as ready for review December 19, 2024 15:10
@Dosant Dosant requested a review from a team as a code owner December 19, 2024 15:10
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@Dosant
Copy link
Contributor Author

Dosant commented Dec 30, 2024

@elasticmachine merge upstream

@Dosant
Copy link
Contributor Author

Dosant commented Dec 30, 2024

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
aiAssistantManagementSelection 76 65 -11
alerting 234 223 -11
apm 1918 1909 -9
canvas 1281 1270 -11
cases 847 838 -9
cloudDefend 105 96 -9
cloudSecurityPosture 650 642 -8
console 225 214 -11
core 415 404 -11
crossClusterReplication 121 110 -11
dashboard 687 676 -11
dataQuality 78 67 -11
dataUsage 131 120 -11
dataViewManagement 238 229 -9
devTools 31 20 -11
discover 935 927 -8
enterpriseSearch 2298 2289 -9
entityManagerApp 34 23 -11
exploratoryView 172 163 -9
filesManagement 189 178 -11
fleet 1241 1232 -9
graph 301 290 -11
home 257 246 -11
indexLifecycleManagement 252 241 -11
indexManagement 688 679 -9
infra 1288 1281 -7
ingestPipelines 344 333 -11
integrationAssistant 475 466 -9
inventory 247 238 -9
investigateApp 591 580 -11
kibanaOverview 127 116 -11
kubernetesSecurity 212 203 -9
lens 1544 1537 -7
licenseManagement 113 102 -11
logstash 63 52 -11
management 120 109 -11
maps 1257 1248 -9
ml 2131 2124 -7
monitoring 489 478 -11
observability 1099 1091 -8
observabilityAIAssistantApp 426 419 -7
observabilityAiAssistantManagement 395 388 -7
observabilityLogsExplorer 201 192 -9
observabilityOnboarding 245 236 -9
osquery 335 324 -11
profiling 302 291 -11
remoteClusters 113 102 -11
rollup 134 123 -11
savedObjectsManagement 112 101 -11
searchHomepage 142 131 -11
searchIndices 233 222 -11
searchInferenceEndpoints 252 243 -9
searchPlayground 259 248 -11
security 517 506 -11
securitySolution 6496 6488 -8
serverlessSearch 298 289 -9
slo 852 844 -8
snapshotRestore 199 188 -11
spaces 328 317 -11
streamsApp 189 178 -11
synthetics 1049 1040 -9
transform 476 469 -7
triggersActionsUi 858 849 -9
upgradeAssistant 159 148 -11
uptime 580 571 -9
ux 195 186 -9
visualizations 482 473 -9
watcher 177 166 -11
total -667

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/ui-shared-deps-src 55 56 +1

Async chunks

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

id before after diff
aiAssistantManagementSelection 93.2KB 93.1KB -85.0B
alerting 99.1KB 95.7KB -3.3KB
apm 5.6MB 5.6MB -1.4KB
canvas 1.0MB 1.0MB -3.3KB
cases 535.5KB 533.0KB -2.5KB
cloudDefend 101.5KB 99.4KB -2.1KB
cloudSecurityPosture 500.9KB 499.0KB -2.0KB
console 207.6KB 204.3KB -3.4KB
core 60.9KB 60.9KB -62.0B
crossClusterReplication 144.1KB 141.5KB -2.6KB
dashboard 669.5KB 666.3KB -3.3KB
dataQuality 26.1KB 22.8KB -3.3KB
dataUsage 240.7KB 237.4KB -3.4KB
dataViewManagement 138.3KB 136.2KB -2.1KB
devTools 5.8KB 2.4KB -3.4KB
discover 784.2KB 782.3KB -1.9KB
enterpriseSearch 2.6MB 2.6MB -2.0KB
entityManagerApp 9.6KB 9.5KB -87.0B
exploratoryView 181.3KB 179.2KB -2.1KB
filesManagement 123.6KB 121.0KB -2.6KB
fleet 1.7MB 1.7MB -6.5KB
graph 416.3KB 413.0KB -3.3KB
home 150.4KB 147.1KB -3.3KB
indexLifecycleManagement 163.9KB 160.6KB -3.3KB
indexManagement 702.8KB 700.8KB -2.1KB
infra 1.3MB 1.3MB -1.0KB
ingestPipelines 407.2KB 403.9KB -3.3KB
integrationAssistant 880.8KB 877.6KB -3.2KB
inventory 231.8KB 231.7KB -88.0B
investigateApp 481.5KB 478.2KB -3.4KB
kibanaOverview 63.5KB 60.2KB -3.3KB
kubernetesSecurity 260.0KB 257.9KB -2.1KB
lens 1.5MB 1.5MB -1.7KB
licenseManagement 44.6KB 41.2KB -3.3KB
logstash 31.3KB 28.0KB -3.4KB
management 46.7KB 43.4KB -3.3KB
maps 3.0MB 3.0MB -3.3KB
ml 4.7MB 4.7MB -1.9KB
monitoring 535.5KB 532.1KB -3.4KB
observability 480.9KB 478.7KB -2.2KB
observabilityAIAssistantApp 294.1KB 294.0KB -85.0B
observabilityAiAssistantManagement 97.7KB 97.6KB -89.0B
observabilityLogsExplorer 145.9KB 142.1KB -3.8KB
observabilityOnboarding 272.3KB 270.2KB -2.1KB
osquery 1.0MB 1.0MB -3.3KB
profiling 404.3KB 404.2KB -86.0B
remoteClusters 77.4KB 74.5KB -2.9KB
rollup 113.7KB 110.7KB -3.1KB
savedObjectsManagement 85.2KB 81.8KB -3.4KB
searchHomepage 53.1KB 49.8KB -3.4KB
searchIndices 167.4KB 164.1KB -3.3KB
searchInferenceEndpoints 216.7KB 214.5KB -2.2KB
searchPlayground 162.5KB 159.1KB -3.4KB
security 542.4KB 542.4KB -26.0B
securitySolution 21.4MB 21.4MB -4.6KB
serverlessSearch 362.7KB 352.1KB -10.6KB
slo 835.3KB 831.7KB -3.7KB
snapshotRestore 261.1KB 257.7KB -3.3KB
spaces 257.0KB 257.0KB -18.0B
streamsApp 182.3KB 182.2KB -88.0B
synthetics 1.1MB 1.1MB -2.3KB
transform 473.7KB 471.9KB -1.7KB
triggersActionsUi 1.7MB 1.7MB -12.0KB
upgradeAssistant 158.9KB 155.5KB -3.3KB
uptime 464.8KB 462.6KB -2.1KB
ux 165.5KB 165.4KB -91.0B
visualizations 322.9KB 320.7KB -2.1KB
watcher 164.1KB 161.4KB -2.8KB
total -182.2KB

Page load bundle

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

id before after diff
aiAssistantManagementSelection 4.7KB 4.7KB -5.0B
alerting 25.6KB 25.6KB -5.0B
apm 38.4KB 38.4KB -5.0B
canvas 14.4KB 14.4KB -5.0B
cases 161.7KB 161.7KB -5.0B
cloudDefend 9.0KB 9.0KB -5.0B
cloudSecurityPosture 17.9KB 17.9KB -5.0B
console 35.6KB 35.6KB -6.0B
core 447.2KB 443.9KB -3.3KB
crossClusterReplication 12.2KB 12.1KB -54.0B
dashboard 49.1KB 49.1KB -5.0B
dataQuality 8.3KB 8.2KB -54.0B
dataUsage 7.5KB 7.4KB -54.0B
dataViewManagement 5.2KB 5.1KB -5.0B
devTools 11.0KB 10.9KB -54.0B
discover 45.2KB 45.2KB -5.0B
enterpriseSearch 52.1KB 52.1KB +61.0B
entityManagerApp 4.8KB 4.7KB -63.0B
exploratoryView 43.9KB 43.9KB -5.0B
filesManagement 3.9KB 3.9KB -5.0B
fleet 171.2KB 171.2KB -13.0B
graph 8.0KB 8.0KB -5.0B
home 11.5KB 11.5KB -5.0B
indexLifecycleManagement 27.5KB 27.5KB -5.0B
indexManagement 44.3KB 44.3KB -5.0B
infra 51.0KB 51.0KB -5.0B
ingestPipelines 15.1KB 15.1KB -54.0B
integrationAssistant 10.4KB 10.4KB -55.0B
inventory 13.0KB 13.0KB -5.0B
investigateApp 11.1KB 11.0KB -5.0B
kbnUiSharedDeps-srcJs 3.5MB 3.5MB +4.2KB
kibanaOverview 5.3KB 5.3KB -5.0B
kubernetesSecurity 6.0KB 5.9KB -63.0B
lens 50.3KB 50.3KB -63.0B
licenseManagement 11.1KB 11.1KB -54.0B
logstash 14.1KB 14.1KB -5.0B
management 10.8KB 10.8KB -5.0B
maps 51.8KB 51.8KB -5.0B
ml 77.7KB 77.7KB -5.0B
monitoring 25.5KB 25.5KB -5.0B
observability 102.2KB 102.2KB -5.0B
observabilityAIAssistantApp 14.7KB 14.7KB -5.0B
observabilityAiAssistantManagement 5.1KB 5.1KB -5.0B
observabilityLogsExplorer 15.0KB 15.0KB -5.0B
observabilityOnboarding 10.3KB 10.4KB +59.0B
osquery 44.5KB 44.4KB -5.0B
profiling 17.6KB 17.6KB -5.0B
remoteClusters 9.0KB 9.0KB -54.0B
rollup 12.0KB 11.9KB -54.0B
savedObjectsManagement 19.8KB 19.8KB -5.0B
searchHomepage 4.7KB 4.6KB -113.0B
searchIndices 7.5KB 7.5KB -5.0B
searchInferenceEndpoints 6.8KB 6.8KB -5.0B
searchPlayground 6.9KB 6.9KB +61.0B
security 66.5KB 62.8KB -3.7KB
securitySolution 88.1KB 88.2KB +61.0B
serverlessSearch 22.8KB 22.8KB -5.0B
slo 29.6KB 29.6KB -5.0B
snapshotRestore 27.1KB 27.1KB -5.0B
spaces 32.6KB 29.3KB -3.3KB
streamsApp 5.4KB 5.4KB -5.0B
synthetics 37.6KB 37.6KB -5.0B
transform 18.3KB 18.3KB -5.0B
triggersActionsUi 126.9KB 126.9KB -13.0B
upgradeAssistant 22.6KB 22.6KB -5.0B
uptime 22.7KB 22.7KB -5.0B
ux 6.8KB 6.8KB -5.0B
visualizations 62.3KB 62.3KB -5.0B
watcher 13.5KB 13.5KB -54.0B
total -7.0KB
Unknown metric groups

API count

id before after diff
@kbn/ui-shared-deps-src 64 65 +1

async chunk count

id before after diff
fleet 10 9 -1
triggersActionsUi 65 64 -1
total -2

History

@Dosant Dosant merged commit b64ff10 into elastic:main Dec 30, 2024
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12546722082

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 204786

Questions ?

Please refer to the Backport tool documentation

Dosant added a commit to Dosant/kibana that referenced this pull request Dec 30, 2024
## Summary

Part of elastic#204761 . This package
is used by every app and I thought it is worth adding it to the shared
bundle. Especially since it is already part of Bazel.

(cherry picked from commit b64ff10)
@Dosant
Copy link
Contributor Author

Dosant commented Dec 30, 2024

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

Dosant added a commit that referenced this pull request Dec 31, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Add `@kbn/shared-ux-router` to shared bundle
(#204786)](#204786)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Anton
Dosov","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-30T12:44:44Z","message":"Add
`@kbn/shared-ux-router` to shared bundle (#204786)\n\n##
Summary\r\n\r\nPart of #204761 .
This package\r\nis used by every app and I thought it is worth adding it
to the shared\r\nbundle. Especially since it is already part of
Bazel.","sha":"b64ff102045585b2f28ced7a612edcafd0944b6e","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor"],"number":204786,"url":"https://github.com/elastic/kibana/pull/204786","mergeCommit":{"message":"Add
`@kbn/shared-ux-router` to shared bundle (#204786)\n\n##
Summary\r\n\r\nPart of #204761 .
This package\r\nis used by every app and I thought it is worth adding it
to the shared\r\nbundle. Especially since it is already part of
Bazel.","sha":"b64ff102045585b2f28ced7a612edcafd0944b6e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204786","number":204786,"mergeCommit":{"message":"Add
`@kbn/shared-ux-router` to shared bundle (#204786)\n\n##
Summary\r\n\r\nPart of #204761 .
This package\r\nis used by every app and I thought it is worth adding it
to the shared\r\nbundle. Especially since it is already part of
Bazel.","sha":"b64ff102045585b2f28ced7a612edcafd0944b6e"}}]}]
BACKPORT-->
stratoula pushed a commit to stratoula/kibana that referenced this pull request Jan 2, 2025
## Summary

Part of elastic#204761 . This package
is used by every app and I thought it is worth adding it to the shared
bundle. Especially since it is already part of Bazel.
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
## Summary

Part of elastic#204761 . This package
is used by every app and I thought it is worth adding it to the shared
bundle. Especially since it is already part of Bazel.
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Jan 2, 2025
## Summary

Part of elastic#204761 . This package
is used by every app and I thought it is worth adding it to the shared
bundle. Especially since it is already part of Bazel.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
## Summary

Part of elastic#204761 . This package
is used by every app and I thought it is worth adding it to the shared
bundle. Especially since it is already part of Bazel.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants