operator/controllers/execution/scheduledscan_controller.go
/*Copyright 2020 iteratec GmbH. Licensed under the Apache License, Version 2.0 (the "License");you may not use this file except in compliance with the License.You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0 Unless required by applicable law or agreed to in writing, softwaredistributed under the License is distributed on an "AS IS" BASIS,WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.See the License for the specific language governing permissions andlimitations under the License.*/ package controllers import ( "context" "fmt" "reflect" "sort" "time" "github.com/go-logr/logr" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/runtime" ctrl "sigs.k8s.io/controller-runtime" "sigs.k8s.io/controller-runtime/pkg/client" executionv1 "github.com/secureCodeBox/secureCodeBox-v2/operator/apis/execution/v1") var ( ownerKey = ".metadata.controller" apiGVStr = executionv1.GroupVersion.String()) // ScheduledScanReconciler reconciles a ScheduledScan objecttype ScheduledScanReconciler struct { client.Client Log logr.Logger Scheme *runtime.Scheme} // +kubebuilder:rbac:groups=execution.securecodebox.io,resources=scheduledscans,verbs=get;list;watch;create;update;patch;delete// +kubebuilder:rbac:groups=execution.securecodebox.io,resources=scheduledscans/status,verbs=get;update;patch// +kubebuilder:rbac:groups=execution.securecodebox.io,resources=scans,verbs=get;list;create// +kubebuilder:rbac:groups=execution.securecodebox.io,resources=scans/status,verbs=get // Reconcile comapares the ScheduledScan Resource with the State of the Cluster and updates both accordinglyMethod `ScheduledScanReconciler.Reconcile` has 79 lines of code (exceeds 50 allowed). Consider refactoring.
Method `ScheduledScanReconciler.Reconcile` has a Cognitive Complexity of 27 (exceeds 20 allowed). Consider refactoring.
Method `ScheduledScanReconciler.Reconcile` has 8 return statements (exceeds 4 allowed).func (r *ScheduledScanReconciler) Reconcile(req ctrl.Request) (ctrl.Result, error) { ctx := context.Background() log := r.Log.WithValues("scheduledscan", req.NamespacedName) // get the ScheduledScan var scheduledScan executionv1.ScheduledScan if err := r.Get(ctx, req.NamespacedName, &scheduledScan); err != nil { // we'll ignore not-found errors, since they can't be fixed by an immediate // requeue (we'll need to wait for a new notification), and we can get them // on deleted requests. log.V(7).Info("Unable to fetch ScheduledScan") return ctrl.Result{}, client.IgnoreNotFound(err) } var childScans executionv1.ScanList if err := r.List(ctx, &childScans, client.InNamespace(req.Namespace), client.MatchingFields{ownerKey: req.Name}); err != nil { log.Error(err, "unable to list child Scans") return ctrl.Result{}, err } log.V(8).Info("Got Child Scans for ScheduledScan", "count", len(childScans.Items)) // Get a sorted list of all successful child scans. var completedScans []executionv1.Scan for _, scan := range childScans.Items { if scan.Status.State == "Done" { completedScans = append(completedScans, scan) } } sort.Slice(completedScans, func(i, j int) bool { return completedScans[i].ObjectMeta.CreationTimestamp.Before(&completedScans[j].ObjectMeta.CreationTimestamp) }) // Update findingsStats of ScheduledScan to match the most recent Scan if len(completedScans) >= 1 { lastFindings := completedScans[len(completedScans)-1].Status.Findings if !reflect.DeepEqual(lastFindings, scheduledScan.Status.Findings) { log.V(4).Info("Updating ScheduledScans Findings as they appear to have changed") scheduledScan.Status.Findings = *lastFindings.DeepCopy() if err := r.Status().Update(ctx, &scheduledScan); err != nil { log.Error(err, "unable to update ScheduledScan status") return ctrl.Result{}, err } } } // Delete Old Scans when exceeding the history limit var historyLimit int32 = 3 if scheduledScan.Spec.SuccessfulJobsHistoryLimit != nil { historyLimit = *scheduledScan.Spec.SuccessfulJobsHistoryLimit } for i, scan := range completedScans { if int32(i) >= int32(len(completedScans))-historyLimit { break } if err := r.Delete(ctx, &scan, client.PropagationPolicy(metav1.DeletePropagationBackground)); (err) != nil { log.Error(err, "Unable to delete old Scan", "scan", scan.Name) } else { log.V(2).Info("Deleted old successful Scan", "scan", scan.Name) } } // Calculate the next schedule var nextSchedule time.Time if scheduledScan.Status.LastScheduleTime != nil { nextSchedule = scheduledScan.Status.LastScheduleTime.Add(scheduledScan.Spec.Interval.Duration) } else { nextSchedule = time.Now().Add(-1 * time.Second) } // check if it is time to start the next Scan if !time.Now().Before(nextSchedule) { // It's time! var scan = &executionv1.Scan{ ObjectMeta: metav1.ObjectMeta{ Namespace: scheduledScan.Namespace, Labels: scheduledScan.ObjectMeta.GetLabels(), }, Spec: *scheduledScan.Spec.ScanSpec.DeepCopy(), } scan.Name = fmt.Sprintf("%s-%d", scheduledScan.Name, nextSchedule.Unix()) if err := ctrl.SetControllerReference(&scheduledScan, scan, r.Scheme); err != nil { log.Error(err, "Unable to set owner reference on Scan") return ctrl.Result{}, err } if err := r.Create(ctx, scan); err != nil { log.Error(err, "Unable to create Scan for ScheduledScan") return ctrl.Result{}, err } var now metav1.Time = metav1.Now() scheduledScan.Status.LastScheduleTime = &now if err := r.Status().Update(ctx, &scheduledScan); err != nil { log.Error(err, "Unable to update ScheduledScan status") return ctrl.Result{}, err } // Recalculate next schedule nextSchedule = time.Now().Add(scheduledScan.Spec.Interval.Duration) } return ctrl.Result{RequeueAfter: nextSchedule.Sub(time.Now())}, nil} // SetupWithManager sets up the controller and initializes every thing it needsMethod `ScheduledScanReconciler.SetupWithManager` has 5 return statements (exceeds 4 allowed).func (r *ScheduledScanReconciler) SetupWithManager(mgr ctrl.Manager) error { if err := mgr.GetFieldIndexer().IndexField(&executionv1.Scan{}, ownerKey, func(rawObj runtime.Object) []string { // grab the job object, extract the owner... scan := rawObj.(*executionv1.Scan) owner := metav1.GetControllerOf(scan) if owner == nil { return nil } // ...make sure it's a Scan belonging to a Target... if owner.APIVersion != apiGVStr || owner.Kind != "ScheduledScan" { return nil } // ...and if so, return it return []string{owner.Name} }); err != nil { return err } return ctrl.NewControllerManagedBy(mgr). For(&executionv1.ScheduledScan{}). Owns(&executionv1.Scan{}). Complete(r)}