Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Added specific metric for graphite unavailability check result #766

Conversation

dmitryanchikov
Copy link

PR Summary

This PR adds a new internal Moira metric moira.checker.remote.unavailable which is supposed to show the number of graphite API call failures.

@dmitryanchikov dmitryanchikov self-assigned this Jul 4, 2022
@dmitryanchikov dmitryanchikov requested a review from a team as a code owner July 4, 2022 14:14
@dmitryanchikov dmitryanchikov force-pushed the feature/graphite-unavailability-handling branch from 026e69a to 5f0678a Compare July 6, 2022 07:28
@dmitryanchikov dmitryanchikov force-pushed the feature/grahpite-unavailability-metric branch from a75c196 to e4aa229 Compare July 6, 2022 07:28
@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2022

Codecov Report

Merging #766 (d5fd4be) into feature/graphite-unavailability-handling (88feba3) will not change coverage.
The diff coverage is n/a.

❗ Current head d5fd4be differs from pull request most recent head faca043. Consider uploading reports for the commit faca043 to get more accurate results

@@                            Coverage Diff                            @@
##           feature/graphite-unavailability-handling     #766   +/-   ##
=========================================================================
  Coverage                                     71.11%   71.11%           
=========================================================================
  Files                                           177      177           
  Lines                                          9160     9160           
=========================================================================
  Hits                                           6514     6514           
  Misses                                         2268     2268           
  Partials                                        378      378           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88feba3...faca043. Read the comment docs.

@dmitryanchikov dmitryanchikov force-pushed the feature/graphite-unavailability-handling branch from 5f0678a to c5b979e Compare July 6, 2022 08:13
@dmitryanchikov dmitryanchikov force-pushed the feature/grahpite-unavailability-metric branch from e4aa229 to 842d30f Compare July 6, 2022 08:14
@dmitryanchikov dmitryanchikov force-pushed the feature/graphite-unavailability-handling branch 2 times, most recently from 1383c1b to 88feba3 Compare July 6, 2022 11:52
@dmitryanchikov dmitryanchikov force-pushed the feature/grahpite-unavailability-metric branch 2 times, most recently from faca043 to 2d00284 Compare July 7, 2022 17:57
@kissken kissken merged commit 8c0f2d3 into feature/graphite-unavailability-handling Aug 24, 2022
@kissken kissken deleted the feature/grahpite-unavailability-metric branch August 24, 2022 08:05
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.

3 participants