(docs): Update Knative Quickstart guide

[kameshsampath]

Update dKnative Quickstart guide for quarkusio/quarkus-quickstarts#112

Fixes: #1698

[kameshsampath]

FYI: @gsmet @cescoffier - sorry for opening the issue in other site repo. moved it here and closed the other one.

[starksm64]

I tried running through the guide. I went to https://kubernetes.io/docs/tasks/tools/install-minikube/ and followed the steps to install minikube and dependcies on an osx box using brew. On the first command minikube crashed:

Scotts-iMacPro:docs starksm$ eval $(minikube docker-env)

💣  Error getting host: Machine does not exist for api.Exists(minikube)

😿  Sorry that minikube crashed. If this was unexpected, we would love to hear from you:
👉  https://github.com/kubernetes/minikube/issues/new

Scotts-iMacPro:docs starksm$ minikube version
minikube version: v1.0.0

[starksm64]

I guess I needed to have started minikube before the eval $(minikube docker-env) command.

[kameshsampath]

oh yeah ! you are right

[starksm64]

It says to run a maven command, but I don't see one in the merged getting-started-knative. If I do the step after docker-env eval, I don't have a Dockerfile so it is failing:

Scotts-iMacPro:docs starksm$ docker build -t starksm64/getting-started-knative . 
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /Users/starksm/Dev/JBoss/Protean/starksm64-quarkus/docs/Dockerfile: no such file or directory

so either I'm supposed to using the solution or there is a maven command to setup a new project that is missing.

[kameshsampath]

@starksm64 - for all the commands in this to work well, you might need the PR quarkusio/quarkus-quickstarts#117 merged as that uses the latest quarkus maven images

[starksm64]

Ok, but the getting-started-knative-guide.html seems to be missing something. Here is what I see in it:

there is a reference to a maven command, but I don't see one.

[kameshsampath]

@starksm64 you dont need the maven command explicitly, the build takes care of it. Also you cant build from macbook directly for the old get started. the new PR which I mentioned ( needs to be merged) can allow you to do the build directly from mac

[starksm64]

Ok, I did try the new quickstart PR branch and left a comment on that about not being able to get past the first docker build -t {username}/getting-started-knative . step as the image build failed with a 137 status code.

[rsvoboda]

Hi. This PR and knative guide are mentioning docker.io. Q is focused on quay.io.
Please update the PR and examples to use quay.io registry.

git grep docker.io
docs/src/main/asciidoc/getting-started-knative-guide.adoc:| https://index.docker.io/v1/
docs/src/main/asciidoc/getting-started-knative-guide.adoc:| docker.io/demo/quarkus-knative-quickstart
docs/src/main/asciidoc/getting-started-knative-guide.adoc:    -Dapp.container.image='docker.io/demo/getting-started-knative' \
extensions/kubernetes/deployment/src/main/java/io/quarkus/kubernetes/deployment/DockerConfig.java:    public String registry = "docker.io";

[kameshsampath]

@rsvoboda - i would also be happy to use quay.io but then for the knative i have to use docker or gcr repos which are v2_2 enabled. Quay is not set default v2_2 for all the registered users. Once I see the quay release happening we can move to use quay.io

[dmlloyd]

Any updates on this? @starksm64 ?

[starksm64]

I'll go through it again today to see the current status. Last time I tried it was failing with remote repository issues.

[cescoffier]

Any update on this? @kameshsampath @starksm64

[kameshsampath]

@cescoffier am waiting on @starksm64 for any update on the issues he was facing. But this gives me a chance to update it for 0.16.0 and push again. WDYT?

[cescoffier]

@kameshsampath yes, please update.

[kameshsampath]

@cescoffier an effective close to this will be merging the PRS:

• (fix): MAVEN_HOME dir and README quarkus-images#6
• (fix): simplify and improve knative quickstart quarkus-quickstarts#117

I have updated and rebased all the PRs for 0.16.0 and graal rc16

[emmanuelbernard]

@cescoffier @kameshsampath so 0.18 is about to land with GraalVM 19.0.2 support. Maybe a good time to refresh that PR in that occasion right after the release.

[dmlloyd]

Now that 0.18 (and 0.19) have been released, do we want to update this further?

/cc @cescoffier @kameshsampath

[cescoffier]

@kameshsampath can you have a look?

[kameshsampath]

@cescoffier - I will mid of next week

[kameshsampath]

@cescoffier - I have updated the doc, dont know why there is conflict I tried to rebase to the latest quickstarts.

I have also updated quarkusio/quarkus-quickstarts#117 the sources. Please merge them together.

Let me know if some action is required.

[cescoffier]

Closing due to inactivity.

[kameshsampath]

rather than inactivity there was no response