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

[7.11] [APM] Updated header icons (#84760) #86310

Merged
merged 1 commit into from
Dec 18, 2020

Conversation

cauemarcondes
Copy link
Contributor

Backports the following commits to 7.11:

* creating service name header

* fixing icons

* removing unused api import

* fixing some stuff

* adding API tests

* refactoring some stuff

* fixing tests

* refactoring some stuff

* fixing i18n

* reverting

* renaming

* applying min width

* addressing PR comments and adding test

* sorting service version

* changing sort type to desc

* addressing pr comments

* changing to show total and not avg

* addressing pr comments

* addressing pr comments

* addressing pr comments

Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1729 1734 +5

Async chunks

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

id before after diff
apm 5.4MB 5.4MB +12.1KB
triggersActionsUi 1.5MB 1.5MB -26.9KB
total -14.8KB

Distributable file count

id before after diff
default 47557 48324 +767

Page load bundle

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

id before after diff
triggersActionsUi 162.0KB 162.1KB +102.0B
Unknown metric groups

async chunk count

id before after diff
triggersActionsUi 31 32 +1

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

@cauemarcondes cauemarcondes merged commit 1d97b8a into elastic:7.11 Dec 18, 2020
@cauemarcondes cauemarcondes deleted the backport/7.11/pr-84760 branch December 18, 2020 05:51
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.

2 participants