Skip to content

Conversation

@mshustov
Copy link
Contributor

@mshustov mshustov commented Mar 31, 2021

Summary

Move @kbn/std and @elastic/safer-lodash-set to shared deps to avoid copying them to every plugin bundle.
@elastic/kibana-operations I had to cleanp limits.yml file content to make node scripts/build_kibana_platform_plugins --update-limits to update it

@mshustov mshustov added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.13.0 labels Mar 31, 2021
@mshustov mshustov requested a review from a team as a code owner March 31, 2021 08:46
@mshustov mshustov mentioned this pull request Mar 31, 2021
10 tasks
Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mshustov mshustov enabled auto-merge (squash) March 31, 2021 15:50
@mshustov mshustov added the auto-backport Deprecated - use backport:version if exact versions are needed label Mar 31, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
beatsManagement 302 244 -58
canvas 1144 1086 -58
core 434 363 -71
dashboard 249 200 -49
data 637 559 -78
esUiShared 211 152 -59
fleet 463 450 -13
graph 171 158 -13
infra 1036 995 -41
kibanaLegacy 122 63 -59
monitoring 620 561 -59
savedObjectsManagement 138 79 -59
securitySolution 2208 2207 -1
upgradeAssistant 129 70 -59
urlDrilldown 33 12 -21
visTypeTimeseries 505 501 -4
visTypeVislib 263 205 -58
visualizations 143 84 -59
watcher 236 177 -59
total -878

Async chunks

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

id before after diff
beatsManagement 503.9KB 503.9KB -13.0B
canvas 1.3MB 1.3MB -15.7KB
core 227.0KB 226.8KB -206.0B
dashboard 147.8KB 134.5KB -13.3KB
data 202.8KB 176.1KB -26.7KB
fleet 716.3KB 707.7KB -8.6KB
graph 1.1MB 1.1MB -8.6KB
infra 1.8MB 1.7MB -11.7KB
kibanaLegacy 112.7KB 112.7KB -33.0B
monitoring 741.3KB 725.7KB -15.6KB
savedObjectsManagement 154.9KB 139.3KB -15.6KB
securitySolution 7.2MB 7.3MB +107.5KB
upgradeAssistant 133.5KB 117.9KB -15.6KB
visTypeTimeseries 1.6MB 1.6MB -2.0KB
visTypeVislib 584.3KB 568.9KB -15.4KB
visualizations 43.0KB 43.0KB -37.0B
watcher 850.7KB 835.6KB -15.2KB
total -56.8KB

Page load bundle

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

id before after diff
beatsManagement 119.8KB 104.3KB -15.5KB
canvas 516.7KB 516.8KB +74.0B
core 396.7KB 373.0KB -23.6KB
dashboard 309.0KB 309.1KB +74.0B
data 797.8KB 782.4KB -15.4KB
esUiShared 205.3KB 189.8KB -15.5KB
fleet 345.4KB 345.3KB -73.0B
graph 10.9KB 10.8KB -72.0B
infra 137.7KB 137.7KB +74.0B
kibanaLegacy 74.6KB 59.2KB -15.5KB
monitoring 45.4KB 45.4KB +74.0B
savedObjectsManagement 34.2KB 34.2KB +74.0B
securitySolution 215.6KB 215.4KB -237.0B
upgradeAssistant 29.8KB 29.9KB +74.0B
urlDrilldown 47.9KB 21.0KB -26.9KB
visTypeTimeseries 109.1KB 109.2KB +49.0B
visTypeVislib 33.4KB 33.5KB +74.0B
visualizations 106.8KB 91.4KB -15.4KB
watcher 22.6KB 22.7KB +72.0B
total -127.5KB
Unknown metric groups

@kbn/ui-shared-deps asset size

id before after diff
kbn-ui-shared-deps.@elastic.js 3.3MB 3.3MB +686.0B
kbn-ui-shared-deps.js 6.7MB 6.7MB +17.5KB
total +18.2KB

async chunk count

id before after diff
securitySolution 26 25 -1
visTypeTimeseries 12 11 -1
total -2

History

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

@mshustov mshustov merged commit 4c0d09a into elastic:master Mar 31, 2021
@mshustov mshustov deleted the move-kbn-to-ui-shared-deps branch March 31, 2021 17:52
@kibanamachine
Copy link
Contributor

💔 Backport failed

❌ 7.x: Commit could not be cherrypicked due to conflicts

To backport manually run:
node scripts/backport --pr 95894

mshustov added a commit to mshustov/kibana that referenced this pull request Mar 31, 2021
* move @kbn/std to kbn-ui-shared-deps

* add @elastic/safer-lodash-set to kbn-ui-shared-deps

* update limits

* Revert "update limits"

This reverts commit f396d47.

* decrase limits for Core team owned code
# Conflicts:
#	packages/kbn-optimizer/limits.yml
mshustov added a commit that referenced this pull request Mar 31, 2021
* move @kbn/std to kbn-ui-shared-deps

* add @elastic/safer-lodash-set to kbn-ui-shared-deps

* update limits

* Revert "update limits"

This reverts commit f396d47.

* decrase limits for Core team owned code
# Conflicts:
#	packages/kbn-optimizer/limits.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes v7.13.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants