AspNetCore.Diagnostics.HealthChecks icon indicating copy to clipboard operation
AspNetCore.Diagnostics.HealthChecks copied to clipboard

#2373 Add option to use trackAvailability() instead of trackEvents() in applicationInsightsPublisher

Open R3d-Dragon opened this issue 10 months ago • 4 comments

What this PR does / why we need it: Provide an option to publish healthCheck metrics through the TelemetryClient.TrackAvailability() method.

With this approach, healthCheck metrics are stored within the "AvailabilityResults" table in application insights. In addition, a CUSTOM web-test will automatically be created in Azure Application Insights which allows to monitor your application easily.

For more details, please have a look at the official documentation: https://learn.microsoft.com/en-us/azure/azure-monitor/app/availability?tabs=track#create-an-availability-test

Which issue(s) this PR fixes:

#2373

Special notes for your reviewer:

Does this PR introduce a user-facing change?: The change does not include breaking-changes. The configuration is designed as optional opt-in.

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • [x] Code compiles correctly
  • [x] Created/updated tests
  • [x] Unit tests passing
  • [x] End-to-end tests passing
  • [x] Extended the documentation
  • [ ] Provided sample for the feature

R3d-Dragon avatar Feb 10 '25 10:02 R3d-Dragon

@dotnet-policy-service agree company="Carl Zeiss Industrielle Messtechnik GmbH"

R3d-Dragon avatar Feb 10 '25 10:02 R3d-Dragon

+1 on this getting approved

jsclifford avatar Feb 12 '25 18:02 jsclifford

Hello @adamsitnik , i don't know who to contact regarding the PR. Is there any chance that my contribution will be reviewed / merged in near future? :)

R3d-Dragon avatar Mar 24 '25 08:03 R3d-Dragon

Hello @Alirexaa , i don't know who to contact regarding the PR. Is there any chance that my contribution will be reviewed / merged in near future? :)

R3d-Dragon avatar Aug 14 '25 13:08 R3d-Dragon