#275 Changed the requeue time from 5min to 1min

Ilyesbdlala · Ilyesbdlala · commit fd48eebbb29c · 2023-07-18T09:29:06.000+02:00
This is done as a compromise between speed and performance see discussion: #1749 (comment) Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
diff --git a/operator/controllers/execution/scheduledscan_controller.go b/operator/controllers/execution/scheduledscan_controller.go
@@ -120,7 +120,7 @@ func (r *ScheduledScanReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 		if scheduledScan.Spec.ConcurrencyPolicy == executionv1.ForbidConcurrent && len(InProgressScans) > 0 {
 			log.V(8).Info("concurrency policy blocks concurrent runs, skipping", "num active", len(InProgressScans))
 			r.Recorder.Event(&scheduledScan, "Normal", "ConcurrencyPolicyBlocks", "Concurrency policy blocks concurrent runs, skipping")
-			return ctrl.Result{RequeueAfter: 5 * time.Minute}, nil
+			return ctrl.Result{RequeueAfter: 1 * time.Minute}, nil
 		}
 
 		// ...or instruct us to replace existing ones...

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ func (r *ScheduledScanReconciler) Reconcile(ctx context.Context, req ctrl.Reques`
`120`	`120`	`if scheduledScan.Spec.ConcurrencyPolicy == executionv1.ForbidConcurrent && len(InProgressScans) > 0 {`
`121`	`121`	`log.V(8).Info("concurrency policy blocks concurrent runs, skipping", "num active", len(InProgressScans))`
`122`	`122`	`r.Recorder.Event(&scheduledScan, "Normal", "ConcurrencyPolicyBlocks", "Concurrency policy blocks concurrent runs, skipping")`
`123`		`- return ctrl.Result{RequeueAfter: 5 * time.Minute}, nil`
	`123`	`+ return ctrl.Result{RequeueAfter: 1 * time.Minute}, nil`
`124`	`124`	`}`
`125`	`125`
`126`	`126`	`// ...or instruct us to replace existing ones...`