feat(terraform: azure-jenkinsinfra-controller) allow DNS to use dis…

…tinct provider or to be undefined (#135) * fix(terraform: azure-jenkinsinfra-controller) do not create DNS records when the DNS Zone name is unspecified Signed-off-by: Damien Duportal <damien.duportal@gmail.com> * feat(terraform: azure-jenkinsinfra-controller) allow different providers Signed-off-by: Damien Duportal <damien.duportal@gmail.com> --------- Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
jenkins-infra · Jan 18, 2024 · c618f89 · c618f89
1 parent d82a181
commit c618f89
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/terraform/modules/azure-jenkinsinfra-controller/main.tf b/terraform/modules/azure-jenkinsinfra-controller/main.tf
@@ -39,7 +39,8 @@ resource "azurerm_management_lock" "controller_publicip" {
   notes      = "Locked because this is a sensitive resource that should not be removed"
 }
 resource "azurerm_dns_a_record" "controller" {
-  count               = var.is_public ? 1 : 0
+  count               = var.is_public && var.dns_zone_name != "" ? 1 : 0
+  provider            = azurerm.dns
   name                = trimsuffix(trimsuffix(local.controller_fqdn, var.dns_zone), ".")
   zone_name           = var.dns_zone_name
   resource_group_name = var.dns_resourcegroup_name
@@ -48,7 +49,8 @@ resource "azurerm_dns_a_record" "controller" {
   tags                = var.default_tags
 }
 resource "azurerm_dns_a_record" "private_controller" {
-  count               = var.is_public ? 1 : 0
+  count               = var.is_public && var.dns_zone_name != "" ? 1 : 0
+  provider            = azurerm.dns
   name                = "private.${azurerm_dns_a_record.controller[0].name}"
   zone_name           = var.dns_zone_name
   resource_group_name = var.dns_resourcegroup_name

diff --git a/terraform/modules/azure-jenkinsinfra-controller/providers.tf b/terraform/modules/azure-jenkinsinfra-controller/providers.tf
@@ -1,7 +1,8 @@
 terraform {
   required_providers {
     azurerm = {
-      source = "hashicorp/azurerm"
+      source                = "hashicorp/azurerm"
+      configuration_aliases = [azurerm.dns]
     }
     azuread = {
       source = "hashicorp/azuread"