Merge pull request #3428 from marseel/pr/marseel/fix_timelimit

k8s-ci-robot · web-flow · commit ad70e60c7204 · 2025-06-23T03:40:53.000-07:00
tuningSet: fix TimeLimitedLoad for zero actions
diff --git a/clusterloader2/pkg/tuningset/time_limited.go b/clusterloader2/pkg/tuningset/time_limited.go
@@ -34,6 +34,9 @@ func newTimeLimitedLoad(params *api.TimeLimitedLoad) TuningSet {
 }
 
 func (t *timeLimitedLoad) Execute(actions []func()) {
+	if len(actions) == 0 {
+		return
+	}
 	sleepDuration := time.Duration(t.params.TimeLimit.ToTimeDuration().Nanoseconds() / int64(len(actions)))
 	var wg wait.Group
 	for i := range actions {

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,9 @@ func newTimeLimitedLoad(params *api.TimeLimitedLoad) TuningSet {`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`func (t *timeLimitedLoad) Execute(actions []func()) {`
	`37`	`+ if len(actions) == 0 {`
	`38`	`+ return`
	`39`	`+ }`
`37`	`40`	`sleepDuration := time.Duration(t.params.TimeLimit.ToTimeDuration().Nanoseconds() / int64(len(actions)))`
`38`	`41`	`var wg wait.Group`
`39`	`42`	`for i := range actions {`