From 0184e967e58fb7663219eacffeae894031794af0 Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Thu, 20 Jun 2019 16:27:32 -0400 Subject: [PATCH] Log the raised message when DNS check_zone_overlap fails The check can fail for a lot of other reasons than there is overlap so the error should be logged. This causes confusion when --auto-reverse is requested and some lookup fails causing the reverse to not be created. Signed-off-by: Rob Crittenden Reviewed-By: Francois Cami --- ipaserver/install/bindinstance.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ipaserver/install/bindinstance.py b/ipaserver/install/bindinstance.py index 3126356c9..6dc0288a4 100644 --- a/ipaserver/install/bindinstance.py +++ b/ipaserver/install/bindinstance.py @@ -327,9 +327,10 @@ def get_auto_reverse_zones(ip_addresses, allow_zone_overlap=False): if not allow_zone_overlap: try: dnsutil.check_zone_overlap(default_reverse) - except ValueError: + except ValueError as e: logger.info("Reverse zone %s for IP address %s already exists", default_reverse, ip) + logger.debug('%s', e) continue auto_zones.append((ip, default_reverse)) return auto_zones