Skip to content

[Maps] capture metrics for layer_groups in telemetry#144068

Merged
nreese merged 6 commits intoelastic:mainfrom
nreese:layer_group_telemetry
Oct 31, 2022
Merged

[Maps] capture metrics for layer_groups in telemetry#144068
nreese merged 6 commits intoelastic:mainfrom
nreese:layer_group_telemetry

Conversation

@nreese
Copy link
Contributor

@nreese nreese commented Oct 26, 2022

Follow up to #142528.

Ensure telemetry is properly captured for Layer groups.

@nreese nreese marked this pull request as ready for review October 27, 2022 16:57
@nreese nreese requested review from a team as code owners October 27, 2022 16:57
@nreese nreese added Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.6.0 labels Oct 27, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

Copy link
Contributor

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm!

code review and tested the captured telemetry

@nreese
Copy link
Contributor Author

nreese commented Oct 31, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
maps 2.7MB 2.7MB +78.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 57 63 +6
osquery 103 108 +5
securitySolution 439 443 +4
total +17

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 65 71 +6
osquery 104 110 +6
securitySolution 516 520 +4
total +18

History

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

@nreese nreese merged commit 8eeba8b into elastic:main Oct 31, 2022
@sophiec20 sophiec20 added Feature:Maps and removed Team:Geo Former Team Label for Geo Team. Now use Team:Presentation labels Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Maps release_note:skip Skip the PR/issue when compiling release notes v8.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants