Skip to content

Add Data Protection Metrics #54451

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

Open
1 task done
amcasey opened this issue Mar 8, 2024 · 3 comments
Open
1 task done

Add Data Protection Metrics #54451

amcasey opened this issue Mar 8, 2024 · 3 comments
Labels
area-dataprotection Includes: DataProtection

Comments

@amcasey
Copy link
Member

amcasey commented Mar 8, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

No response

Describe the solution you'd like

We should have KestrelMetrics-style metrics for Data Protection. In particular, we should have simple (vs up-down) counters for the most important error conditions, especially key-not-found.

Additional context

No response

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically label Mar 8, 2024
@amcasey amcasey added the area-dataprotection Includes: DataProtection label Mar 8, 2024
@amcasey amcasey self-assigned this Mar 8, 2024
@amcasey amcasey removed the needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically label Mar 8, 2024
@gfoidl
Copy link
Member

gfoidl commented Mar 9, 2024

We should have KestrelMetrics-style metrics for Data Protection.

Did you mean OTEL-style metrics? (😉)

especially key-not-found.

For easier diagnostics should this be expanded to add an (failure) event to Activity.Current?

@amcasey
Copy link
Member Author

amcasey commented Mar 11, 2024

For easier diagnostics should this be expanded to add an (failure) event to Activity.Current?

@gfoidl Tell me more? I'm new to OTEL conventions.

@amcasey
Copy link
Member Author

amcasey commented Jul 18, 2024

Personally, I think this is the highest ROI telemetry we could add, but there's some concern about whether it should be dotnet-wide or, conversely, whether this offers any value over log querying, so it's on hold for now.

@amcasey amcasey added this to the .NET 10 Planning milestone Oct 4, 2024
@amcasey amcasey removed their assignment Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-dataprotection Includes: DataProtection
Projects
None yet
Development

No branches or pull requests

2 participants