Skip to content

Commit edc3ddc

Browse files
authored
Merge pull request #630 from AndrewSirenko/leader-metrics
Add leader election, work-queue, process, and Go runtime metrics
2 parents 8073914 + c0cac75 commit edc3ddc

File tree

8 files changed

+242
-11
lines changed

8 files changed

+242
-11
lines changed

cmd/csi-attacher/main.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,9 @@ import (
3434
"k8s.io/component-base/logs"
3535
logsapi "k8s.io/component-base/logs/api/v1"
3636
_ "k8s.io/component-base/logs/json/register"
37+
"k8s.io/component-base/metrics/legacyregistry"
38+
_ "k8s.io/component-base/metrics/prometheus/clientgo/leaderelection" // register leader election in the default legacy registry
39+
_ "k8s.io/component-base/metrics/prometheus/workqueue" // register work queues in the default legacy registry
3740
csitrans "k8s.io/csi-translation-lib"
3841
"k8s.io/klog/v2"
3942

@@ -188,6 +191,10 @@ func main() {
188191
}
189192
}
190193

194+
// Add default legacy registry so that metrics manager serves Go runtime and process metrics.
195+
// Also registers the `k8s.io/component-base/` work queue and leader election metrics we anonymously import.
196+
metricsManager.WithAdditionalRegistry(legacyregistry.DefaultGatherer)
197+
191198
// Prepare http endpoint for metrics + leader election healthz
192199
mux := http.NewServeMux()
193200
if addr != "" {

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ require (
88
github.com/evanphx/json-patch v5.9.0+incompatible
99
github.com/golang/mock v1.6.0
1010
github.com/golang/protobuf v1.5.4
11-
github.com/kubernetes-csi/csi-lib-utils v0.20.0
11+
github.com/kubernetes-csi/csi-lib-utils v0.21.0
1212
github.com/kubernetes-csi/csi-test/v5 v5.3.1
1313
google.golang.org/grpc v1.69.4
1414
k8s.io/api v0.32.1

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@ github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
6565
github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
6666
github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
6767
github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
68-
github.com/kubernetes-csi/csi-lib-utils v0.20.0 h1:JTvHRJugn+cByMnIU4nCnqPqOOUhuPzhlLqRvenwjDA=
69-
github.com/kubernetes-csi/csi-lib-utils v0.20.0/go.mod h1:3b/HFVURW11oxV/gUAKyhhkvFpxXO/zRdvh1wdEfCZY=
68+
github.com/kubernetes-csi/csi-lib-utils v0.21.0 h1:dUN/iIgXLucAxyML2iPyhniIlACQumIeAJmIzsMBddc=
69+
github.com/kubernetes-csi/csi-lib-utils v0.21.0/go.mod h1:ZCVRTYuup+bwX9tOeE5Q3LDw64QvltSwMUQ3M3g2T+Q=
7070
github.com/kubernetes-csi/csi-test/v5 v5.3.1 h1:Wiukp1In+kif+BFo6q2ExjgB+MbrAz4jZWzGfijypuY=
7171
github.com/kubernetes-csi/csi-test/v5 v5.3.1/go.mod h1:7hA2cSYJ6T8CraEZPA6zqkLZwemjBD54XAnPsPC3VpA=
7272
github.com/kylelemons/godebug v1.1.0 h1:RPNrshWIDI6G2gRW9EHilWtl7Z6Sb1BR0xunSBf0SNc=

vendor/github.com/kubernetes-csi/csi-lib-utils/metrics/metrics.go

Lines changed: 19 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/kubernetes-csi/csi-lib-utils/protosanitizer/protosanitizer.go

Lines changed: 5 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/k8s.io/component-base/metrics/prometheus/clientgo/leaderelection/metrics.go

Lines changed: 68 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/k8s.io/component-base/metrics/prometheus/workqueue/metrics.go

Lines changed: 137 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/modules.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ github.com/klauspost/compress/internal/cpuinfo
9494
github.com/klauspost/compress/internal/snapref
9595
github.com/klauspost/compress/zstd
9696
github.com/klauspost/compress/zstd/internal/xxhash
97-
# github.com/kubernetes-csi/csi-lib-utils v0.20.0
97+
# github.com/kubernetes-csi/csi-lib-utils v0.21.0
9898
## explicit; go 1.23.1
9999
github.com/kubernetes-csi/csi-lib-utils/accessmodes
100100
github.com/kubernetes-csi/csi-lib-utils/connection
@@ -795,7 +795,9 @@ k8s.io/component-base/logs/json/register
795795
k8s.io/component-base/logs/klogflags
796796
k8s.io/component-base/metrics
797797
k8s.io/component-base/metrics/legacyregistry
798+
k8s.io/component-base/metrics/prometheus/clientgo/leaderelection
798799
k8s.io/component-base/metrics/prometheus/feature
800+
k8s.io/component-base/metrics/prometheus/workqueue
799801
k8s.io/component-base/metrics/prometheusextension
800802
k8s.io/component-base/version
801803
# k8s.io/csi-translation-lib v0.32.1

0 commit comments

Comments
 (0)