-
Notifications
You must be signed in to change notification settings - Fork 164
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 section to configure labels #1610
Conversation
@themr0c |
modules/installation-guide/partials/proc_configuring-labels-for-routes-ingresses.adoc
Outdated
Show resolved
Hide resolved
modules/installation-guide/partials/proc_configuring-labels-for-routes-ingresses.adoc
Outdated
Show resolved
Hide resolved
modules/installation-guide/partials/proc_configuring-labels-for-routes-ingresses.adoc
Outdated
Show resolved
Hide resolved
|
||
.Procedure | ||
|
||
. To configure labels for routes update the Custom Resource with the following commands: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I understood, Kubernetes <=> Ingresses and OpenShift <=> Routes. Is that right?
Then we should have 2 procedures to distinguish the 2 distinct scenarios:
- Configuring labels for OpenShift Routes using the CLI
- Configuring labels for Kubernetes Ingresses using the CLI
Additionally, it would be awesome to have one procedure describing the usage of the WebUI, probably like:
- Configuring labels for OpenShift Routes using the OpenShift Web Console
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Divided into 2 procedures.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The procedure describing the usage of the WebUI make sense only for OpenShift and it will be much longer then using CLI.
I don't think it worth describing it.
modules/installation-guide/partials/proc_configuring-labels-for-routes-ingresses.adoc
Outdated
Show resolved
Hide resolved
@tolusha, Nice improvements! Very welcome! I added some comments. |
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
…r-routes-ingresses.adoc Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
…r-routes-ingresses.adoc Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
…r-routes-ingresses.adoc Co-authored-by: Fabrice Flore-Thébault <ffloreth@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
0d54a63
to
fb5b2b9
Compare
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@themr0c |
Signed-off-by: Anatolii Bazko abazko@redhat.com
What does this PR do?
Add section to configure labels
What issues does this PR fix or reference?
https://issues.redhat.com/browse/CRW-1034
PR Checklist
As the author of this Pull Request I made sure that:
vale
has been run successfully against the PR branchShould be merged after eclipse-che/che-operator#449