resmon: ann: make sure to report meaningful times

Make sure we always report meaningful data. Most notable, 1 second granularity is sufficient for all the expected use cases (e2e tests) In general, if the update period is less than 1 second, the configuration is very suspicious anyway. Signed-off-by: Francesco Romani <[email protected]>
k8stopologyawareschedwg · Jun 14, 2022 · 7e38bdb · 7e38bdb
1 parent 94dd528
commit 7e38bdb
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/pkg/resourcetopologyexporter/resourceobserver.go b/pkg/resourcetopologyexporter/resourceobserver.go
@@ -63,8 +63,8 @@ func (rm *ResourceObserver) Run(eventsChan <-chan notification.Event, condChan c
 			tsEnd := time.Now()
 
 			if rm.exposeTiming {
-				monInfo.Annotations[k8sannotations.SleepDuration] = tsWakeupDiff.String()
-				monInfo.Annotations[k8sannotations.UpdateInterval] = ev.TimerInterval.String()
+				monInfo.Annotations[k8sannotations.SleepDuration] = clampTime(tsWakeupDiff.Round(time.Second)).String()
+				monInfo.Annotations[k8sannotations.UpdateInterval] = clampTime(ev.TimerInterval).String()
 			}
 
 			condStatus := v1.ConditionTrue
@@ -85,3 +85,10 @@ func (rm *ResourceObserver) Run(eventsChan <-chan notification.Event, condChan c
 		}
 	}
 }
+
+func clampTime(t time.Duration) time.Duration {
+	if t < 0 {
+		return 0
+	}
+	return t
+}