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

incubator/crd.yaml tlsSupport:true by default breaks server:start in minikube #16409

Closed
gattytto opened this issue Mar 20, 2020 · 6 comments
Closed
Labels
area/install Issues related to installation, including offline/air gap and initial setup severity/P1 Has a major impact to usage or development of the system. status/duplicate Issue identified as a duplicate of another issue

Comments

@gattytto
Copy link

changes introduced in latest versions of chectl the file:
/usr/local/lib/chectl/templates/che-operator/crds/org_v1_che_cr.yaml
set the following default:

tlsSupport: true
selfSignedCert: false

while the comment above tlsSupport instates:

# TLS mode for Che. Make sure you either have public cert, or set selfSignedCert to true

after issuing "chectl server:start -m -p minikube" to start a new che, che-server breaks on startup because of:

Caused` by: javax.net.ssl.SSLHandshakeException: java.security.cert.CertificateException: No subject alternative DNS name matching keycloak-che.10.0.2.33.nip.io found.
@gattytto gattytto added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 20, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Mar 20, 2020
@mmorhun
Copy link
Contributor

mmorhun commented Mar 20, 2020

@gattytto could you please provide step by step flow how you tried to deploy Che. Also full logs attached increases chances of problem understanding.

@mmorhun mmorhun added the status/info-needed More information is needed before the issue can move into the “analyzing” state for engineering. label Mar 20, 2020
@tolusha tolusha added status/duplicate Issue identified as a duplicate of another issue and removed status/duplicate Issue identified as a duplicate of another issue status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Mar 20, 2020
@tolusha
Copy link
Contributor

tolusha commented Mar 20, 2020

Duplicates
#16396 ?

@tolusha tolusha closed this as completed Mar 20, 2020
@tolusha tolusha added status/duplicate Issue identified as a duplicate of another issue severity/P1 Has a major impact to usage or development of the system. area/install Issues related to installation, including offline/air gap and initial setup and removed status/info-needed More information is needed before the issue can move into the “analyzing” state for engineering. labels Mar 20, 2020
@mmorhun mmorhun removed the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 20, 2020
@gattytto
Copy link
Author

@gattytto could you please provide step by step flow how you tried to deploy Che. Also full logs attached increases chances of problem understanding.

just emptied out /usr/local/lib/chectl and deleted /usr/local/bin/chectl
went to incubator/chectl git page and ran the :next "bash" install method pointed out there.

then it downloads all the yamls again and puts the chectl binary in place, then run it using:
"chectl server:start -m -p minikube".

as mentioned, those 2 settings come by default in the crd.yaml file downloaded by the script. Switching tlsSupport back to "false" allowed me to deploy che normally.

@tolusha
Copy link
Contributor

tolusha commented Mar 23, 2020

@gattytto
We are working on providing clearer steps how to use tls

@gattytto
Copy link
Author

@tolusha tlsSupport is still enabled by default along with selfSigned set to false, meaning chectl+che-operator+kube will try tls without expecting self signed (I'm guessing self signed will still be used by default) so this will break?

@tolusha
Copy link
Contributor

tolusha commented May 25, 2020

@gattytto
You are right.
We are working on deprecating selfSignedCert or setting it to true at least .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/install Issues related to installation, including offline/air gap and initial setup severity/P1 Has a major impact to usage or development of the system. status/duplicate Issue identified as a duplicate of another issue
Projects
None yet
Development

No branches or pull requests

4 participants