From 7b7213f1eedfbb68217fa760edaa3392dab5f507 Mon Sep 17 00:00:00 2001 From: Jelmer Vernooij Date: Sun, 13 Jun 2010 17:43:25 +0200 Subject: s4-provision: Use logger in provision backends. --- source4/scripting/python/samba/provision.py | 43 +++++++++++----------- source4/scripting/python/samba/provisionbackend.py | 37 +++++++++---------- 2 files changed, 38 insertions(+), 42 deletions(-) (limited to 'source4/scripting/python') diff --git a/source4/scripting/python/samba/provision.py b/source4/scripting/python/samba/provision.py index 876ca67dcf..dcf84624bf 100644 --- a/source4/scripting/python/samba/provision.py +++ b/source4/scripting/python/samba/provision.py @@ -29,11 +29,10 @@ from base64 import b64encode import os import pwd import grp +import logging import time import uuid import socket -import param -import registry import urllib import shutil @@ -56,6 +55,8 @@ from samba.provisionbackend import ( LDBBackend, OpenLDAPBackend, ) +import samba.param +import samba.registry from samba.schema import Schema from samba.samdb import SamDB @@ -443,7 +444,7 @@ def make_smbconf(smbconf, setup_path, hostname, domain, realm, serverrole, assert realm is not None realm = realm.upper() - default_lp = param.LoadParm() + default_lp = samba.param.LoadParm() #Load non-existant file if os.path.exists(smbconf): default_lp.load(smbconf) @@ -714,10 +715,10 @@ def setup_registry(path, setup_path, session_info, lp): :param credentials: Credentials :param lp: Loadparm context """ - reg = registry.Registry() - hive = registry.open_ldb(path, session_info=session_info, + reg = samba.registry.Registry() + hive = samba.registry.open_ldb(path, session_info=session_info, lp_ctx=lp) - reg.mount_hive(hive, registry.HKEY_LOCAL_MACHINE) + reg.mount_hive(hive, samba.registry.HKEY_LOCAL_MACHINE) provision_reg = setup_path("provision.reg") assert os.path.exists(provision_reg) reg.diff_apply(provision_reg) @@ -1175,7 +1176,7 @@ def provision(setup_dir, logger, session_info, if targetdir is not None: smbconf = os.path.join(targetdir, "etc", "smb.conf") elif smbconf is None: - smbconf = param.default_path() + smbconf = samba.param.default_path() if not os.path.exists(os.path.dirname(smbconf)): os.makedirs(os.path.dirname(smbconf)) @@ -1193,7 +1194,7 @@ def provision(setup_dir, logger, session_info, make_smbconf(smbconf, setup_path, hostname, domain, realm, serverrole, targetdir, sid_generator, useeadb) - lp = param.LoadParm() + lp = samba.param.LoadParm() lp.load(smbconf) names = guess_names(lp=lp, hostname=hostname, domain=domain, @@ -1247,20 +1248,20 @@ def provision(setup_dir, logger, session_info, paths=paths, setup_path=setup_path, lp=lp, credentials=credentials, names=names, - message=logger.info) + logger=logger) elif backend_type == "existing": provision_backend = ExistingBackend(backend_type, paths=paths, setup_path=setup_path, lp=lp, credentials=credentials, names=names, - message=logger.info, + logger=logger, ldapi_url=ldapi_url) elif backend_type == "fedora-ds": provision_backend = FDSBackend(backend_type, paths=paths, setup_path=setup_path, lp=lp, credentials=credentials, names=names, - message=logger.info, + logger=logger, domainsid=domainsid, schema=schema, hostname=hostname, @@ -1275,7 +1276,7 @@ def provision(setup_dir, logger, session_info, paths=paths, setup_path=setup_path, lp=lp, credentials=credentials, names=names, - message=logger.info, + logger=logger, domainsid=domainsid, schema=schema, hostname=hostname, @@ -1380,7 +1381,7 @@ def provision(setup_dir, logger, session_info, # Only make a zone file on the first DC, it should be replicated # with DNS replication - create_zone_file(lp, logger.info, paths, targetdir, setup_path, + create_zone_file(lp, logger, paths, targetdir, setup_path, dnsdomain=names.dnsdomain, hostip=hostip, hostip6=hostip6, hostname=names.hostname, realm=names.realm, domainguid=domainguid, ntdsguid=names.ntdsguid) @@ -1402,7 +1403,7 @@ def provision(setup_dir, logger, session_info, "generated at %s", paths.krb5conf) if serverrole == "domain controller": - create_dns_update_list(lp, logger.info, paths, setup_path) + create_dns_update_list(lp, logger, paths, setup_path) provision_backend.post_setup() provision_backend.shutdown() @@ -1472,13 +1473,10 @@ def provision_become_dc(setup_dir=None, ldap_backend=None, ldap_backend_type=None, sitename=None, debuglevel=1): - def message(text): - """print a message if quiet is not set.""" - print text - + logger = logging.getLogger("provision") samba.set_debug_level(debuglevel) - return provision(setup_dir, message, system_session(), None, + return provision(setup_dir, logger, system_session(), None, smbconf=smbconf, targetdir=targetdir, samdb_fill=FILL_DRS, realm=realm, rootdn=rootdn, domaindn=domaindn, schemadn=schemadn, configdn=configdn, serverdn=serverdn, domain=domain, @@ -1497,7 +1495,7 @@ def create_phpldapadmin_config(path, setup_path, ldapi_uri): {"S4_LDAPI_URI": ldapi_uri}) -def create_zone_file(lp, message, paths, targetdir, setup_path, dnsdomain, +def create_zone_file(lp, logger, paths, targetdir, setup_path, dnsdomain, hostip, hostip6, hostname, realm, domainguid, ntdsguid): """Write out a DNS zone file, from the info in the current database. @@ -1572,19 +1570,20 @@ def create_zone_file(lp, message, paths, targetdir, setup_path, dnsdomain, os.chmod(dns_dir, 0775) os.chmod(paths.dns, 0664) except OSError: - message("Failed to chown %s to bind gid %u" % (dns_dir, paths.bind_gid)) + logger.error("Failed to chown %s to bind gid %u" % (dns_dir, paths.bind_gid)) if targetdir is None: os.system(rndc + " unfreeze " + lp.get("realm")) -def create_dns_update_list(lp, message, paths, setup_path): +def create_dns_update_list(lp, logger, paths, setup_path): """Write out a dns_update_list file""" # note that we use no variable substitution on this file # the substitution is done at runtime by samba_dnsupdate setup_file(setup_path("dns_update_list"), paths.dns_update_list, None) setup_file(setup_path("spn_update_list"), paths.spn_update_list, None) + def create_named_conf(paths, setup_path, realm, dnsdomain, private_dir): """Write out a file containing zone statements suitable for inclusion in a diff --git a/source4/scripting/python/samba/provisionbackend.py b/source4/scripting/python/samba/provisionbackend.py index b8abc28eaa..a40a4b347a 100644 --- a/source4/scripting/python/samba/provisionbackend.py +++ b/source4/scripting/python/samba/provisionbackend.py @@ -55,14 +55,14 @@ class SlapdAlreadyRunning(Exception): class ProvisionBackend(object): def __init__(self, backend_type, paths=None, setup_path=None, lp=None, - credentials=None, names=None, message=None): + credentials=None, names=None, logger=None): """Provision a backend for samba4""" self.paths = paths self.setup_path = setup_path self.lp = lp self.credentials = credentials self.names = names - self.message = message + self.logger = logger self.type = backend_type @@ -108,11 +108,11 @@ class LDBBackend(ProvisionBackend): class ExistingBackend(ProvisionBackend): def __init__(self, backend_type, paths=None, setup_path=None, lp=None, - credentials=None, names=None, message=None, ldapi_uri=None): + credentials=None, names=None, logger=None, ldapi_uri=None): super(ExistingBackend, self).__init__(backend_type=backend_type, paths=paths, setup_path=setup_path, lp=lp, - credentials=credentials, names=names, message=message) + credentials=credentials, names=names, logger=logger) self.ldapi_uri = ldapi_uri @@ -134,13 +134,13 @@ class ExistingBackend(ProvisionBackend): class LDAPBackend(ProvisionBackend): def __init__(self, backend_type, paths=None, setup_path=None, lp=None, - credentials=None, names=None, message=None, domainsid=None, + credentials=None, names=None, logger=None, domainsid=None, schema=None, hostname=None, ldapadminpass=None, slapd_path=None, ldap_backend_extra_port=None, ldap_dryrun_mode=False): super(LDAPBackend, self).__init__(backend_type=backend_type, paths=paths, setup_path=setup_path, lp=lp, - credentials=credentials, names=names, message=message) + credentials=credentials, names=names, logger=logger) self.domainsid = domainsid self.schema = schema @@ -179,7 +179,7 @@ class LDAPBackend(ProvisionBackend): else: p = f.read() f.close() - self.message("Check for slapd Process with PID: " + str(p) + " and terminate it manually.") + self.logger.info("Check for slapd Process with PID: " + str(p) + " and terminate it manually.") raise SlapdAlreadyRunning(self.ldapi_uri) except LdbError: # XXX: We should never be catching all Ldb errors @@ -190,9 +190,7 @@ class LDAPBackend(ProvisionBackend): if self.slapd_path is None: raise ProvisioningError("Warning: LDAP-Backend must be setup with path to slapd, e.g. --slapd-path=\"/usr/local/libexec/slapd\"!") if not os.path.exists(self.slapd_path): - self.message (self.slapd_path) - raise ProvisioningError("Warning: Given Path to slapd does not exist!") - + self.logger.warning("Path (%s) to slapd does not exist!", self.slapd_path) if not os.path.isdir(self.ldapdir): os.makedirs(self.ldapdir, 0700) @@ -255,10 +253,10 @@ class LDAPBackend(ProvisionBackend): count = count + 1 if count > 15: - self.message("Could not connect to slapd started with: %s" % "\'" + "\' \'".join(self.slapd_provision_command) + "\'") + self.logger.error("Could not connect to slapd started with: %s" % "\'" + "\' \'".join(self.slapd_provision_command) + "\'") raise ProvisioningError("slapd never accepted a connection within 15 seconds of starting") - self.message("Could not start slapd with: %s" % "\'" + "\' \'".join(self.slapd_provision_command) + "\'") + self.logger.error("Could not start slapd with: %s" % "\'" + "\' \'".join(self.slapd_provision_command) + "\'") raise ProvisioningError("slapd died before we could make a connection to it") def shutdown(self): @@ -279,13 +277,13 @@ class LDAPBackend(ProvisionBackend): class OpenLDAPBackend(LDAPBackend): def __init__(self, backend_type, paths=None, setup_path=None, lp=None, - credentials=None, names=None, message=None, domainsid=None, + credentials=None, names=None, logger=None, domainsid=None, schema=None, hostname=None, ldapadminpass=None, slapd_path=None, ldap_backend_extra_port=None, ldap_dryrun_mode=False, ol_mmr_urls=None, nosync=False): super(OpenLDAPBackend, self).__init__( backend_type=backend_type, paths=paths, setup_path=setup_path, lp=lp, - credentials=credentials, names=names, message=message, + credentials=credentials, names=names, logger=logger, domainsid=domainsid, schema=schema, hostname=hostname, ldapadminpass=ldapadminpass, slapd_path=slapd_path, ldap_backend_extra_port=ldap_backend_extra_port, @@ -367,7 +365,7 @@ class OpenLDAPBackend(LDAPBackend): url_list=filter(None,self.ol_mmr_urls.split(',')) for url in url_list: - self.message("Using LDAP-URL: "+url) + self.logger.info("Using LDAP-URL: "+url) if (len(url_list) == 1): raise ProvisioningError("At least 2 LDAP-URLs needed for MMR!") @@ -557,14 +555,14 @@ class OpenLDAPBackend(LDAPBackend): class FDSBackend(LDAPBackend): def __init__(self, backend_type, paths=None, setup_path=None, lp=None, - credentials=None, names=None, message=None, domainsid=None, + credentials=None, names=None, logger=None, domainsid=None, schema=None, hostname=None, ldapadminpass=None, slapd_path=None, ldap_backend_extra_port=None, ldap_dryrun_mode=False, root=None, setup_ds_path=None): super(FDSBackend, self).__init__(backend_type=backend_type, paths=paths, setup_path=setup_path, lp=lp, - credentials=credentials, names=names, message=message, + credentials=credentials, names=names, logger=logger, domainsid=domainsid, schema=schema, hostname=hostname, ldapadminpass=ldapadminpass, slapd_path=slapd_path, ldap_backend_extra_port=ldap_backend_extra_port, @@ -720,10 +718,9 @@ class FDSBackend(LDAPBackend): # Try to print helpful messages when the user has not specified the path to the setup-ds tool if self.setup_ds_path is None: - raise ProvisioningError("Warning: Fedora DS LDAP-Backend must be setup with path to setup-ds, e.g. --setup-ds-path=\"/usr/sbin/setup-ds.pl\"!") + raise ProvisioningError("Fedora DS LDAP-Backend must be setup with path to setup-ds, e.g. --setup-ds-path=\"/usr/sbin/setup-ds.pl\"!") if not os.path.exists(self.setup_ds_path): - self.message (self.setup_ds_path) - raise ProvisioningError("Warning: Given Path to slapd does not exist!") + self.logger.warning("Path (%s) to slapd does not exist!", self.setup_ds_path) # Run the Fedora DS setup utility retcode = subprocess.call([self.setup_ds_path, "--silent", "--file", self.fedoradsinf], close_fds=True, shell=False) -- cgit