Skip to content

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