Hello,

DNS: Fix realm domains integration with DNS zone add.

Realmdomains integration into DNS commands pre-dates split of DNS forward zones
and DNS master zones into two distinct commands.

There was an forgotten condition in dnszone_add command which caused omission
of DNS master zones with non-empty forwarders from realmdomain list.

https://fedorahosted.org/freeipa/ticket/5980

-- 
Petr^2 Spacek
From 5e5c04ec3bd8b4c57afdcbe6dfefab25786b06b9 Mon Sep 17 00:00:00 2001
From: Petr Spacek <pspa...@redhat.com>
Date: Mon, 20 Jun 2016 14:29:08 +0200
Subject: [PATCH] DNS: Fix realm domains integration with DNS zone add.

Realmdomains integration into DNS commands pre-dates split of DNS forward zones
and DNS master zones into two distinct commands.

There was an forgotten condition in dnszone_add command which caused omission
of DNS master zones with non-empty forwarders from realmdomain list.

https://fedorahosted.org/freeipa/ticket/5980
---
 ipaserver/plugins/dns.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/ipaserver/plugins/dns.py b/ipaserver/plugins/dns.py
index 094f8eb5703124a658a1becc63070b5f72fe82d8..06425affc0b9306b3f774877b713dce0c53c9b10 100644
--- a/ipaserver/plugins/dns.py
+++ b/ipaserver/plugins/dns.py
@@ -2743,11 +2743,10 @@ class dnszone_add(DNSZoneBase_add):
         assert isinstance(dn, DN)
 
         # Add entry to realmdomains
-        # except for our own domain, forward zones, reverse zones and root zone
+        # except for our own domain, reverse zones and root zone
         zone = keys[0]
 
         if (zone != DNSName(api.env.domain).make_absolute() and
-                not options.get('idnsforwarders') and
                 not zone.is_reverse() and
                 zone != DNSName.root):
             try:
-- 
2.5.5

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to