func run --build should prompt for registry details on first run

[zedalaye]

In this page : https://knative.dev/docs/getting-started/build-run-deploy-func/ at [kn ] func run step :

Expected Behavior

The function is built/deployed/run as expected

Actual Behavior

Experiments/knative/hello via 🐹 v1.25.4
❯ kn func run --build
Building function image
Error: registry required
Error: exit status 1

Experiments/knative/hello via 🐹 v1.25.4
❯ kn func run
Building function image
Error: registry required
Error: exit status 1

Steps to Reproduce the Problem

2. I downloaded the dependencies (kind, kn, kn-quickstart, kn-func)
3. Cluster has been created using kn quickstart kind
4. Create the sample function and run kn func run

I also tried to rebuild the quickstart cluster with the --registry option and the results are the same:

❯ docker ps
CONTAINER ID   IMAGE                  COMMAND                  CREATED          STATUS          PORTS                                              NAMES
25db8df61e3a   registry:2             "/entrypoint.sh /etc…"   16 minutes ago   Up 16 minutes   0.0.0.0:5001->5000/tcp                             kind-registry
7b6b9c99f1ae   kindest/node:v1.32.0   "/usr/local/bin/entr…"   17 minutes ago   Up 17 minutes   127.0.0.1:41717->6443/tcp, 0.0.0.0:80->31080/tcp   knative-control-plane

Additional Info

Additional context

I'm on Windows with WSL2, docker is installed using Docker Desktop for windows. $
Everything works as expected in WSL2

Install information:

• Platform (GKE, IKS, AKS, etc.): ?
• Knative Version: 1.20.0

[dprotaso]

FWIW - it seems like you need to run func build first and then func run.

Though I see a point that since you've passed func run --build that it should do the registry prompt in that flow

[dprotaso]

Moving this to the func repo

[dprotaso]

/assign @gauron99 @matejvasek for triage

[knative-prow]

@dprotaso: GitHub didn't allow me to assign the following users: for, triage.

Note that only knative members with read permissions, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

Details

In response to this:

/assign @gauron99 @matejvasek for triage

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[dprotaso]

I'll subsequently update the docs

[dprotaso]

Actually the docs say to use the --registry under the func tab but not the kn func tab. I'll pull that out into a common place.

[dprotaso]

Note this issue remains that on first run --build if the registry isn't set it should prompt

[dprotaso]

docs PR is here: knative/docs#6510

this updates the kn func flow to suggest the --registry flag if you haven't built before