-
Notifications
You must be signed in to change notification settings - Fork 377
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
Fix Unified Alerting Service Configuration #868
Fix Unified Alerting Service Configuration #868
Conversation
@@ -702,6 +702,10 @@ func (i *GrafanaIni) cfgUnifiedAlerting(config map[string][]string) map[string][ | |||
items = appendStr(items, "evaluation_timeout", i.cfg.UnifiedAlerting.EvaluationTimeout) | |||
items = appendInt(items, "max_attempts", i.cfg.UnifiedAlerting.MaxAttempts) | |||
items = appendStr(items, "min_interval", i.cfg.UnifiedAlerting.MinInterval) | |||
|
|||
items = appendStr(items, "ha_advertise_address", "${POD_IP}:9094") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this always hardcoded?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well it would need the pod IP I believe, but if you're referring to the port I believe that can be made configurable if we want to add that field to the Grafana CRD.
@celestialorb I'm not familiar with Grafana Alerting, can you please quickly describe why an extra service is required? Thanks! |
Extra service is required for ha configuration. Without it grafana will send each alert from every running pod. Also this configuration requires |
Is there a documentation how to enable this feature? |
Description
Adds another managed Kubernetes service for unified alerting, and adds the appropriate configuration to Grafana to utilize it for such.
Relevant issues/tickets
#855
Type of change
Checklist
Verification steps
While I have verified that the manifests are updated appropriately, I haven't yet verified that the unified alerting functions as intended with these manifests/configuration. I'm throwing this PR up now in order not to forget about it later / gather feedback on it before proceeding further.