Skip to content

Conversation

@Dariocent
Copy link
Contributor

@Dariocent Dariocent commented Dec 23, 2025

Note

Introduces optional TLS termination via existing ACM certificates and updates generated Helm configs accordingly.

  • Adds acm_certificate_arn input (with ARN validation); trims/propagates to locals and template rendering in main.tf
  • templates/helm-values-bootstrap.tpl.yaml: when acm_certificate_arn is set, configures ingress-nginx Service for NLB with SSL annotations and HTTP backend
  • templates/helm-values.tpl.yaml: conditionally removes cert-manager clusterIssuer and tls sections when ACM is used; otherwise preserves Let’s Encrypt defaults
  • Docs: updates CHANGELOG.md (v0.13.0) and README Inputs table to document the new option

Written by Cursor Bugbot for commit 2f7f0f2. This will update automatically on new commits. Configure here.

@Dariocent Dariocent requested a review from diegofiori December 23, 2025 13:59
@diegofiori
Copy link
Contributor

@cursor review

@Dariocent Dariocent merged commit eccbb9e into main Dec 23, 2025
2 checks passed
@Dariocent Dariocent deleted the add-acm-certificates branch December 23, 2025 14:52
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

Successfully merging this pull request may close these issues.

3 participants