chore: bump helm chart (#1619)

charts/actions-runner-controller/Chart.yaml

@@ -15,10 +15,10 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.20.0
+version: 0.20.1
 
 # Used as the default manager tag value when no tag property is provided in the values.yaml
-appVersion: 0.25.0
+appVersion: 0.25.1
 
 home: https://github.com/actions-runner-controller/actions-runner-controller
 

controllers/runnerdeployment_controller.go

@@ -165,8 +165,6 @@ func (r *RunnerDeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Req
 			return ctrl.Result{}, err
 		}
 
-		log.V(1).Info("Updated runnerreplicaset due to selector change")
-
 		// At this point, we are already sure that there's no need to create a new replicaset
 		// as the runner template hash is not changed.
 		//
@@ -184,14 +182,7 @@ func (r *RunnerDeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Req
 	//
 	// If we missed taking the EffectiveTime diff into account, you might end up experiencing scale-ups being delayed scale-down.
 	// See https://github.com/actions-runner-controller/actions-runner-controller/pull/1477#issuecomment-1164154496
-	var et1, et2 time.Time
-	if newestSet.Spec.EffectiveTime != nil {
-		et1 = newestSet.Spec.EffectiveTime.Time
-	}
-	if rd.Spec.EffectiveTime != nil {
-		et2 = rd.Spec.EffectiveTime.Time
-	}
-	if currentDesiredReplicas != newDesiredReplicas || et1 != et2 {
+	if currentDesiredReplicas != newDesiredReplicas || newestSet.Spec.EffectiveTime != rd.Spec.EffectiveTime {
 		newestSet.Spec.Replicas = &newDesiredReplicas
 		newestSet.Spec.EffectiveTime = rd.Spec.EffectiveTime
 
@@ -201,13 +192,6 @@ func (r *RunnerDeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Req
 			return ctrl.Result{}, err
 		}
 
-		log.V(1).Info("Updated runnerreplicaset due to spec change",
-			"currentDesiredReplicas", currentDesiredReplicas,
-			"newDesiredReplicas", newDesiredReplicas,
-			"currentEffectiveTime", newestSet.Spec.EffectiveTime,
-			"newEffectiveTime", rd.Spec.EffectiveTime,
-		)
-
 		return ctrl.Result{}, err
 	}