AspNetCore.Diagnostics.HealthChecks
AspNetCore.Diagnostics.HealthChecks copied to clipboard
[Feature] Adding Additional Azure Service Bus HealthCheck Checking for healthy Message- and Deadletter-Queue Count
What this PR does / why we need it: As mentioned in #821 this feature creates two new Health Checks for setting a threshold on how many messages that can be in an Azure Service Bus Queue or Dead-letter queue. Internally we use this feature to determine whether our services are running correctly of we see an increased amount of messages in our error that we need to be aware of. We thought others might benefit for this as well 🙌
Adding ServiceBus Queue Message Count Threshold
services
. AddHealthChecks()
. AddAzureServiceBusQueueMessageCountThreshold(connectionString: "", queueName: "", degradedThresholdCount: 5, unhealthyThresholdCount: 10);
Adding ServiceBus Deadletter Queue Message Count Threshold
services
. AddHealthChecks()
. AddAzureServiceBusDeadLetterQueueMessageCountThreshold(connectionString: "", queueName: "", degradedThresholdCount: 5, unhealthyThresholdCount: 10);
Which issue(s) this PR fixes: #821
Please reference the issue this PR will close: #821
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
No it doesn't not change any user-facing changes, it adds an additional user facing feature for people whom uses HealthChecks.AzureServiceBus
package
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
- [ ] Extended the documentation
- [ ] Provided sample for the feature
I would love to provide documentation for this, however I was unable to find the correct markdown 🙈. If you can point me in the right direction I would be happy to add some notes 🙋 ✍️
@mclausen Please rebase onto master
if you are going to continue working on this PR.
ping @mclausen
@mclausen Do you intend to continue working on that PR?
Superseded by #1861. Thanks @mclausen and @cieciurm.