Merge pull request #504 from aleksandra-malinowska/no-lead-no-metrics

Don't register metrics unless on leading master
kubernetes · Dec 15, 2017 · af51acc · af51acc
2 parents 6b9c9ec + 312f989
commit af51acc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/cluster-autoscaler/main.go b/cluster-autoscaler/main.go
@@ -233,6 +233,7 @@ func registerSignalHandlers(autoscaler core.Autoscaler) {
 }
 
 func run(healthCheck *metrics.HealthCheck) {
+	metrics.RegisterAll()
 	kubeClient := createKubeClient()
 	kubeEventRecorder := kube_util.CreateEventRecorder(kubeClient)
 	opts := createAutoscalerOptions()

diff --git a/cluster-autoscaler/metrics/metrics.go b/cluster-autoscaler/metrics/metrics.go
@@ -212,7 +212,8 @@ var (
 	)
 )
 
-func init() {
+// RegisterAll registers all metrics.
+func RegisterAll() {
 	prometheus.MustRegister(clusterSafeToAutoscale)
 	prometheus.MustRegister(nodesCount)
 	prometheus.MustRegister(nodeGroupsCount)