This PR adds annotations to the event even if the threshold check is normal.
In addition, the format is changed so that the min-max range can be output in a single annotation.
See #4987
Why is this change necessary?
Adding the threshold annotation when normal will allow to ensure that the thresholds are normal when notifying some communication platform of the Event.
This change also makes it easier to ensure that the threshold checks are working correctly.
Does your change need a Changelog entry?
Yes, required as Added and Changed.
Do you need clarification on anything?
This PR is for Sensu Go 7.0 (main branch).
Were there any complications while making this change?
Have you reviewed and updated the documentation for this change? Is new documentation required?
What is this change?
This PR adds annotations to the event even if the threshold check is normal. In addition, the format is changed so that the min-max range can be output in a single annotation.
See #4987
Why is this change necessary?
Adding the threshold annotation when normal will allow to ensure that the thresholds are normal when notifying some communication platform of the Event. This change also makes it easier to ensure that the threshold checks are working correctly.
Does your change need a Changelog entry?
Yes, required as Added and Changed.
Do you need clarification on anything?
This PR is for Sensu Go 7.0 (main branch).
Were there any complications while making this change?
Have you reviewed and updated the documentation for this change? Is new documentation required?
This section Add event annotations based on metric threshold evaluation may need to be rewritten.
But there is no documentation for 7.0 yet.
How did you verify this change?
Setup a test environment using
output_metric_thresholdsand verified that the annotations have been added.Is this change a patch?
No.