Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Migrate to cert-manager 0.10.0 #126

Merged
merged 5 commits into from
Sep 6, 2019
Merged

Conversation

alexellis
Copy link
Member

Signed-off-by: Alex Ellis (OpenFaaS Ltd) alexellis2@gmail.com

Description

The format for Issuers, the solver config, and the annotations
for Ingress all changed. This patch outputs the files in the
new format.

Fixes: 122

How Has This Been Tested?

Unit tests were added for the new format, but full e2e testing
needs to be done.

Users will need to upgrade as soon as possible to avoid known issues with the current cert-manager version being blocked by LetsEncrypt.

Checklist:

I have:

  • checked my changes follow the style of the existing code / OpenFaaS repos
  • updated the documentation and/or roadmap in README.md
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

The format for Issuers, the solver config, and the annotations
for Ingress all changed. This patch outputs the files in the
new format.

Unit tests were added for the new format, but full e2e testing
needs to be done.

Fixes: 122

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
There was an issue that caused two certificates to fight with
each other - one created by Ingress shim and one by the
bootstrap as a Certificate object, for the same domain.

This patch moves away from using ingress-shim.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
There was a mis-match in the cert-manager docs, so these fields
are actually no-longer required and stopped the cert from being
created.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis merged commit 027a4a9 into master Sep 6, 2019
@alexellis
Copy link
Member Author

@matipan @LucasRoesler @stefanprodan @ewilde @doowb @martindekov PTAL at the changeset.

@alexellis alexellis deleted the ofltd/cert-manager-0.9.0 branch September 6, 2019 12:46
@alexellis alexellis changed the title Migrate to cert-manager 0.9.0 Migrate to cert-manager 0.10.0 Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant