fix lint errors
This commit is contained in:
		
							parent
							
								
									88b8ba6aaf
								
							
						
					
					
						commit
						501beb8385
					
				| 
						 | 
					@ -394,7 +394,7 @@ func (r *AutoscalingRunnerSetReconciler) removeFinalizersFromDependentResources(
 | 
				
			||||||
 | 
					
 | 
				
			||||||
func (r *AutoscalingRunnerSetReconciler) createRunnerScaleSet(ctx context.Context, autoscalingRunnerSet *v1alpha1.AutoscalingRunnerSet, logger logr.Logger) (ctrl.Result, error) {
 | 
					func (r *AutoscalingRunnerSetReconciler) createRunnerScaleSet(ctx context.Context, autoscalingRunnerSet *v1alpha1.AutoscalingRunnerSet, logger logr.Logger) (ctrl.Result, error) {
 | 
				
			||||||
	logger.Info("Creating a new runner scale set")
 | 
						logger.Info("Creating a new runner scale set")
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, autoscalingRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, autoscalingRunnerSet)
 | 
				
			||||||
	if len(autoscalingRunnerSet.Spec.RunnerScaleSetName) == 0 {
 | 
						if len(autoscalingRunnerSet.Spec.RunnerScaleSetName) == 0 {
 | 
				
			||||||
		autoscalingRunnerSet.Spec.RunnerScaleSetName = autoscalingRunnerSet.Name
 | 
							autoscalingRunnerSet.Spec.RunnerScaleSetName = autoscalingRunnerSet.Name
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					@ -490,7 +490,7 @@ func (r *AutoscalingRunnerSetReconciler) updateRunnerScaleSetRunnerGroup(ctx con
 | 
				
			||||||
		return ctrl.Result{}, err
 | 
							return ctrl.Result{}, err
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, autoscalingRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, autoscalingRunnerSet)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
							logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
				
			||||||
		return ctrl.Result{}, err
 | 
							return ctrl.Result{}, err
 | 
				
			||||||
| 
						 | 
					@ -538,7 +538,7 @@ func (r *AutoscalingRunnerSetReconciler) updateRunnerScaleSetName(ctx context.Co
 | 
				
			||||||
		return ctrl.Result{}, nil
 | 
							return ctrl.Result{}, nil
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, autoscalingRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, autoscalingRunnerSet)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
							logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
				
			||||||
		return ctrl.Result{}, err
 | 
							return ctrl.Result{}, err
 | 
				
			||||||
| 
						 | 
					@ -589,7 +589,7 @@ func (r *AutoscalingRunnerSetReconciler) deleteRunnerScaleSet(ctx context.Contex
 | 
				
			||||||
		return nil
 | 
							return nil
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, autoscalingRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, autoscalingRunnerSet)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
							logger.Error(err, "Failed to initialize Actions service client for updating a existing runner scale set")
 | 
				
			||||||
		return err
 | 
							return err
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -528,7 +528,7 @@ func (r *EphemeralRunnerReconciler) deletePodAsFailed(ctx context.Context, ephem
 | 
				
			||||||
func (r *EphemeralRunnerReconciler) updateStatusWithRunnerConfig(ctx context.Context, ephemeralRunner *v1alpha1.EphemeralRunner, log logr.Logger) (*ctrl.Result, error) {
 | 
					func (r *EphemeralRunnerReconciler) updateStatusWithRunnerConfig(ctx context.Context, ephemeralRunner *v1alpha1.EphemeralRunner, log logr.Logger) (*ctrl.Result, error) {
 | 
				
			||||||
	// Runner is not registered with the service. We need to register it first
 | 
						// Runner is not registered with the service. We need to register it first
 | 
				
			||||||
	log.Info("Creating ephemeral runner JIT config")
 | 
						log.Info("Creating ephemeral runner JIT config")
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, ephemeralRunner)
 | 
						actionsClient, err := r.GetActionsService(ctx, ephemeralRunner)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		return &ctrl.Result{}, fmt.Errorf("failed to get actions client for generating JIT config: %w", err)
 | 
							return &ctrl.Result{}, fmt.Errorf("failed to get actions client for generating JIT config: %w", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					@ -755,7 +755,7 @@ func (r *EphemeralRunnerReconciler) updateRunStatusFromPod(ctx context.Context,
 | 
				
			||||||
// runnerRegisteredWithService checks if the runner is still registered with the service
 | 
					// runnerRegisteredWithService checks if the runner is still registered with the service
 | 
				
			||||||
// Returns found=false and err=nil if ephemeral runner does not exist in GitHub service and should be deleted
 | 
					// Returns found=false and err=nil if ephemeral runner does not exist in GitHub service and should be deleted
 | 
				
			||||||
func (r EphemeralRunnerReconciler) runnerRegisteredWithService(ctx context.Context, runner *v1alpha1.EphemeralRunner, log logr.Logger) (found bool, err error) {
 | 
					func (r EphemeralRunnerReconciler) runnerRegisteredWithService(ctx context.Context, runner *v1alpha1.EphemeralRunner, log logr.Logger) (found bool, err error) {
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, runner)
 | 
						actionsClient, err := r.GetActionsService(ctx, runner)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		return false, fmt.Errorf("failed to get Actions client for ScaleSet: %w", err)
 | 
							return false, fmt.Errorf("failed to get Actions client for ScaleSet: %w", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					@ -782,7 +782,7 @@ func (r EphemeralRunnerReconciler) runnerRegisteredWithService(ctx context.Conte
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
func (r *EphemeralRunnerReconciler) deleteRunnerFromService(ctx context.Context, ephemeralRunner *v1alpha1.EphemeralRunner, log logr.Logger) error {
 | 
					func (r *EphemeralRunnerReconciler) deleteRunnerFromService(ctx context.Context, ephemeralRunner *v1alpha1.EphemeralRunner, log logr.Logger) error {
 | 
				
			||||||
	client, err := r.SecretResolver.GetActionsService(ctx, ephemeralRunner)
 | 
						client, err := r.GetActionsService(ctx, ephemeralRunner)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		return fmt.Errorf("failed to get actions client for runner: %w", err)
 | 
							return fmt.Errorf("failed to get actions client for runner: %w", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -331,7 +331,7 @@ func (r *EphemeralRunnerSetReconciler) cleanUpEphemeralRunners(ctx context.Conte
 | 
				
			||||||
		return false, nil
 | 
							return false, nil
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, ephemeralRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, ephemeralRunnerSet)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		return false, err
 | 
							return false, err
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					@ -439,7 +439,7 @@ func (r *EphemeralRunnerSetReconciler) deleteIdleEphemeralRunners(ctx context.Co
 | 
				
			||||||
		log.Info("No pending or running ephemeral runners running at this time for scale down")
 | 
							log.Info("No pending or running ephemeral runners running at this time for scale down")
 | 
				
			||||||
		return nil
 | 
							return nil
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
	actionsClient, err := r.SecretResolver.GetActionsService(ctx, ephemeralRunnerSet)
 | 
						actionsClient, err := r.GetActionsService(ctx, ephemeralRunnerSet)
 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		return fmt.Errorf("failed to create actions client for ephemeral runner replica set: %w", err)
 | 
							return fmt.Errorf("failed to create actions client for ephemeral runner replica set: %w", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue