Skip to content

Commit

Permalink
Remove reference to per-instance labels in GossipMembersMismatch warn…
Browse files Browse the repository at this point in the history
…ing (#6146)

* Remove reference to per-instance labels in GossipMembersMismatch warning.

* Add changelog entry.
  • Loading branch information
charleskorn authored Sep 28, 2023
1 parent b7c052c commit 934201d
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 7 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@

### Mixin

* [BUGFIX] Alerts: fixed issue where `GossipMembersMismatch` warning message referred to per-instance labels that were not produced by the alert query. #6146

### Jsonnet

* [ENHANCEMENT] Double the amount of rule groups for each user tier. #5897
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -489,8 +489,8 @@ spec:
rules:
- alert: MimirGossipMembersMismatch
annotations:
message: Mimir instance {{ $labels.pod }} in {{ $labels.cluster }}/{{ $labels.namespace
}} sees incorrect number of gossip members.
message: One or more Mimir instances in {{ $labels.cluster }}/{{ $labels.namespace
}} see incorrect number of gossip members.
runbook_url: https://grafana.com/docs/mimir/latest/operators-guide/mimir-runbooks/#mimirgossipmembersmismatch
expr: |
avg by (cluster, namespace) (memberlist_client_cluster_members_count) != sum by (cluster, namespace) (up{job=~".+/(admin-api|alertmanager|compactor.*|distributor|ingester.*|querier.*|ruler|ruler-querier.*|store-gateway.*|cortex|mimir|mimir-write.*|mimir-read.*|mimir-backend.*)"})
Expand Down
4 changes: 2 additions & 2 deletions operations/mimir-mixin-compiled-baremetal/alerts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -467,8 +467,8 @@ groups:
rules:
- alert: MimirGossipMembersMismatch
annotations:
message: Mimir instance {{ $labels.instance }} in {{ $labels.cluster }}/{{ $labels.namespace
}} sees incorrect number of gossip members.
message: One or more Mimir instances in {{ $labels.cluster }}/{{ $labels.namespace
}} see incorrect number of gossip members.
runbook_url: https://grafana.com/docs/mimir/latest/operators-guide/mimir-runbooks/#mimirgossipmembersmismatch
expr: |
avg by (cluster, namespace) (memberlist_client_cluster_members_count) != sum by (cluster, namespace) (up{job=~".+/(admin-api|alertmanager|compactor.*|distributor|ingester.*|querier.*|ruler|ruler-querier.*|store-gateway.*|cortex|mimir|mimir-write.*|mimir-read.*|mimir-backend.*)"})
Expand Down
4 changes: 2 additions & 2 deletions operations/mimir-mixin-compiled/alerts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -477,8 +477,8 @@ groups:
rules:
- alert: MimirGossipMembersMismatch
annotations:
message: Mimir instance {{ $labels.pod }} in {{ $labels.cluster }}/{{ $labels.namespace
}} sees incorrect number of gossip members.
message: One or more Mimir instances in {{ $labels.cluster }}/{{ $labels.namespace
}} see incorrect number of gossip members.
runbook_url: https://grafana.com/docs/mimir/latest/operators-guide/mimir-runbooks/#mimirgossipmembersmismatch
expr: |
avg by (cluster, namespace) (memberlist_client_cluster_members_count) != sum by (cluster, namespace) (up{job=~".+/(admin-api|alertmanager|compactor.*|distributor|ingester.*|querier.*|ruler|ruler-querier.*|store-gateway.*|cortex|mimir|mimir-write.*|mimir-read.*|mimir-backend.*)"})
Expand Down
2 changes: 1 addition & 1 deletion operations/mimir-mixin/alerts/alerts.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -708,7 +708,7 @@ local utils = import 'mixin-utils/utils.libsonnet';
severity: 'warning',
},
annotations: {
message: '%(product)s instance %(alert_instance_variable)s in %(alert_aggregation_variables)s sees incorrect number of gossip members.' % $._config,
message: 'One or more %(product)s instances in %(alert_aggregation_variables)s see incorrect number of gossip members.' % $._config,
},
},
],
Expand Down

0 comments on commit 934201d

Please sign in to comment.