Small fixes

pkg/controller/jenkins/configuration/base/reconcile.go

@@ -232,8 +232,8 @@ func (r *ReconcileJenkinsBaseConfiguration) createUserConfigurationConfigMap(met
 
 func (r *ReconcileJenkinsBaseConfiguration) createRBAC(meta metav1.ObjectMeta) error {
 	serviceAccount := resources.NewServiceAccount(meta)
-	err := r.createOrUpdateResource(serviceAccount)
+	err := r.createResource(serviceAccount)
-	if err != nil {
+	if err != nil && !errors.IsAlreadyExists(err) {
 		return err
 	}
 

pkg/controller/jenkins/configuration/user/seedjobs/seedjobs.go

@@ -20,7 +20,7 @@ import (
 
 const (
 	// ConfigureSeedJobsName this is the fixed seed job name
-	ConfigureSeedJobsName = "Configure Seed Jobs"
+	ConfigureSeedJobsName = constants.OperatorName + "-configure-seed-job"
 
 	deployKeyIDParameterName      = "DEPLOY_KEY_ID"
 	privateKeyParameterName       = "PRIVATE_KEY"
@@ -128,42 +128,42 @@ func (s *SeedJobs) privateKeyFromSecret(namespace string, seedJob virtuslabv1alp
 var seedJobConfigXML = `
 <flow-definition plugin="workflow-job@2.30">
   <actions/>
-  <description></description>
+  <description>Configure Seed Jobs</description>
   <keepDependencies>false</keepDependencies>
   <properties>
     <hudson.model.ParametersDefinitionProperty>
       <parameterDefinitions>
         <hudson.model.StringParameterDefinition>
-          <name>DEPLOY_KEY_ID</name>
+          <name>` + deployKeyIDParameterName + `</name>
           <description></description>
           <defaultValue></defaultValue>
           <trim>false</trim>
         </hudson.model.StringParameterDefinition>
         <hudson.model.StringParameterDefinition>
-          <name>PRIVATE_KEY</name>
+          <name>` + privateKeyParameterName + `</name>
           <description></description>
           <defaultValue></defaultValue>
         </hudson.model.StringParameterDefinition>
         <hudson.model.StringParameterDefinition>
-          <name>REPOSITORY_URL</name>
+          <name>` + repositoryURLParameterName + `</name>
           <description></description>
           <defaultValue></defaultValue>
           <trim>false</trim>
         </hudson.model.StringParameterDefinition>
         <hudson.model.StringParameterDefinition>
-          <name>REPOSITORY_BRANCH</name>
+          <name>` + repositoryBranchParameterName + `</name>
           <description></description>
           <defaultValue>master</defaultValue>
           <trim>false</trim>
         </hudson.model.StringParameterDefinition>
         <hudson.model.StringParameterDefinition>
-          <name>SEED_JOB_DISPLAY_NAME</name>
+          <name>` + displayNameParameterName + `</name>
           <description></description>
           <defaultValue></defaultValue>
           <trim>false</trim>
         </hudson.model.StringParameterDefinition>
         <hudson.model.StringParameterDefinition>
-          <name>TARGETS</name>
+          <name>` + targetsParameterName + `</name>
           <description></description>
           <defaultValue>cicd/jobs/*.jenkins</defaultValue>
           <trim>false</trim>

pkg/controller/jenkins/handler.go

@@ -15,7 +15,7 @@ type enqueueRequestForJenkins struct{}
 
 func (e *enqueueRequestForJenkins) Create(evt event.CreateEvent, q workqueue.RateLimitingInterface) {
 	if req := e.getOwnerReconcileRequests(evt.Meta); req != nil {
-		q.Add(req)
+		q.Add(*req)
 	}
 }
 

pkg/controller/jenkins/jenkins_controller.go

@@ -2,7 +2,6 @@ package jenkins
 
 import (
 	"context"
-	"fmt"
 
 	virtuslabv1alpha1 "github.com/VirtusLab/jenkins-operator/pkg/apis/virtuslab/v1alpha1"
 	"github.com/VirtusLab/jenkins-operator/pkg/controller/jenkins/configuration/base"
@@ -91,11 +90,9 @@ func (r *ReconcileJenkins) Reconcile(request reconcile.Request) (reconcile.Resul
 	logger.Info("Reconciling Jenkins")
 
 	result, err := r.reconcile(request, logger)
-	if err != nil {
+	if err != nil && errors.IsConflict(err) {
-		if errors.IsConflict(err) {
+		logger.V(log.VWarn).Info(err.Error())
-			logger.V(log.VWarn).Info(fmt.Sprintf("The %s object has been modified, requeue reconciliation loop", request.Name))
+		return reconcile.Result{Requeue: true}, nil
-			return reconcile.Result{Requeue: true}, nil
-		}
 	}
 	return result, err
 }