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

update namespace strategies with new CHE_INFRA_KUBERNETES_NAMESPACE v… #855

Merged
merged 5 commits into from
Oct 24, 2019

Conversation

sparkoo
Copy link
Member

@sparkoo sparkoo commented Oct 18, 2019

…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 and che.infra.openshift.project will became obsolete and are replaced with che.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

…ariable and <workspaceid> placeholder

Signed-off-by: Michal Vala <mvala@redhat.com>
Copy link

@rkratky rkratky left a comment

Choose a reason for hiding this comment

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

@sparkoo, I fixed a few thinks but then realized that I needed to edit the sidebar YAML to include the content in the navig. menu. So, I submitted the rest as a PR against your feature branch at sparkoo#2

@sparkoo
Copy link
Member Author

sparkoo commented Oct 19, 2019

@rkratky great, thanks. I've merged that to the branch.

@sparkoo
Copy link
Member Author

sparkoo commented Oct 19, 2019

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.

@rkratky
Copy link

rkratky commented Oct 20, 2019

@sparkoo, sure, please, let me know you're ready.

@sparkoo
Copy link
Member Author

sparkoo commented Oct 24, 2019

@rkratky the code PR was merged. We're good to go with this one.

@rkratky rkratky merged commit 569dec8 into eclipse-che:master Oct 24, 2019
@rkratky
Copy link

rkratky commented Oct 24, 2019

@sparkoo, thanks. Merged.

@sparkoo sparkoo deleted the workspaceidPlaceholder branch October 24, 2019 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants