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

Remove letsencrypt-cluster-issuer from chart #250

Open
batpad opened this issue Jul 21, 2022 · 1 comment
Open

Remove letsencrypt-cluster-issuer from chart #250

batpad opened this issue Jul 21, 2022 · 1 comment

Comments

@batpad
Copy link
Member

batpad commented Jul 21, 2022

Thanks for the catch from @dakotabenjamin in #249

The letsencrypt cluster issuer is defined here: https://github.com/developmentseed/osm-seed/blob/develop/osm-seed/templates/letsencrypt-issuer.yaml and this works fine if there is only one instance of osm-seed on the cluster. If there are multiple instances, this fail to install because the cluster issuer already exists.

I think the best solution is probably to remove the cluster-issuer from the osm-seed helm chart and add docs on setting that up as a separate helm chart that you can setup once on the cluster.

I was really hoping for people to not have to fiddle with too many things and be able to install everything needed with a single install of this helm chart - maybe we can add it as an option, but osm-seed should definitely not by default create the cluster issuer if there are multiple instances on a cluster. The best solution for now I think is to take it out, and install the Cluster Issuer separately on the cluster.

cc @Rub21 @geohacker

@batpad
Copy link
Member Author

batpad commented Jul 21, 2022

Turns out it's fine to have multiple issuers on the cluster (thanks @yuvipanda ), so we will go with the approach in #249 .

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

No branches or pull requests

1 participant