[SCM] Samba Shared Repository - branch master updated

Stefan Metzmacher metze at samba.org
Tue Oct 16 02:44:02 MDT 2012


The branch, master has been updated
       via  1861213 selftest/knownfail: add samba3.rpc.lsa.privileges.lsa.Privileges
       via  266b4c5 Revert "provision: Always create DNS user."
      from  2c3a808 s4-dns: Fix the comments about ignoring zones in internal server

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 1861213d147e0d96fd637813c5badb4908ec14d1
Author: Stefan Metzmacher <metze at samba.org>
Date:   Tue Oct 16 08:34:35 2012 +0200

    selftest/knownfail: add samba3.rpc.lsa.privileges.lsa.Privileges
    
    This failed more than 20 times in the last few weeks, e.g.
    https://git.samba.org/autobuild.flakey/2012-10-16-0629/samba3.stdout
    https://git.samba.org/autobuild.flakey/2012-10-16-0829/samba3.stdout
    
    	[530/717 in 14m32s] samba3.rpc.lsa.privileges(s3dc)
    	Using seed 1350368974
    
    	Testing OpenPolicy
    
    	Testing OpenPolicy2
    
    	Testing CreateAccount
    
    	Testing Delete
    
    	Testing DeleteObject
    
    	Testing EnumAccounts
    
    	Testing LookupSids
    
    	Testing LookupNames with 7 names
    	LookupName of sharesec_user was unmapped
    	LookupName of Everyone failed to return a result
    	UNEXPECTED(failure): samba3.rpc.lsa.privileges.lsa.Privileges(s3dc)
    	REASON: _StringException: _StringException: ../source4/torture/rpc/lsa.c:319: r.out.result was STATUS_SOME_UNMAPPED, expected NT_STATUS_OK: LookupNames failed
    
    	FAILED (1 failures, 0 errors and 0 unexpected successes in 0 testsuites)
    
    metze
    
    Autobuild-User(master): Stefan Metzmacher <metze at samba.org>
    Autobuild-Date(master): Tue Oct 16 10:43:02 CEST 2012 on sn-devel-104

commit 266b4c596346095f71a651e0a0231256c7409b0f
Author: Stefan Metzmacher <metze at samba.org>
Date:   Tue Oct 16 08:30:17 2012 +0200

    Revert "provision: Always create DNS user."
    
    This reverts commit c2d14747d608d406de6410556807d467cd0b85ef.
    
    samba_upgradedns handles creates/removed the dns acount.
    
    See
    https://lists.samba.org/archive/samba-technical/2012-October/thread.html#87578
    
    metze

-----------------------------------------------------------------------

Summary of changes:
 selftest/flapping                                  |    1 +
 .../scripting/python/samba/provision/__init__.py   |   31 +++++++++++--------
 2 files changed, 19 insertions(+), 13 deletions(-)


Changeset truncated at 500 lines:

diff --git a/selftest/flapping b/selftest/flapping
index f0b1528..afeae65 100644
--- a/selftest/flapping
+++ b/selftest/flapping
@@ -15,6 +15,7 @@
 ^samba3.rpc.spoolss.printer.*addprinterex.print_test # another intermittent failure
 ^samba3.rap.printing # fails sometimes on sn-devel
 ^samba3.rpc.spoolss.printer.*addprinter.print_test # fails on some hosts due to timing issues ?
+^samba3.rpc.lsa.privileges.lsa.Privileges\(s3dc\) # fails sometimes on sn-devel
 ^samba3.smb2.lock.*.rw-exclusive # another intermittent failure
 ^samba4.blackbox.gentest # is flakey due to timing
 ^samba3.smb2.acls.INHERITANCE\(plugin_s4_dc\)  # Seems to flap - succeeds on sn-devel, fails on Fedora 16
diff --git a/source4/scripting/python/samba/provision/__init__.py b/source4/scripting/python/samba/provision/__init__.py
index d9ba90c..9966192 100644
--- a/source4/scripting/python/samba/provision/__init__.py
+++ b/source4/scripting/python/samba/provision/__init__.py
@@ -969,7 +969,7 @@ def setup_samdb_rootdse(samdb, names):
 
 
 def setup_self_join(samdb, admin_session_info, names, fill, machinepass,
-        dnspass, domainsid, next_rid, invocationid,
+        dns_backend, dnspass, domainsid, next_rid, invocationid,
         policyguid, policyguid_dc,
         domainControllerFunctionality, ntdsguid=None, dc_rid=None):
     """Join a host to its own domain."""
