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

fix(helm): fix deploying che with helm installer when self-signed-cert is specified #248

Merged
merged 2 commits into from
Aug 12, 2019

Conversation

sleshchenko
Copy link
Contributor

@sleshchenko sleshchenko commented Aug 9, 2019

What does this PR do?

Fixes deploying Che with helm installer when self-signed-cert is specified

What issues does this PR fix or reference?

It's related to eclipse-che/che#14035
It depends on eclipse-che/che#14176

@sleshchenko
Copy link
Contributor Author

@l0rd @slemeur Maybe you would like set 7.0.0 milestone to this and che related PR otherwise, it won't be possible to deploy Che with chectl (helm as installer) or bare helm if self-signed-cert is configured.

…t is specified

Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
@sleshchenko sleshchenko changed the title fix (helm): fix deploying che with helm installer when self-signed-cert is specified fix(helm): fix deploying che with helm installer when self-signed-cert is specified Aug 9, 2019
@sleshchenko sleshchenko added the bug Something isn't working label Aug 9, 2019
@l0rd
Copy link
Collaborator

l0rd commented Aug 12, 2019

I am +1 to include this in milestone 7.0.0

@l0rd
Copy link
Collaborator

l0rd commented Aug 12, 2019

cc @slemeur @benoitf @rhopp

@benoitf
Copy link
Collaborator

benoitf commented Aug 12, 2019

@sleshchenko could you please fix compilation issue ?

…ned-cert is specified

Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
@benoitf benoitf added this to the 7.0.0 milestone Aug 12, 2019
@sleshchenko sleshchenko merged commit b607a04 into che-incubator:master Aug 12, 2019
@sleshchenko sleshchenko deleted the helmSelfSignedCert branch August 12, 2019 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants