Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

docs/monitoring: add sample alert rules #159

Merged
merged 1 commit into from
Jun 25, 2018

Conversation

alexbrand
Copy link
Contributor

Add sample alert rules to the alert manager documentation.

Open to suggestions on other (or alternative) alerts that might make sense to showcase.

To update alert rules, add new rules config values to the `prometheus-alert-rules` [configmap](../deployment/prometheus/02-prometheus-alertrules-configmap.yaml) in the `gimbal-monitoring` namespace.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does Prometheus need to be restarted to pick this change up?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm I believe it does. You can get prom to reload config sending a SIGHUP, but did not play around with that.

@stevesloka
Copy link
Member

@alexbrand could you rebase this?

@alexbrand alexbrand force-pushed the alertmanager-docs branch from 949178f to 56a3aea Compare June 25, 2018 13:27
Signed-off-by: Alexander Brand <alexbrand09@gmail.com>
@alexbrand alexbrand force-pushed the alertmanager-docs branch from 56a3aea to 5c2f222 Compare June 25, 2018 13:46
@stevesloka stevesloka merged commit 535e717 into projectcontour:master Jun 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants