Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add (and test) Native S2I Guide #6

Open
vorburger opened this issue Apr 5, 2019 · 4 comments
Open

Add (and test) Native S2I Guide #6

vorburger opened this issue Apr 5, 2019 · 4 comments

Comments

@vorburger
Copy link
Collaborator

Try https://github.com/quarkusio/quarkus/blob/master/docs/src/main/asciidoc/openshift-s2i-guide.adoc (which should appear on https://quarkus.io/guides/openshift-s2i-guide very soon..) on the Workshop Demo Cluster.

Be particularly mindful of future participants not running with a privileged admin account; the patch to increase the limit may be a a problem, and need a solution in a set up such as this.

Ultimate goal of this issue would be to create a Guide for this Native S2I.

@vorburger
Copy link
Collaborator Author

future participants not running with a privileged admin account

Just FTR: Following the (my..) Quarkus Doc steps when logging into the cluster with a token obtained from the /console/ UI to which one logged in with cluster admin account, it works just fine.

@vorburger
Copy link
Collaborator Author

FTR: It seems to work, even for a "normal user" (via oc login and typing Username and Password, no token).

BTW: The guide (above) is actually wrong, I fixed it in quarkusio/quarkus#1898.

@tqvarnst
Copy link
Owner

tqvarnst commented Apr 5, 2019

So it seems that the problem is that we cannot patch the build config until we have created it and oc new-app will both create the build config and start a build. One solution may be to use oc cancel-build bc/quarkus-quickstart-native after the oc new-app and then patch the bc and then use oc start-build bc/quarkus-quickstart-native.

vorburger added a commit to vorburger/quarkus that referenced this issue Apr 9, 2019
This is step 1; when merged, a follow-up applying the same to gradle-it is planned.

related to tqvarnst/quarkus-workshop#6
@vorburger
Copy link
Collaborator Author

FTR: @tqvarnst meanwhile documented this with an update to https://quarkus.io/guides/openshift-s2i-guide - we can probably close this issue now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants