]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
samba-tool: fix replication after dns partition fsmo role transfer
authorStefan Metzmacher <metze@samba.org>
Tue, 28 May 2019 12:53:09 +0000 (14:53 +0200)
committerKarolin Seeger <kseeger@samba.org>
Thu, 8 Aug 2019 07:32:21 +0000 (07:32 +0000)
The new role owner need to replicate from the old role owner.

Before we told the old role owner to replicate from itself.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13973

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
(cherry picked from commit 4793f8ed584a4e6d8a26b06b691ec636e77d8f2a)

python/samba/netcmd/fsmo.py
selftest/knownfail.d/fsmo [deleted file]

index 91de5dad563a0323d638dd8db950286551945389..c276f5fced7e179b0200f3b1123fd9f7637883db 100644 (file)
@@ -64,6 +64,8 @@ def transfer_dns_role(outf, sambaopts, credopts, role, samdb):
         forest_dn = samba.dn_from_dns_name(samdb.forest_dns_name())
         role_object = "CN=Infrastructure,DC=ForestDnsZones," + forest_dn
 
+    new_host_dns_name = samdb.host_dns_name()
+
     res = samdb.search(role_object,
                        attrs=["fSMORoleOwner"],
                        scope=ldb.SCOPE_BASE,
@@ -128,7 +130,7 @@ def transfer_dns_role(outf, sambaopts, credopts, role, samdb):
             raise CommandError("Failed to add role '%s': %s" % (role, msg))
 
         try:
-            connection = samba.drs_utils.drsuapi_connect(samdb.host_dns_name(),
+            connection = samba.drs_utils.drsuapi_connect(new_host_dns_name,
                                                          lp, creds)
         except samba.drs_utils.drsException as e:
             raise CommandError("Drsuapi Connect failed", e)
diff --git a/selftest/knownfail.d/fsmo b/selftest/knownfail.d/fsmo
deleted file mode 100644 (file)
index da665d2..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-^samba4.drs.fsmo.python.*.fsmo.DrsFsmoTestCase.test_DomainDnsZonesMasterTransfer
-^samba4.drs.fsmo.python.*.fsmo.DrsFsmoTestCase.test_ForestDnsZonesMasterTransfer