From 56e15bd5948fb820775d181ef1eb5d18fe65ece8 Mon Sep 17 00:00:00 2001
From: Patrick de Ruiter <pderuiter@bsdserver.nl>
Date: Thu, 20 Nov 2025 10:21:25 +0100
Subject: [PATCH] fix: Only set DNS if dns_servers list is not empty

Changed dns configuration to use conditional assignment to avoid
setting empty DNS list which Docker might ignore.

Changes:
- dns = length(var.dns_servers) > 0 ? var.dns_servers : null

This ensures that:
- If dns_servers is empty, dns is set to null (Docker uses defaults)
- If dns_servers has values, they are properly applied to container
---
 main.tf | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/main.tf b/main.tf
index bc7dd67..026c624 100644
--- a/main.tf
+++ b/main.tf
@@ -50,7 +50,8 @@ resource "docker_container" "renovate" {
   }
 
   # DNS configuration for internal hostname resolution
-  dns = var.dns_servers
+  # Only set if dns_servers is not empty
+  dns = length(var.dns_servers) > 0 ? var.dns_servers : null
 
   # Volumes
   volumes {