@@ -1048,14 +1048,17 @@ def setup_self_join(samdb, admin_session_info, names, fill, machinepass,
 
     samdb.set_session_info(admin_session_info)
 
-    setup_add_ldif(samdb, setup_path("provision_dns_add_samba.ldif"), {
-          "DNSDOMAIN": names.dnsdomain,
-          "DOMAINDN": names.domaindn,
-          "DNSPASS_B64": b64encode(dnspass.encode('utf-16-le')),
-          "HOSTNAME" : names.hostname,
-          "DNSNAME" : '%s.%s' % (
-              names.netbiosname.lower(), names.dnsdomain.lower())
-          })
+    if dns_backend != "SAMBA_INTERNAL":
+        # This is Samba4 specific and should be replaced by the correct
+        # DNS AD-style setup
+        setup_add_ldif(samdb, setup_path("provision_dns_add_samba.ldif"), {
+              "DNSDOMAIN": names.dnsdomain,
+              "DOMAINDN": names.domaindn,
+              "DNSPASS_B64": b64encode(dnspass.encode('utf-16-le')),
+              "HOSTNAME" : names.hostname,
+              "DNSNAME" : '%s.%s' % (
+                  names.netbiosname.lower(), names.dnsdomain.lower())
+              })
 
 
 def getpolicypath(sysvolpath, dnsdomain, guid):
@@ -1143,7 +1146,7 @@ def setup_samdb(path, session_info, provision_backend, lp, names,
 
 
 def fill_samdb(samdb, lp, names, logger, domainsid, domainguid, policyguid,
-        policyguid_dc, fill, adminpass, krbtgtpass, machinepass,
+        policyguid_dc, fill, adminpass, krbtgtpass, machinepass, dns_backend,
         dnspass, invocationid, ntdsguid, serverrole, am_rodc=False,
         dom_for_fun_level=None, schema=None, next_rid=None, dc_rid=None):
 
@@ -1331,6 +1334,7 @@ def fill_samdb(samdb, lp, names, logger, domainsid, domainguid, policyguid,
             logger.info("Setting up self join")
             setup_self_join(samdb, admin_session_info, names=names, fill=fill,
                 invocationid=invocationid,
+                dns_backend=dns_backend,
                 dnspass=dnspass,
                 machinepass=machinepass,
                 domainsid=domainsid,
@@ -1603,8 +1607,8 @@ def provision_fill(samdb, secrets_ldb, logger, names, paths,
                    next_rid=1000, dc_rid=None, adminpass=None, krbtgtpass=None,
                    domainguid=None, policyguid=None, policyguid_dc=None,
                    invocationid=None, machinepass=None, ntdsguid=None,
-                   dns_backend=None,
-                   dnspass=None, serverrole=None, dom_for_fun_level=None,
+                   dns_backend=None, dnspass=None,
+                   serverrole=None, dom_for_fun_level=None,
                    am_rodc=False, lp=None, use_ntvfs=False, skip_sysvolacl=False):
     # create/adapt the group policy GUIDs
     # Default GUID for default policy are described at
@@ -1632,7 +1636,8 @@ def provision_fill(samdb, secrets_ldb, logger, names, paths,
                    policyguid=policyguid, policyguid_dc=policyguid_dc,
                    fill=samdb_fill, adminpass=adminpass, krbtgtpass=krbtgtpass,
                    invocationid=invocationid, machinepass=machinepass,
-                   dnspass=dnspass, ntdsguid=ntdsguid, serverrole=serverrole,
+                   dns_backend=dns_backend, dnspass=dnspass,
+                   ntdsguid=ntdsguid, serverrole=serverrole,
                    dom_for_fun_level=dom_for_fun_level, am_rodc=am_rodc,
                    next_rid=next_rid, dc_rid=dc_rid)
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list