s4 dns: use the internal DNS server per default

2 files changed, 15 insertions, 15 deletions

diff --git a/source4/scripting/python/samba/netcmd/domain.py b/source4/scripting/python/samba/netcmd/domain.py
index d35ee577ea..119c19757f 100644
--- a/source4/scripting/python/samba/netcmd/domain.py
+++ b/source4/scripting/python/samba/netcmd/domain.py
@@ -174,11 +174,11 @@ class cmd_domain_provision(Command):
                 help="choose machine password (otherwise random)"),
          Option("--dns-backend", type="choice", metavar="NAMESERVER-BACKEND",
                 choices=["SAMBA_INTERNAL", "BIND9_FLATFILE", "BIND9_DLZ", "NONE"],
-                help="The DNS server backend. SAMBA_INTERNAL is the builtin name server, " \
+                help="The DNS server backend. SAMBA_INTERNAL is the builtin name server (default), " \
                      "BIND9_FLATFILE uses bind9 text database to store zone information, " \
-                     "BIND9_DLZ uses samba4 AD to store zone information (default), " \
+                     "BIND9_DLZ uses samba4 AD to store zone information, " \
                      "NONE skips the DNS setup entirely (not recommended)",
-                default="BIND9_DLZ"),
+                default="SAMBA_INTERNAL"),
          Option("--dnspass", type="string", metavar="PASSWORD",
                 help="choose dns password (otherwise random)"),
          Option("--ldapadminpass", type="string", metavar="PASSWORD",
@@ -297,7 +297,7 @@ class cmd_domain_provision(Command):
 
             server_role = ask("Server Role (dc, member, standalone)", "dc")
 
-            dns_backend = ask("DNS backend (SAMBA_INTERNAL, BIND9_FLATFILE, BIND9_DLZ, NONE)", "BIND9_DLZ")
+            dns_backend = ask("DNS backend (SAMBA_INTERNAL, BIND9_FLATFILE, BIND9_DLZ, NONE)", "SAMBA_INTERNAL")
             if dns_backend in (None, ''):
                 raise CommandError("No DNS backend set!")
 
@@ -412,10 +412,10 @@ class cmd_domain_dcpromo(Command):
                action="store_true"),
         Option("--dns-backend", type="choice", metavar="NAMESERVER-BACKEND",
                choices=["SAMBA_INTERNAL", "BIND9_DLZ", "NONE"],
-               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server, " \
-                   "BIND9_DLZ uses samba4 AD to store zone information (default), " \
+               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server (default), " \
+                   "BIND9_DLZ uses samba4 AD to store zone information, " \
                    "NONE skips the DNS setup entirely (this DC will not be a DNS server)",
-               default="BIND9_DLZ")
+               default="SAMBA_INTERNAL")
        ]
 
     takes_args = ["domain", "role?"]
@@ -479,10 +479,10 @@ class cmd_domain_join(Command):
                action="store_true"),
         Option("--dns-backend", type="choice", metavar="NAMESERVER-BACKEND",
                choices=["SAMBA_INTERNAL", "BIND9_DLZ", "NONE"],
-               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server, " \
-                   "BIND9_DLZ uses samba4 AD to store zone information (default), " \
+               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server (default), " \
+                   "BIND9_DLZ uses samba4 AD to store zone information, " \
                    "NONE skips the DNS setup entirely (this DC will not be a DNS server)",
-               default="BIND9_DLZ")
+               default="SAMBA_INTERNAL")
        ]
 
     takes_args = ["domain", "role?"]
@@ -1177,11 +1177,11 @@ class cmd_domain_classicupgrade(Command):
                action="store_true"),
         Option("--dns-backend", type="choice", metavar="NAMESERVER-BACKEND",
                choices=["SAMBA_INTERNAL", "BIND9_FLATFILE", "BIND9_DLZ", "NONE"],
-               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server, " \
+               help="The DNS server backend. SAMBA_INTERNAL is the builtin name server (default), " \
                    "BIND9_FLATFILE uses bind9 text database to store zone information, " \
-                   "BIND9_DLZ uses samba4 AD to store zone information (default), " \
+                   "BIND9_DLZ uses samba4 AD to store zone information, " \
                    "NONE skips the DNS setup entirely (this DC will not be a DNS server)",
-               default="BIND9_DLZ")
+               default="SAMBA_INTERNAL")
     ]
 
     takes_args = ["smbconf"]
diff --git a/source4/scripting/python/samba/provision/__init__.py b/source4/scripting/python/samba/provision/__init__.py
index 862a0c7790..8b0737c226 100644
--- a/source4/scripting/python/samba/provision/__init__.py
+++ b/source4/scripting/python/samba/provision/__init__.py
@@ -1816,8 +1816,8 @@ def provision(logger, session_info, credentials, smbconf=None,
     if use_rfc2307:
         global_param["idmap_ldb:use rfc2307"] = ["yes"]
 
-    if dns_backend == "SAMBA_INTERNAL":
-        server_services.append("+dns")
+    if dns_backend != "SAMBA_INTERNAL":
+        server_services.append("-dns")
 
     if use_ntvfs:
         server_services.append("+smb")