Skip to content

Add Prometheus metrics for forwarded profile bytes in GRPCForwarder#5987

Closed
metalmatze wants to merge 1 commit into
mainfrom
forwarder-bytes-metrics
Closed

Add Prometheus metrics for forwarded profile bytes in GRPCForwarder#5987
metalmatze wants to merge 1 commit into
mainfrom
forwarder-bytes-metrics

Conversation

@metalmatze

@metalmatze metalmatze commented Oct 7, 2025

Copy link
Copy Markdown
Member

Works nicely.
image

@metalmatze
metalmatze requested a review from a team as a code owner October 7, 2025 17:42
@alwaysmeticulous

alwaysmeticulous Bot commented Oct 7, 2025

Copy link
Copy Markdown

✅ Meticulous spotted 0 visual differences across 489 screens tested: view results.

Meticulous evaluated ~4 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit b3b015e. This comment will update as new commits are pushed.

@manojVivek manojVivek left a comment

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.

Lgtm!

@metalmatze

Copy link
Copy Markdown
Member Author

It turns out that we actually parse the profile with the scraper before forwarding. Closing this in favor of #5988

@metalmatze metalmatze closed this Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants