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

.NET docs to include metrics emitted by extensions #4522

Closed
xakep139 opened this issue Oct 6, 2023 · 7 comments · Fixed by dotnet/docs#37875
Closed

.NET docs to include metrics emitted by extensions #4522

xakep139 opened this issue Oct 6, 2023 · 7 comments · Fixed by dotnet/docs#37875

Comments

@xakep139
Copy link
Contributor

xakep139 commented Oct 6, 2023

See #4482 (comment) and #4482 (comment) for details
Tl;DR: we need to update dotnet/docs#37213 with metric/instrument names that our components emit.
Additionally, ResourceMonitoring metrics need to be marked as experimental.

@joperezr
Copy link
Member

joperezr commented Oct 9, 2023

Thanks for logging this @xakep139. I'm assuming this is one we need for November so I've marked it as such. Given you are working on making sure all of the telemetry is aligned, is this something you were planning on working on next?

@joperezr
Copy link
Member

This one we want to make sure we have for November.

@joperezr
Copy link
Member

Update:

  • Once the PR with final instruments is done, @xakep139 will look into this one.

@xakep139
Copy link
Contributor Author

@iliar-turdushev the full list of metrics with their attributes/tags is listed in #4482 (comment)

@davidfowl
Copy link
Member

cc @IEvangelist

@iliar-turdushev
Copy link
Contributor

Documentation describing extensions' metrics has been merged dotnet/docs#37875.

@xakep139
Copy link
Contributor Author

xakep139 commented Nov 8, 2023

Thanks @iliar-turdushev !

@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants