Skip to content

Commit 46cb558

Browse files
authored
Merge pull request #11 from DataDog/fricounet/fix-panic
[metrics] Fix panic during metrics manager startup
2 parents 7486c62 + b48dc6f commit 46cb558

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

pkg/metrics/metrics.go

+1-6
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,6 @@ const (
3838
)
3939

4040
var (
41-
gkeComponentVersion *metrics.GaugeVec
42-
pdcsiOperationErrorsMetric *metrics.CounterVec
43-
)
44-
45-
func initMetrics() {
4641
// This metric is exposed only from the controller driver component when GKE_PDCSI_VERSION env variable is set.
4742
gkeComponentVersion = metrics.NewGaugeVec(&metrics.GaugeOpts{
4843
Name: "component_version",
@@ -57,7 +52,7 @@ func initMetrics() {
5752
StabilityLevel: metrics.ALPHA,
5853
},
5954
[]string{"driver_name", "method_name", "grpc_status_code", "disk_type", "enable_confidential_storage", "enable_storage_pools"})
60-
}
55+
)
6156

6257
type MetricsManager struct {
6358
registry metrics.KubeRegistry

pkg/metrics/metrics_test_util.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,6 @@ package metrics
1919
// Test-only method used for resetting metric counts.
2020
func (mm *MetricsManager) ResetMetrics() {
2121
// Re-initialize metrics
22-
initMetrics()
22+
gkeComponentVersion.Reset()
23+
pdcsiOperationErrorsMetric.Reset()
2324
}

0 commit comments

Comments
 (0)