Skip to content

[APM] Add telemetry for calculating global labels used in computing aggrega…#167305

Merged
achyutjhunjhunwala merged 14 commits intoelastic:mainfrom
achyutjhunjhunwala:telemetry-for-global-labels
Oct 2, 2023
Merged

[APM] Add telemetry for calculating global labels used in computing aggrega…#167305
achyutjhunjhunwala merged 14 commits intoelastic:mainfrom
achyutjhunjhunwala:telemetry-for-global-labels

Conversation

@achyutjhunjhunwala
Copy link
Copy Markdown
Contributor

@achyutjhunjhunwala achyutjhunjhunwala commented Sep 26, 2023

Summary

Closes #161983

Adds telemetry for calculating global labels used in computing aggregation key for internal APM Metrics

How to test

Navigate to internal/apm/debug-telemetry on any of the deployed environments

You will find the global_labels count field like this

image

@achyutjhunjhunwala achyutjhunjhunwala added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. release_note:skip Skip the PR/issue when compiling release notes labels Sep 26, 2023
@achyutjhunjhunwala achyutjhunjhunwala self-assigned this Sep 26, 2023
@achyutjhunjhunwala achyutjhunjhunwala requested review from a team as code owners September 26, 2023 15:29
@achyutjhunjhunwala achyutjhunjhunwala requested a review from a team September 26, 2023 15:29
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/apm-ui (Team:APM)

@ghost
Copy link
Copy Markdown

ghost commented Sep 26, 2023

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Copy link
Copy Markdown
Contributor

@yngrdyn yngrdyn left a comment

Choose a reason for hiding this comment

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

LGTM #️⃣


const globalLabelCount = Object.keys(response.fields).length;

// Skip the top level Labels field which is sometimes present in the response
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you know why this happens? Why we get it just sometimes and not always?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, thats the top level labels object. If you see the actual payload, you will see all others are type: keyword and only this one is type: object

@achyutjhunjhunwala achyutjhunjhunwala enabled auto-merge (squash) October 2, 2023 13:55
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @achyutjhunjhunwala

@achyutjhunjhunwala achyutjhunjhunwala merged commit 5a88ccf into elastic:main Oct 2, 2023
@kibanamachine kibanamachine added v8.11.0 backport:skip This PR does not require backporting labels Oct 2, 2023
@achyutjhunjhunwala achyutjhunjhunwala deleted the telemetry-for-global-labels branch October 2, 2023 14:31
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 release_note:skip Skip the PR/issue when compiling release notes Team:APM - DEPRECATED Use Team:obs-ux-infra_services. v8.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[APM] Add telemetry for the number of global labels available in the last day

6 participants