Fix for multiple /'s in application image name (#327)

Signed-off-by: Jason Yong <[email protected]>
OpenLiberty · Mar 16, 2022 · e9bba48 · e9bba48
1 parent 37f3a47
commit e9bba48
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/controllers/openlibertyapplication_controller.go b/controllers/openlibertyapplication_controller.go
@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"os"
+    "strings"
 
 	networkingv1 "k8s.io/api/networking/v1"
 
@@ -200,7 +201,7 @@ func (r *ReconcileOpenLiberty) Reconcile(ctx context.Context, request ctrl.Reque
 				if image.DockerImageReference != "" {
 					instance.Status.ImageReference = image.DockerImageReference
 				}
-			} else if err != nil && !kerrors.IsNotFound(err) && !kerrors.IsForbidden(err) {
+			} else if err != nil && !kerrors.IsNotFound(err) && !kerrors.IsForbidden(err) && !strings.Contains(isTagName, "/") {
 				return r.ManageError(err, common.StatusConditionTypeReconciled, instance)
 			}
 		}