-
Notifications
You must be signed in to change notification settings - Fork 169
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
update namespace strategies with new CHE_INFRA_KUBERNETES_NAMESPACE v… #855
Conversation
…ariable and <workspaceid> placeholder Signed-off-by: Michal Vala <mvala@redhat.com>
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.
@rkratky great, thanks. I've merged that to the branch. |
Please don NOT merge this! It's documenting changes from PR eclipse-che/che#14828 so we need to merge that one first, then the docs. |
@sparkoo, sure, please, let me know you're ready. |
@rkratky the code PR was merged. We're good to go with this one. |
@sparkoo, thanks. Merged. |
…ariable and placeholder
Signed-off-by: Michal Vala mvala@redhat.com
What does this PR do?
Updates workspace namespaces strategies docs. In this PR eclipse-che/che#14828, the
che.infra.kubernetes.namespace
andche.infra.openshift.project
will became obsolete and are replaced withche.infra.kubernetes.namespace.default
. This PR updates the docs.Also There is new
<workspaceid>
placeholder. This effectively replaces the default empty value resolving to workspace id.What issues does this PR fix or reference?
eclipse-che/che#14585