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

[statsdreceiver] add obsreport metrics for accepted/refused count #31822

Conversation

hardproblems
Copy link
Contributor

@hardproblems hardproblems commented Mar 18, 2024

Description:
Add accepted/refused metrics to statsdreceiver

Link to tracking Issue:
#24278

Testing:

Documentation:

cc @jmacd @dmitryax

@hardproblems hardproblems requested a review from a team March 18, 2024 22:50
@github-actions github-actions bot added the receiver/statsd statsd related issues label Mar 18, 2024
@hardproblems hardproblems force-pushed the feature-statsdreceiver-metrics branch 2 times, most recently from 04e895f to f6e0622 Compare March 18, 2024 23:32
@sirianni
Copy link
Contributor

@TylerHelmuth FYI see also a related PR #31839

 gci write --skip-generated -s standard -s default .
@hardproblems hardproblems force-pushed the feature-statsdreceiver-metrics branch 2 times, most recently from 66a1938 to 6d2cd3a Compare March 20, 2024 19:05
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

@hardproblems have you run this locally and confirmed the new metrics appear as expected?

@hardproblems
Copy link
Contributor Author

hardproblems commented Mar 26, 2024

@hardproblems have you run this locally and confirmed the new metrics appear as expected?

@TylerHelmuth actually no metrics appeared. I'm going to try ctx.Background()

@TylerHelmuth
Copy link
Member

Make sure you send some data through, I don't believe 0 will get reported

Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Apr 10, 2024
Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Apr 24, 2024
@hardproblems hardproblems deleted the feature-statsdreceiver-metrics branch May 29, 2024 23:59
@hardproblems hardproblems restored the feature-statsdreceiver-metrics branch June 6, 2024 17:56
dmitryax pushed a commit that referenced this pull request Jun 11, 2024
Description:
Add received statsd / accepted/refused metrics to statsdreceiver

Resurrecting
#31822
since #31839 is closed.

Link to tracking Issue:

#24278

Testing:
This works in our internal testing environments. Took a profile and
didn't see metric recording taking much cpu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
receiver/statsd statsd related issues Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants