Skip to content

Conversation

@JamesNK
Copy link
Member

@JamesNK JamesNK commented Aug 18, 2023

cc @noahfalk


Internal previews

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

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.

👍

dotnet-counters monitor -n WebMetric --counters Microsoft.AspNetCore.Hosting[requests-per-second,total-requests]
```

For more information, see [dotnet-counters](/dotnet/core/diagnostics/dotnet-counters). To learn more about metrics in .NET, see [built-in metrics](/dotnet/core/diagnostics/available-counters).
Copy link
Member Author

Choose a reason for hiding this comment

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

I remove the link to the available counters page as it just shows event counters for now.

@JamesNK JamesNK mentioned this pull request Aug 18, 2023
host=localhost,method=GET,port=5045,protocol=HTTP/1.1,ro 0 12
```

Run `WebMetric>dotnet-counters list` to show all available metrics.
Copy link
Member Author

Choose a reason for hiding this comment

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

dotnet-counters list isn't very useful right now. Removed.

Comment on lines -93 to -97
The following command shows the `Microsoft-AspNetCore-Server-Kestrel` meter with the `requests-per-second` and `total-requests` counters.

```dotnetcli
dotnet-counters monitor -n WebMetric --counters Microsoft.AspNetCore.Hosting[requests-per-second,total-requests]
```
Copy link
Member Author

Choose a reason for hiding this comment

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

This kind of detail can be left out. People who want to know can look at the dotnet-counters detail page.

@Rick-Anderson Rick-Anderson merged commit 2d48b4b into main Aug 18, 2023
@Rick-Anderson Rick-Anderson deleted the JamesNK-patch-1 branch August 18, 2023 20:08
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.

4 participants