diff --git a/controllers/horizontal_runner_autoscaler_webhook.go b/controllers/horizontal_runner_autoscaler_webhook.go index 9553c33c..7bf1e44d 100644 --- a/controllers/horizontal_runner_autoscaler_webhook.go +++ b/controllers/horizontal_runner_autoscaler_webhook.go @@ -351,9 +351,7 @@ func (autoscaler *HorizontalRunnerAutoscalerGitHubWebhook) findHRAsByKey(ctx con return nil, err } - for _, d := range hraList.Items { - hras = append(hras, d) - } + hras = append(hras, hraList.Items...) } return hras, nil @@ -797,7 +795,7 @@ func (autoscaler *HorizontalRunnerAutoscalerGitHubWebhook) tryScale(ctx context. after := len(copy.Spec.CapacityReservations) autoscaler.Log.V(1).Info( - "Patching hra for capacityReservations update", + fmt.Sprintf("Patching hra %s for capacityReservations update", target.HorizontalRunnerAutoscaler.Name), "before", before, "expired", expired, "amount", amount, @@ -837,6 +835,7 @@ func (autoscaler *HorizontalRunnerAutoscalerGitHubWebhook) SetupWithManager(mgr hra := rawObj.(*v1alpha1.HorizontalRunnerAutoscaler) if hra.Spec.ScaleTargetRef.Name == "" { + autoscaler.Log.V(1).Info(fmt.Sprintf("scale target ref name not set for hra %s", hra.Name)) return nil } diff --git a/controllers/runnerreplicaset_controller.go b/controllers/runnerreplicaset_controller.go index e4ac040f..ecaa70ba 100644 --- a/controllers/runnerreplicaset_controller.go +++ b/controllers/runnerreplicaset_controller.go @@ -199,7 +199,7 @@ func (r *RunnerReplicaSetReconciler) Reconcile(ctx context.Context, req ctrl.Req n := current - desired - log.V(0).Info(fmt.Sprintf("Deleting %d runners", n), "desired", desired, "current", current, "ready", ready) + log.V(0).Info(fmt.Sprintf("Deleting %d runners from RunnerReplicaSet %s", n, req.NamespacedName), "desired", desired, "current", current, "ready", ready) // get runners that are currently offline/not busy/timed-out to register var deletionCandidates []v1alpha1.Runner @@ -277,7 +277,7 @@ func (r *RunnerReplicaSetReconciler) Reconcile(ctx context.Context, req ctrl.Req } r.Recorder.Event(&rs, corev1.EventTypeNormal, "RunnerDeleted", fmt.Sprintf("Deleted runner '%s'", deletionCandidates[i].Name)) - log.Info("Deleted runner") + log.Info(fmt.Sprintf("Deleted runner %s", deletionCandidates[i].Name)) } } else if desired > current { n := desired - current