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

Remove the graphite checks #233

Merged
merged 1 commit into from
May 17, 2024
Merged

Remove the graphite checks #233

merged 1 commit into from
May 17, 2024

Commits on May 13, 2024

  1. Remove the graphite checks

    This is a breaking change. We're removing the various health check types
    that rely on reading Graphite metrics because, as of Q3 2024, we will no
    longer be able to read this data.
    
    For now we recommend removing these checks from your apps. The alerts
    aren't very commonly-used and we have better ways of monitoring uptime
    now. If you still need alerts based on Graphite metrics then you can do
    so by creating a dashboard in the grafana monorepo and adding alerts:
    
    https://github.com/Financial-Times/grafana/tree/main/terraform/dashboards
    
    See-Also: https://financialtimes.atlassian.net/browse/CPREL-1056
    rowanmanning committed May 13, 2024
    Configuration menu
    Copy the full SHA
    88c6fae View commit details
    Browse the repository at this point in the history