Skip to content

Commit a41a7bb

Browse files
authored
Merge pull request #1442 from hsolberg/feature/make-honorLabels-configurable
Helm: Add option to configure honorLabels for serviceMonitor
2 parents 2992a99 + 93a3aaa commit a41a7bb

File tree

3 files changed

+3
-1
lines changed

3 files changed

+3
-1
lines changed

charts/flagger/README.md

+1
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ The following tables lists the configurable parameters of the Flagger chart and
141141
| `prometheus.retention` | Prometheus data retention | `2h` |
142142
| `selectorLabels` | List of labels that Flagger uses to create pod selectors | `app,name,app.kubernetes.io/name` |
143143
| `serviceMonitor.enabled` | If `true`, creates service and serviceMonitor for monitoring Flagger metrics | `false` |
144+
| `serviceMonitor.honorLabels` | If `true`, label conflicts are resolved by keeping label values from the scraped data and ignoring the conflicting server-side labels | `false` |
144145
| `configTracking.enabled` | If `true`, flagger will track changes in Secrets and ConfigMaps referenced in the target deployment | `true` |
145146
| `eventWebhook` | If set, Flagger will publish events to the given webhook | None |
146147
| `slack.url` | Slack incoming webhook | None |

charts/flagger/templates/servicemonitor.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ spec:
1313
port: http
1414
interval: 30s
1515
scrapeTimeout: 30s
16-
honorLabels: false
16+
honorLabels: {{ .Values.serviceMonitor.honorLabels }}
1717
namespaceSelector:
1818
matchNames:
1919
- {{ .Release.Namespace }}

charts/flagger/values.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ metricsServer: "http://prometheus:9090"
2626
# creates serviceMonitor for monitoring Flagger metrics
2727
serviceMonitor:
2828
enabled: false
29+
honorLabels: false
2930

3031
# accepted values are kubernetes, istio, linkerd, appmesh, contour, nginx, gloo, skipper, traefik, apisix, osm
3132
meshProvider: ""

0 commit comments

Comments
 (0)