Skip to content

Split Nginx alert into seperate PrometheusRule and fix expressions #237

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

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

poornima-krishnasamy
Copy link
Contributor

@poornima-krishnasamy poornima-krishnasamy commented Jun 6, 2024

  • Seperate nginx ingress controller related alerts to seperate Prometheus rule
  • Fix ingress-controller queries to match the correct controller pod names
  • Add alerts for production-only ingress
  • add new alert to measure ingress controller success rate - Related to: Review ingress controller count cloud-platform#4886

@poornima-krishnasamy poornima-krishnasamy changed the title Splid Nginx alert into seperate PrometheusRule and fix expressions Split Nginx alert into seperate PrometheusRule and fix expressions Jun 6, 2024
@poornima-krishnasamy poornima-krishnasamy force-pushed the fix-nginx-alerts branch 2 times, most recently from 57a668a to b7a4e07 Compare June 6, 2024 16:37
mikebell
mikebell previously approved these changes Jun 7, 2024
@poornima-krishnasamy poornima-krishnasamy merged commit 8e4bb1f into main Jun 7, 2024
@poornima-krishnasamy poornima-krishnasamy deleted the fix-nginx-alerts branch June 7, 2024 10:20
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.

2 participants