Skip to content

Commit bc1fb20

Browse files
committed
Make acme environment boolean (staging or prod)
1 parent 342de5d commit bc1fb20

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

kubernetes/terraform/modules/kubernetes/cert_manager.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
locals {
2-
cert_manager_namespace = "kube-system"
3-
cert_manager_version = "0.14.2"
2+
cert_manager_namespace = "kube-system"
3+
cert_manager_version = "0.14.2"
44
cluster_issuer_name = "clusterissuer-letsencrypt-${var.cert_manager_acme_environment}"
5-
cert_manager_acme_server = var.cert_manager_acme_environment == "production" ? "https://acme-v02.api.letsencrypt.org/directory" : "https://acme-${var.cert_manager_acme_environment}-v02.api.letsencrypt.org/directory"
5+
cert_manager_acme_server = var.cert_manager_use_production_acme_environment ? "https://acme-v02.api.letsencrypt.org/directory" : "https://acme-staging-v02.api.letsencrypt.org/directory"
66
}
77

88
# Reference an existing route53 zone

kubernetes/terraform/modules/kubernetes/variables.tf

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,10 @@ variable "external_dns_owner_id" {
1818
description = "Unique id of the TXT record that external-dns will use to store state (can just be a uuid)"
1919
}
2020

21-
variable "cert_manager_acme_environment" {
22-
default = "production"
23-
description = "ACME (LetsEncrypt) Environment - only production creates valid certificates but it has lower rate limits"
21+
variable "cert_manager_use_production_acme_environment" {
22+
description = "ACME (LetsEncrypt) Environment - only production creates valid certificates but it has lower rate limits than staging"
23+
type = bool
24+
default = true
2425
}
2526

2627
variable "cert_manager_acme_registration_email" {

0 commit comments

Comments
 (0)