-
Notifications
You must be signed in to change notification settings - Fork 95
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
Namespace "cert-manager" already exists #72
Labels
Comments
hferentschik
added a commit
to hferentschik/terraform-google-jx
that referenced
this issue
Apr 28, 2020
We need to always create cert-manager and exdns namespaces and service accounts since the current boot pipeline will always apply the underlying Helm charts. If a namespace or service account does not exist before running `jx boot` it will be created as part of `helm apply`. Once it is created by Helm, one cannot go back to Terraform and enable DNS there, since Terraform will not be able to create the already existing resources. fixers jenkins-x#72
hferentschik
added a commit
that referenced
this issue
May 4, 2020
We need to always create cert-manager and exdns namespaces and service accounts since the current boot pipeline will always apply the underlying Helm charts. If a namespace or service account does not exist before running `jx boot` it will be created as part of `helm apply`. Once it is created by Helm, one cannot go back to Terraform and enable DNS there, since Terraform will not be able to create the already existing resources. fixers #72
🎉 This issue has been resolved in version 1.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When the tf module is run without
parent_domain
configured tf does not create thecert-manager
namespace. However if one decides to configureparent_domain
and reapply the module afterwards it fails with the following error since the namespace already exists:As a fix the
cert-manager
namespace could always be created by tf.The text was updated successfully, but these errors were encountered: