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

[8.0] [kbn/rule-data-utils] add submodules and require public use them (#117963) #118628

Merged
merged 1 commit into from
Nov 16, 2021

Conversation

kibanamachine
Copy link
Contributor

Backports the following commits to 8.0:

…stic#117963)

* [kbn/rule-data-utils] add submodules and require public use them

* fix lint errors

Co-authored-by: Kibana Machine <[email protected]>
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] Default CI Group #10 / Actions and Triggers app Connectors should test a connector and display a failure result

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1183 1181 -2
cases 341 337 -4
infra 931 927 -4
observability 345 343 -2
securitySolution 2751 2747 -4
uptime 641 638 -3
total -19

Async chunks

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

id before after diff
apm 2.7MB 2.7MB +8.7KB
cases 323.5KB 321.6KB -2.0KB
infra 921.6KB 921.6KB -25.0B
observability 381.4KB 390.1KB +8.7KB
securitySolution 4.5MB 4.5MB -4.0B
timelines 236.9KB 231.0KB -6.0KB
uptime 573.3KB 573.3KB -7.0B
total +9.4KB

Page load bundle

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

id before after diff
apm 29.3KB 29.4KB +111.0B
infra 90.3KB 88.3KB -2.0KB
securitySolution 254.0KB 252.0KB -2.0KB
uptime 24.9KB 23.1KB -1.8KB
total -5.6KB

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

cc @spalger

@kibanamachine kibanamachine merged commit 21f3957 into elastic:8.0 Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants