Skip to content

Commit

Permalink
Remove version mismatch check if web replicas=0
Browse files Browse the repository at this point in the history
closes: #1067
  • Loading branch information
git-hyagi committed Sep 28, 2024
1 parent 26ac1d9 commit 6466467
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGES/1067.removal
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Removed the validation for version mismatch between pulp-minimal and pulp-web images
when the number of replicas for pulp-web is set to 0.
2 changes: 1 addition & 1 deletion controllers/repo_manager/precheck.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ func initializeStatusCondition(ctx context.Context, r *RepoManagerReconciler, pu

// checkImageVersion verifies if pulp-webcheckImageVersion image version matches pulp-minimal
func checkImageVersion(r *RepoManagerReconciler, pulp *repomanagerpulpprojectorgv1beta2.Pulp) *ctrl.Result {
if r.needsPulpWeb(pulp) && pulp.Spec.ImageVersion != pulp.Spec.ImageWebVersion {
if r.needsPulpWeb(pulp) && pulp.Spec.ImageVersion != pulp.Spec.ImageWebVersion && pulp.Spec.Web.Replicas > 0 {
if pulp.Spec.InhibitVersionConstraint {
controllers.CustomZapLogger().Warn("image_version should be equal to image_web_version! Using different versions is not recommended and can make the application unreachable")
} else {
Expand Down
2 changes: 1 addition & 1 deletion main.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ func main() {
os.Exit(1)
}

setupLog.Info("pulp-operator version: 1.0.4-beta.5")
setupLog.Info("pulp-operator version: 1.0.5-beta.5")
setupLog.Info("starting manager")
if err := mgr.Start(ctrl.SetupSignalHandler()); err != nil {
setupLog.Error(err, "problem running manager")
Expand Down

0 comments on commit 6466467

Please sign in to comment.