Merge pull request #94740 from tangwz/feat_use_Ticker_in_throughput

scheduler_perf: use time.Ticker in throughput measurement
This commit is contained in:
Kubernetes Prow Robot 2020-09-22 18:34:08 -07:00 committed by GitHub
commit 61a2d0d34e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -238,12 +238,13 @@ func (tc *throughputCollector) run(ctx context.Context) {
klog.Fatalf("%v", err) klog.Fatalf("%v", err)
} }
lastScheduledCount := len(podsScheduled) lastScheduledCount := len(podsScheduled)
ticker := time.NewTicker(throughputSampleFrequency)
defer ticker.Stop()
for { for {
select { select {
case <-ctx.Done(): case <-ctx.Done():
return return
// TODO(#94665): use time.Ticker instead case <-ticker.C:
case <-time.After(throughputSampleFrequency):
podsScheduled, err := getScheduledPods(tc.podInformer, tc.namespaces...) podsScheduled, err := getScheduledPods(tc.podInformer, tc.namespaces...)
if err != nil { if err != nil {
klog.Fatalf("%v", err) klog.Fatalf("%v", err)