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

che #6500 - Removing optional 'che-ws-' prefix during tag creation on OpenShift #6604

Merged
merged 1 commit into from
Oct 9, 2017

Conversation

ibuziuk
Copy link
Member

@ibuziuk ibuziuk commented Oct 5, 2017

Signed-off-by: Ilya Buziuk ibuziuk@redhat.com

What does this PR do?

Removing optional 'che-ws-' prefix during tag creation OpenShift / k8s names / labels max length is 63 chars and removing optional prefix (7 chars) would allow creating multi-container workspaces on minishift

What issues does this PR fix or reference?

hot fix for #6500
docker image for testing ibuziuk/che-server:chews

Removing optional 'che-ws-' prefix during tag creation OpenShift

Release Notes

N / A

Docs PR

N / A

…creation

Signed-off-by: Ilya Buziuk <ibuziuk@redhat.com>
@ibuziuk ibuziuk requested a review from l0rd as a code owner October 5, 2017 21:50
@ibuziuk ibuziuk requested a review from amisevsk October 5, 2017 21:50
@codenvy-ci
Copy link

Can one of the admins verify this patch?

@codenvy-ci
Copy link

Can one of the admins verify this patch?

@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/bug Outline of a bug - must adhere to the bug report template. labels Oct 5, 2017
@benoitf
Copy link
Contributor

benoitf commented Oct 6, 2017

ci-build

@codenvy-ci
Copy link

@ibuziuk ibuziuk merged commit c147f8b into eclipse-che:master Oct 9, 2017
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Oct 9, 2017
@benoitf benoitf added this to the 5.19.0 milestone Oct 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants