Fix set operator version in Jenkins CR status
This commit is contained in:
		
							parent
							
								
									9204964d69
								
							
						
					
					
						commit
						ff2e76c977
					
				|  | @ -510,6 +510,7 @@ func (r *ReconcileJenkinsBaseConfiguration) ensureJenkinsMasterPod(meta metav1.O | ||||||
| 
 | 
 | ||||||
| 		now := metav1.Now() | 		now := metav1.Now() | ||||||
| 		r.Configuration.Jenkins.Status = v1alpha2.JenkinsStatus{ | 		r.Configuration.Jenkins.Status = v1alpha2.JenkinsStatus{ | ||||||
|  | 			OperatorVersion:     version.Version, | ||||||
| 			ProvisionStartTime:  &now, | 			ProvisionStartTime:  &now, | ||||||
| 			LastBackup:          r.Configuration.Jenkins.Status.LastBackup, | 			LastBackup:          r.Configuration.Jenkins.Status.LastBackup, | ||||||
| 			PendingBackup:       r.Configuration.Jenkins.Status.LastBackup, | 			PendingBackup:       r.Configuration.Jenkins.Status.LastBackup, | ||||||
|  |  | ||||||
|  | @ -18,7 +18,6 @@ import ( | ||||||
| 	"github.com/jenkinsci/kubernetes-operator/pkg/controller/jenkins/notifications/reason" | 	"github.com/jenkinsci/kubernetes-operator/pkg/controller/jenkins/notifications/reason" | ||||||
| 	"github.com/jenkinsci/kubernetes-operator/pkg/controller/jenkins/plugins" | 	"github.com/jenkinsci/kubernetes-operator/pkg/controller/jenkins/plugins" | ||||||
| 	"github.com/jenkinsci/kubernetes-operator/pkg/log" | 	"github.com/jenkinsci/kubernetes-operator/pkg/log" | ||||||
| 	"github.com/jenkinsci/kubernetes-operator/version" |  | ||||||
| 
 | 
 | ||||||
| 	"github.com/go-logr/logr" | 	"github.com/go-logr/logr" | ||||||
| 	"github.com/pkg/errors" | 	"github.com/pkg/errors" | ||||||
|  | @ -419,16 +418,6 @@ func (r *ReconcileJenkins) setDefaults(jenkins *v1alpha2.Jenkins, logger logr.Lo | ||||||
| 		changed = true | 		changed = true | ||||||
| 		jenkins.Spec.Master.BasePlugins = basePlugins() | 		jenkins.Spec.Master.BasePlugins = basePlugins() | ||||||
| 	} | 	} | ||||||
| 	if len(jenkins.Status.OperatorVersion) > 0 && version.Version != jenkins.Status.OperatorVersion { |  | ||||||
| 		logger.Info("Setting default operator plugins after Operator version change") |  | ||||||
| 		changed = true |  | ||||||
| 		jenkins.Spec.Master.BasePlugins = basePlugins() |  | ||||||
| 	} |  | ||||||
| 	if len(jenkins.Status.OperatorVersion) == 0 { |  | ||||||
| 		logger.Info("Setting operator version") |  | ||||||
| 		changed = true |  | ||||||
| 		jenkins.Status.OperatorVersion = version.Version |  | ||||||
| 	} |  | ||||||
| 	if isResourceRequirementsNotSet(jenkinsContainer.Resources) { | 	if isResourceRequirementsNotSet(jenkinsContainer.Resources) { | ||||||
| 		logger.Info("Setting default Jenkins master container resource requirements") | 		logger.Info("Setting default Jenkins master container resource requirements") | ||||||
| 		changed = true | 		changed = true | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue