URL: https://github.com/freeipa/freeipa/pull/857
Author: Tiboris
 Title: #857: baseldap.py: Removes dnss-server configuration from ldap
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/857/head:pr857
git checkout pr857
From 4898dfd1391d6ef4cae484e232177c7506438574 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tibor=20Dudl=C3=A1k?= <tdud...@redhat.com>
Date: Tue, 6 Jun 2017 15:13:26 +0200
Subject: [PATCH] baseldap.py: Removes dns-server configuration from ldap

After invocation of the ipa server-del <hostname>
command there was still record in ldap if DNS
was installed on the <hostname> server.

Fixes: https://pagure.io/freeipa/issue/6572
---
 ipaserver/plugins/server.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/ipaserver/plugins/server.py b/ipaserver/plugins/server.py
index 7aea619b34..eae14fc06f 100644
--- a/ipaserver/plugins/server.py
+++ b/ipaserver/plugins/server.py
@@ -676,6 +676,16 @@ def _cleanup_server_dns_records(self, hostname, **options):
                 **options):
             return
 
+        dn = self.api.Object.dnsserver.get_dn(hostname)
+        assert isinstance(dn, DN)
+
+        ldap = self.obj.backend
+
+        try:
+            ldap.delete_entry(dn)
+        except errors.NotFound:
+            pass
+
         try:
             bindinstance.remove_master_dns_records(
                 hostname, self.api.env.realm)
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org

Reply via email to