Add 3 or more '/' fix (#344)

halim-lee · web-flow · commit 8c26ed65fe9d · 2022-03-08T11:55:23.000-05:00
diff --git a/controllers/runtimecomponent_controller.go b/controllers/runtimecomponent_controller.go
@@ -20,6 +20,7 @@ import (
 	"context"
 	"fmt"
 	"os"
+	"strings"
 
 	"github.com/application-stacks/runtime-component-operator/common"
 	"github.com/pkg/errors"
@@ -183,7 +184,7 @@ func (r *RuntimeComponentReconciler) Reconcile(ctx context.Context, req ctrl.Req
 				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)
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ import (`
`20`	`20`	`"context"`
`21`	`21`	`"fmt"`
`22`	`22`	`"os"`
	`23`	`+ "strings"`
`23`	`24`
`24`	`25`	`"github.com/application-stacks/runtime-component-operator/common"`
`25`	`26`	`"github.com/pkg/errors"`
`@@ -183,7 +184,7 @@ func (r *RuntimeComponentReconciler) Reconcile(ctx context.Context, req ctrl.Req`
`183`	`184`	`if image.DockerImageReference != "" {`
`184`	`185`	`instance.Status.ImageReference = image.DockerImageReference`
`185`	`186`	`}`
`186`		`- } else if err != nil && !kerrors.IsNotFound(err) && !kerrors.IsForbidden(err) {`
	`187`	`+ } else if err != nil && !kerrors.IsNotFound(err) && !kerrors.IsForbidden(err) && !strings.Contains(isTagName, "/") {`
`187`	`188`	`return r.ManageError(err, common.StatusConditionTypeReconciled, instance)`
`188`	`189`	`}`
`189`	`190`	`}`