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

docs: Adding 'Integrating with Kubernetes' user guide #2250

Merged
merged 1 commit into from
Mar 31, 2022

Conversation

ibuziuk
Copy link
Member

@ibuziuk ibuziuk commented Mar 28, 2022

What does this pull request change

image

Adding 'Integrating with Kubernetes' user guide

  • Adding automatic token injection section
  • Updating image in the navigation from OpenShift web console to Che
  • Adding a section for navigating from dashboard to OpenShift web console

The PR does not contain optional Managing workspaces via kubectl (optional) section that might be provided later on as a separate PR

What issues does this pull request fix or reference

eclipse-che/che#21300

Specify the version of the product this pull request applies to

for master branch

cherry-picking to the main branch is expected to be handled by the docs team

Pull request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@github-actions
Copy link

Click here to review and test in web IDE: Contribute

[id="navigating-openshift-web-console-from-{prod-id-short_{context}"]
= Navigating OpenShift web console from {prod-short}

This section describes how to access OpenShif web console from {prod-short}.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This section describes how to access OpenShif web console from {prod-short}.
This section describes how to access OpenShift web console from {prod-short}.

@@ -1,8 +1,9 @@
:navtitle: Integrating with {orch-name}
:navtitle: {orch-name} integrations
Copy link
Contributor

@themr0c themr0c Mar 28, 2022

Choose a reason for hiding this comment

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

Keep the gerund in the title. It is part of our requirements.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok, this was how the issue was titled see - eclipse-che/che#21300

:keywords: overview, integrating with {orch-name}
:page-aliases:

[id="integrating-with-kubernetes_{context}"]
= Integrating with {orch-name}
= {orch-name} integrations
Copy link
Contributor

Choose a reason for hiding this comment

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

Keep the gerund in the title. It is part of our requirements.

Copy link
Member Author

Choose a reason for hiding this comment

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

ok, this was how the issue was titled see - eclipse-che/che#21300

Copy link
Member Author

Choose a reason for hiding this comment

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

Integrating with Kubernetes vs Kubernetes integrations
those titles have different meanings and the current one is ambiguous imo

Copy link
Member Author

Choose a reason for hiding this comment

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

reverted back to Integrating with Kubernetes as per request

@ibuziuk ibuziuk force-pushed the che-21300 branch 6 times, most recently from ea3fd6a to dc6f567 Compare March 29, 2022 07:33
@ibuziuk ibuziuk force-pushed the che-21300 branch 2 times, most recently from 7f54516 to 714773a Compare March 29, 2022 09:49
@ibuziuk ibuziuk marked this pull request as ready for review March 29, 2022 09:50
@ibuziuk ibuziuk requested a review from rkratky as a code owner March 29, 2022 09:50
@ibuziuk ibuziuk requested a review from l0rd March 29, 2022 10:09
+
image::integration/token-injection.png[Token Injection in IDE,link="../_images/integration/token-injection.png"]

WARNING: The automatic token injection functionality currently works only on the OpenShift infrastructure.
Copy link
Contributor

Choose a reason for hiding this comment

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

"function" is easier than "functionality" and provides the same content.

Copy link
Member Author

Choose a reason for hiding this comment

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

I decided to drop functionality wording since it is redundant

[id="automatic-token-injection_{context}"]
= Automatic {orch-name} token injection

This section describes how to use the {orch-name} user token that is automatically injected into workspace containers.
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be cool if this sentence could provide information on what you can do with the user token.

Copy link
Member Author

Choose a reason for hiding this comment

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

fair point, will rework it

[id="navigating-openshift-web-console-from-{prod-id-short_{context}"]
= Navigating OpenShift web console from {prod-short}

This section describes how to access OpenShift web console from {prod-short}.
Copy link
Contributor

Choose a reason for hiding this comment

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

The OS web console.

Copy link
Member Author

Choose a reason for hiding this comment

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

we do not use such contractions I believe - the official naming is OpenShift web console

Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't mean to contract, I meant to add an article.

[id="navigating-openshift-web-console-from-{prod-id-short_{context}"]
= Navigating OpenShift web console from {prod-short}

This section describes how to access OpenShift web console from {prod-short}.
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't mean to contract, I meant to add an article.

Signed-off-by: Ilya Buziuk <ibuziuk@redhat.com>
@ibuziuk ibuziuk merged commit 81118eb into eclipse-che:master Mar 31, 2022
@themr0c themr0c added this to the 7.44 milestone Apr 1, 2022
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

Successfully merging this pull request may close these issues.

5 participants