Skip to content

Commit 043f4c2

Browse files
fix(ingress/legacy): drop cluster_issuer_override fallback (not supported on legacy)
Legacy only sets the DNS-01 issuer (gts-production) when private; otherwise null. No user-facing cluster_issuer_override input, so don't read it from spec. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 485ba7b commit 043f4c2

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

  • modules/ingress
    • nginx_gateway_fabric_legacy_aws/1.0
    • nginx_gateway_fabric_legacy_azure/1.0
    • nginx_gateway_fabric_legacy_gcp/1.0

modules/ingress/nginx_gateway_fabric_legacy_aws/1.0/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ locals {
5555
# Private LB → HTTP-01 can't validate an internal NLB; issue certs via the
5656
# gts-production DNS-01 ClusterIssuer instead. No effect in acm_mode (NLB
5757
# terminates TLS with the ACM cert, no cert-manager involvement).
58-
cluster_issuer_override = lookup(var.instance.spec, "private", false) && !local.acm_mode ? "gts-production" : lookup(var.instance.spec, "cluster_issuer_override", null)
58+
cluster_issuer_override = lookup(var.instance.spec, "private", false) && !local.acm_mode ? "gts-production" : null
5959

6060
modified_instance = merge(var.instance, {
6161
spec = merge(var.instance.spec, {

modules/ingress/nginx_gateway_fabric_legacy_azure/1.0/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ locals {
2323

2424
# Private LB → HTTP-01 can't validate an internal LB; issue certs via the
2525
# gts-production DNS-01 ClusterIssuer instead.
26-
cluster_issuer_override = lookup(var.instance.spec, "private", false) ? "gts-production" : lookup(var.instance.spec, "cluster_issuer_override", null)
26+
cluster_issuer_override = lookup(var.instance.spec, "private", false) ? "gts-production" : null
2727

2828
modified_instance = merge(var.instance, {
2929
spec = merge(var.instance.spec, {

modules/ingress/nginx_gateway_fabric_legacy_gcp/1.0/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ locals {
2525

2626
# Private LB → HTTP-01 can't validate an internal LB; issue certs via the
2727
# gts-production DNS-01 ClusterIssuer instead.
28-
cluster_issuer_override = lookup(var.instance.spec, "private", false) ? "gts-production" : lookup(var.instance.spec, "cluster_issuer_override", null)
28+
cluster_issuer_override = lookup(var.instance.spec, "private", false) ? "gts-production" : null
2929

3030
modified_instance = merge(var.instance, {
3131
spec = merge(var.instance.spec, {

0 commit comments

Comments
 (0)