search

awslabs / aws-c-mqtt

C99 implementation of the MQTT 3.1.1 specification.
Apache License 2.0
93 stars 29 forks source link

Adjust the canary not to crash if it cannot poll the metric alarms #279

Closed TwistedTwigleg closed 1 year ago

TwistedTwigleg commented 1 year ago

Description of changes:

Minor canary script fix: If the canary cannot, for whatever reason, pull the state of the Canary status alarms, instead of crashing it will just skip and try again later. This is to prevent the canary from going down when there is too much network congestion or the service is temporarily down for a little bit.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (9b9eb47) 81.28% compared to head (d9e8030) 81.28%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #279 +/- ## ======================================= Coverage 81.28% 81.28% ======================================= Files 16 16 Lines 7208 7208 ======================================= Hits 5859 5859 Misses 1349 1349 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

TwistedTwigleg commented 1 year ago

Thanks! Merging into main...