squakez commented on code in PR #5109:
URL: https://github.com/apache/camel-k/pull/5109#discussion_r1467685433
##########
pkg/cmd/operator/operator.go:
##########
@@ -295,7 +295,8 @@ func getOperatorImage(ctx context.Context, c ctrl.Reader)
(string, error) {
ns := platform.GetOperatorNamespace()
name := platform.GetOperatorPodName()
if ns == "" || name == "" {
- return "", nil
+ // We are most likely running out of cluster. Let's take a
chance and use the default value
Review Comment:
That's the point. The local execution should not allow to run a pipeline but
only execute the code that can find "locally". It's because the local operator
is running a given binary (typically built from source), whilst the builder
would be run in the Pod with potentially a different software (as it is
packaged in a container from somewhere else). I think it's wise to keep this
design as it to avoid consistency problems.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
