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

Add existing certificate to route after redeployment of a service #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add existing certificate to route after redeployment of a service #27

wants to merge 1 commit into from

Conversation

macmacs
Copy link

@macmacs macmacs commented Sep 16, 2017

This way, an existing certificate is also added to a route if the corresponding service is redeployed.

This way, an existing certificate is also added to a route if the corresponding service is redeployed.
@macmacs macmacs changed the title Adds existing certificate to route after redeployment of a service Add existing certificate to route after redeployment of a service Sep 16, 2017
@ibotty
Copy link
Owner

ibotty commented Sep 17, 2017

Do you mean: route redeployment?

I prefer recording above whether the route has a certificate and checking below.

What do you think?

@macmacs
Copy link
Author

macmacs commented Sep 17, 2017

Yes, route deployment (e.g. when setting up a new application).

I noticed, that if a cert for a certain app/route already exists in openshift-letsencrypt storage, the route will NOT be changed to https when I redeploy this app/route.

This pullrequest would take care of this case.

@ibotty
Copy link
Owner

ibotty commented Sep 17, 2017 via email

@macmacs
Copy link
Author

macmacs commented Sep 25, 2017

Hmm, I don't see a problem with the api server, because existing, valid certificates will be used.
And if there's no valid certificate existing, the route has not beed deployed for x weeks.

@ibotty
Copy link
Owner

ibotty commented Sep 25, 2017

It does not matter whether the certificate is different, it will nonetheless create a new route revision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants