github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #98391 from Jiawei0227/process_start_metric

Browse Source 
Bug Fix for process_start_time_metric initialization
This commit is contained in:
Kubernetes Prow Robot
2021-01-26 10:30:44 -08:00
committed by GitHub
parent ed4025a47b edf9e03660
commit 013024864e
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
staging/src/k8s.io/component-base/metrics/processstarttime.go
View File

@@ -44,13 +44,13 @@ func RegisterProcessStartTime(registrationFunc func(Registerable) error) error {
start = float64(time.Now().Unix()) start = float64(time.Now().Unix())
} }
// processStartTime is a lazy metric which only get initialized after registered. // processStartTime is a lazy metric which only get initialized after registered.
// so we have to explicitly create it before setting the label value. Otherwise // so we need to register the metric first and then set the value for it
// it is a noop. if err = registrationFunc(processStartTime); err != nil {
if !processStartTime.IsCreated() { return err
processStartTime.initializeMetric()
} }
processStartTime.WithLabelValues().Set(start) processStartTime.WithLabelValues().Set(start)
return registrationFunc(processStartTime) return nil
} }
func getProcessStart() (float64, error) { func getProcessStart() (float64, error) {