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

Metrics /2 #29646

Merged
merged 28 commits into from
Jul 10, 2023
Merged

Metrics /2 #29646

merged 28 commits into from
Jul 10, 2023

Conversation

Rick-Anderson
Copy link
Contributor

@Rick-Anderson Rick-Anderson commented Jun 27, 2023

Fixes #29292


Internal previews

📄 File 🔗 Preview link
aspnetcore/log-mon/metrics/metrics.md aspnetcore/log-mon/metrics/metrics

Copy link
Contributor Author

@Rick-Anderson Rick-Anderson left a comment

Choose a reason for hiding this comment

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

@JamesNK what can I do to see ASP.NET Core metrics in the two places I commented the MD file? This is early draft, but can you review what I've got and how I'm starting?

aspnetcore/log-mon/metrics.md Outdated Show resolved Hide resolved
aspnetcore/log-mon/metrics.md Outdated Show resolved Hide resolved
@Rick-Anderson

This comment was marked as resolved.

@Rick-Anderson

This comment was marked as resolved.

aspnetcore/log-mon/metrics.md Outdated Show resolved Hide resolved
aspnetcore/log-mon/metrics/samples/Program.cs Outdated Show resolved Hide resolved
@JamesNK

This comment was marked as resolved.

@Rick-Anderson

This comment was marked as resolved.

@JamesNK

This comment was marked as resolved.

@Rick-Anderson
Copy link
Contributor Author

Rick-Anderson commented Jun 29, 2023

@JamesNK I can see them now that I've updated to the daily build.

@JamesNK it's ready for review.

@Rick-Anderson Rick-Anderson marked this pull request as ready for review June 29, 2023 02:21
@Rick-Anderson
Copy link
Contributor Author

@JamesNK please suggest the correct terminology. Counters and metrics are used interchangeably.

aspnetcore/log-mon/metrics/metrics.md Outdated Show resolved Hide resolved
aspnetcore/log-mon/metrics/metrics.md Outdated Show resolved Hide resolved
aspnetcore/log-mon/metrics/metrics.md Show resolved Hide resolved
@JamesNK JamesNK requested review from noahfalk and tarekgh July 6, 2023 23:48
* **Instrumentation:** Code in .NET libraries takes measurements and associates these measurements with a metric name.
* **Collection:** A .NET app configures named metrics to be transmitted from the app for external storage and analysis. Some tools may perform configuration outside the app using configuration files or a UI tool.

Instrumented code can record numeric measurements, but the measurements need to be aggregated, transmitted, and stored to create useful metrics for monitoring. The process of aggregating, transmitting, and storing data is called collection. This tutorial shows several examples of collecting metrics:
Copy link
Member

Choose a reason for hiding this comment

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

Instrumented code can record numeric measurements

do we need to mention the dimensions/tags here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

do we need to mention the dimensions/tags here?

@tarekgh can you suggest text for dimensions/tags or provide a link?

Copy link
Member

Choose a reason for hiding this comment

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

You may add the sentence Measurements can also be associated with key-value pairs called tags that allow data to be categorized for analysis. and optionally add the link Multi-dimensional metrics.

@Rick-Anderson Rick-Anderson merged commit 4c9a0ec into main Jul 10, 2023
@Rick-Anderson Rick-Anderson deleted the metrics branch July 10, 2023 23:06
Copy link
Member

@noahfalk noahfalk left a comment

Choose a reason for hiding this comment

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

A few comments on some of the metrics content you all added a little while back. @JamesNK @Rick-Anderson

* **Instrumentation:** Code in .NET libraries takes measurements and associates these measurements with a metric name.
* **Collection:** A .NET app configures named metrics to be transmitted from the app for external storage and analysis. Some tools may perform configuration outside the app using configuration files or a UI tool.

Instrumented code can record numeric measurements, but the measurements need to be aggregated, transmitted, and stored to create useful metrics for monitoring. The process of aggregating, transmitting, and storing data is called collection. This tutorial shows several examples of collecting metrics:
Copy link
Member

Choose a reason for hiding this comment

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

Instrumented code can record numeric measurements, but the measurements need to be aggregated, transmitted, and stored to create useful metrics for monitoring. The process of aggregating, transmitting, and storing data is called collection.

These two sentences feel duplicative with the two bullets right above. I think you could either keep the bullet list or keep these two sentences but having both seems unnecessary.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll get this fixed in #30092

aspnetcore/log-mon/metrics/metrics.md Show resolved Hide resolved
aspnetcore/log-mon/metrics/metrics.md Show resolved Hide resolved
aspnetcore/log-mon/metrics/metrics.md Show resolved Hide resolved
@Rick-Anderson Rick-Anderson mentioned this pull request Aug 18, 2023
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.

ASP.NET Core metrics
4 participants