Skip to content
This repository was archived by the owner on Nov 28, 2022. It is now read-only.

CodeReady Workspace quick guide #613

Merged

Conversation

j-c-berger
Copy link
Contributor

For issue #2407-Support for CodeReady Workspaces

@AndyRWatson, tagging you since you opened the issue. @micgibso, tagging you for your awareness as well.

I'm not sure if these instructions for developing a microservice are applicable for CodeReady Workspaces (I took these instructions from our other quick guides), but I thought it at least gives us a starting point.

Note, I took a lot of this content for Knowledge Center's Installing CodeReady Workspaces. I worked on this content a couple months ago.

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
…m more Codewind specific

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier, new commit is up. If it looks good, I can get SVT to run through and test it.

Copy link
Contributor

@johnmcollier johnmcollier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some review comments. Rest looks good.

I'll also get some screenshots for the CRW install step, as I think they will help things.

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier, thanks, I addressed your comments and made a new commit. I reordered some content, per your comments. I'll include the screenshots once you get them to me.

@naveenkaratekid, I think it's ready for you to test it since we're only waiting on screenshots now. Thanks!

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier and @naveenkaratekid, new commit with images included is up, thanks!

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier and @naveenkaratekid, new commit is up which includes the new instructions and yaml file John provided.

Should we include instructions or a link to setting up an OpenShift cluster? The first step for Installing CodeReady Workspaces requires users to log into their OpenShift clusters, and I wonder if some users won't have an OpenShift cluster already.

I have the same thought when we say in the Prerequisites to apply the Codewind cluster role with oc apply -f. I thought I could provide more context to users. Are they applying the oc apply -f command in the command line?

I think my images are showing up as broken links because they don't exist in the images folder. They only exist on this PR. So let me know if you have any questions about the order of the screenshots.

Thanks!

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier, please see new commit and let me know if the instructions are accurate and what you intended. Same goes for placement of screenshots.

@naveenkaratekid, if you could test it today with these new instructions, that'd be great!

@johnmcollier
Copy link
Contributor

@j-c-berger I think we just need to make an OpenShift cluster a prerequisite for the quick guide.

And yes, the oc apply commands are done on the command line.

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier, thank for the review! New commit is up, reflecting the changes.

For the prerequisites, I know we want to add installing an OpenShift cluster. I know we talk about clusters some in our installing Eclipse Che doc. I didn't know if you wanted to adapt some of the instructions there.

@naveenkaratekid
Copy link
Contributor

@johnmcollier Step 6 in the guide is not very clear

CodeReady Workspaces now installs and you can access the CodeReady Workspaces URL and Red Hat SSO Admin Console URL

I assumed that I would need to click on this link codeready-default.apps.cretic.os.fyre.ibm.com , but when I did that, it took me to this page.

image

The guide didn't state how we are supposed to log in

@johnmcollier
Copy link
Contributor

@naveenkaratekid You'll need to click register to create an account.

@j-c-berger Can you add that to the post-install instructions? Basically before we log in to CodeReady Workspaces to create a codewind workspace

@AndyRWatson
Copy link

AndyRWatson commented Jun 10, 2020

I just tried following the guide, using the updates. When running prerequisite steps, I setup a OCP on IBM cloud, then I assumed I should login from the command line (need to state this step) and the performed

oc apply -f

and got the response

error: error reading ".": open .Trash: operation not permitted

I assume more is required to apply a codewind cluster or there maybe a missing step(s) here ?

@AndyRWatson
Copy link

For the step


1. Log in to your OpenShift cluster.

is this still command line or webui ? I assume this is the Openshift web console ?

@AndyRWatson
Copy link

AndyRWatson commented Jun 10, 2020

    If you did not set up an auth provider, set openShiftoAuth to false.
    To enable HTTPS, set tlsSupport to true.
    If you use a self-signed certificate, set selfSignedCert to true.

For a guide I think we should let people know how to do this, I think some people may drop out here unless we are prerequing users must be skilled at OpenShift setup and configuration ?

It would be useful for the guide to also explain why these are relevant and why you have/may consider making these changes

@AndyRWatson
Copy link

AndyRWatson commented Jun 10, 2020

when installing the codeready operator, is the recomendation step 4 or step 5 ? I could update my tls in the editor in 4 and then overwrite that change via c+p in step 5 ?

As a guide i'd like to understand what I am changing and why those pieces matter to me as a user of codewind.

@AndyRWatson
Copy link


    To manually install Codewind with CodeReady Workspaces, save this yaml to a disk and run oc apply -f <yaml>.
    If you did not set up an auth provider, set openShiftoAuth to false.
    To enable HTTPS, set tlsSupport to true.
    If you use a self-signed certificate, set selfSignedCert to true.

how do I save the yaml to a disk ? And if we're now going back to the command line window maybe we should clarify the switch For the next steps do we set auth/https/certs via the command line, in the yaml or by going back to the openshift console ?

@AndyRWatson
Copy link

6. CodeReady Workspaces now installs and you can access the CodeReady Workspaces URL and Red Hat SSO Admin Console URL:

How do I discover the URLs ?

@johnmcollier
Copy link
Contributor

@AndyRWatson For the configuration:

    If you did not set up an auth provider, set openShiftoAuth to false.
    To enable HTTPS, set tlsSupport to true.
    If you use a self-signed certificate, set selfSignedCert to true.

I'd rather we almost leave them out, as it's outside of the scope of this quick guide. The only one applicable would be set selfSignedCert to true if on IBM Public Cloud.

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@AndyRWatson and @johnmcollier, great review and feedback! I put the new commit up. Let me know what the two of you think. Hopefully, Andy, these new instructions make the development process easier and clearer.

…de solely focuses on installing CodeReady Workspaces and setting up a Codewind workspace

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@johnmcollier, please see new commit, and let me know if the content looks look. Per the issues with Appsody on Kube, the guide focuses solely on installing CodeReady Workspaces and Codewind and setting up a Codewind workspace.

@micgibso, as you'll see, I made the metadata changes. However, this is what I see when I render the guide locally:
Screen Shot 2020-06-11 at 12 06 42 PM
I believe it's because my branch was made off master before James made these changes to guides' metadata. I think once this PR is merged, the objectives and icon will display properly because the branch will then be in sync with the newest master. I wanted to confirm with you though before anything is merged. Thanks!

Copy link
Contributor

@johnmcollier johnmcollier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from one nit

… need for OS clusters.

Signed-off-by: Jacob Berger <jacob.berger@ibm.com>
@j-c-berger
Copy link
Contributor Author

@micgibso, PR is ready to merge. @johnmcollier reviewed it and had one change to the prerequisite. See the new commit for those changes.

Also, please note my earlier comment about the metadata and this branch being made off an old form of master. I don't think that will be an issue after merging this PR but thought you should know. Thanks!

Copy link
Contributor

@micgibso micgibso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@micgibso micgibso merged commit f0a5c5e into eclipse-archived:master Jun 12, 2020
jcockbain pushed a commit to jcockbain/codewind-docs that referenced this pull request Jun 15, 2020
…_CRWquickguide

CodeReady Workspace quick guide
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants