Skip to content

Commit

Permalink
Adjust the Prometheus service discovery for the etcd pods (gardener#8371
Browse files Browse the repository at this point in the history
)

The labels on the etcd service changed with
gardener/etcd-druid#559 that was introduced in
gardener#8299.

The `app` and `role` labels are no longer present on the service, but they are
still available on the pods.

So by using the pod's labels for the service discovery, we can restore the
previous behavior.

https://prometheus.io/docs/prometheus/latest/configuration/configuration/#endpoints

Co-authored-by: Jeremy Rickards <jeremy.rickards@sap.com>
  • Loading branch information
istvanballok and rickardsjp authored Aug 18, 2023
1 parent bdcda41 commit e6767ed
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 12 deletions.
12 changes: 6 additions & 6 deletions pkg/component/etcd/monitoring.go
Original file line number Diff line number Diff line change
Expand Up @@ -302,12 +302,12 @@ kubernetes_sd_configs:
names: [{{ .namespace }}]
relabel_configs:
- source_labels:
- __meta_kubernetes_service_label_` + v1beta1constants.LabelApp + `
- __meta_kubernetes_service_label_` + v1beta1constants.LabelRole + `
- __meta_kubernetes_pod_label_` + v1beta1constants.LabelApp + `
- __meta_kubernetes_pod_label_` + v1beta1constants.LabelRole + `
- __meta_kubernetes_endpoint_port_name
action: keep
regex: ` + LabelAppValue + `;{{ .role }};` + portNameClient + `
- source_labels: [ __meta_kubernetes_service_label_` + v1beta1constants.LabelRole + ` ]
- source_labels: [ __meta_kubernetes_pod_label_` + v1beta1constants.LabelRole + ` ]
target_label: role
- source_labels: [ __meta_kubernetes_pod_name ]
target_label: pod
Expand All @@ -332,12 +332,12 @@ kubernetes_sd_configs:
names: [{{ .namespace }}]
relabel_configs:
- source_labels:
- __meta_kubernetes_service_label_` + v1beta1constants.LabelApp + `
- __meta_kubernetes_service_label_` + v1beta1constants.LabelRole + `
- __meta_kubernetes_pod_label_` + v1beta1constants.LabelApp + `
- __meta_kubernetes_pod_label_` + v1beta1constants.LabelRole + `
- __meta_kubernetes_endpoint_port_name
action: keep
regex: ` + LabelAppValue + `;{{ .role }};` + portNameBackupRestore + `
- source_labels: [ __meta_kubernetes_service_label_role ]
- source_labels: [ __meta_kubernetes_pod_label_role ]
target_label: role
- source_labels: [ __meta_kubernetes_pod_name ]
target_label: pod
Expand Down
12 changes: 6 additions & 6 deletions pkg/component/etcd/monitoring_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -175,12 +175,12 @@ kubernetes_sd_configs:
names: [` + testNamespace + `]
relabel_configs:
- source_labels:
- __meta_kubernetes_service_label_app
- __meta_kubernetes_service_label_role
- __meta_kubernetes_pod_label_app
- __meta_kubernetes_pod_label_role
- __meta_kubernetes_endpoint_port_name
action: keep
regex: etcd-statefulset;` + testRole + `;client
- source_labels: [ __meta_kubernetes_service_label_role ]
- source_labels: [ __meta_kubernetes_pod_label_role ]
target_label: role
- source_labels: [ __meta_kubernetes_pod_name ]
target_label: pod
Expand All @@ -204,12 +204,12 @@ kubernetes_sd_configs:
names: [` + testNamespace + `]
relabel_configs:
- source_labels:
- __meta_kubernetes_service_label_app
- __meta_kubernetes_service_label_role
- __meta_kubernetes_pod_label_app
- __meta_kubernetes_pod_label_role
- __meta_kubernetes_endpoint_port_name
action: keep
regex: etcd-statefulset;` + testRole + `;backuprestore
- source_labels: [ __meta_kubernetes_service_label_role ]
- source_labels: [ __meta_kubernetes_pod_label_role ]
target_label: role
- source_labels: [ __meta_kubernetes_pod_name ]
target_label: pod
Expand Down

0 comments on commit e6767ed

Please sign in to comment.