-
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
Adding missing information #1344
Adding missing information #1344
Conversation
Signed-off-by: Michal Maléř <mmaler@redhat.com>
@@ -15,4 +15,7 @@ The {prod-short} server, also known as *wsmaster*, is the central service of the | |||
|
|||
| Container image | |||
| `eclipse/che-server` | |||
|
|||
| Environment variables | |||
| link:https://www.eclipse.org/che/docs/che-7/advanced-configuration-options-for-the-che-server-component/[Advanced configuration options for the Che server component] |
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.
This is not downstream-friendly. Can we use an attribute here?
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.
Ok, will do
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.
From what I know from Serhii Leshchenko, currently on PTO, the Controller is not part of CRW yet, so this is pure upstream ATM. I will add the attribute for this link in the comments, so it could be easily used in the future when a CRW counterpart will exist.
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.
Even though we have it in the downstream docs already:
https://access.redhat.com/documentation/en-us/red_hat_codeready_workspaces/2.1/html/administration_guide/codeready-workspaces-architectural-elements_crw#codeready-workspaces-workspace-controller_crw
Signed-off-by: Michal Maléř <mmaler@redhat.com>
Signed-off-by: Michal Maléř <mmaler@redhat.com>
* Adding missing information Signed-off-by: Michal Maléř <mmaler@redhat.com> * reaction to a need for a link attribute Signed-off-by: Michal Maléř <mmaler@redhat.com> * adding a link attribute for a synergy with dowstream Signed-off-by: Michal Maléř <mmaler@redhat.com>
* Adding missing information Signed-off-by: Michal Maléř <mmaler@redhat.com> * reaction to a need for a link attribute Signed-off-by: Michal Maléř <mmaler@redhat.com> * adding a link attribute for a synergy with dowstream Signed-off-by: Michal Maléř <mmaler@redhat.com>
Merged and Cherry-picked |
Signed-off-by: Michal Maléř mmaler@redhat.com
Adding info and some missing values for the included tables