Move constant for /etc/hosts to pathutils
authorMichael Hanselmann <hansmi@google.com>
Wed, 10 Oct 2012 08:42:38 +0000 (10:42 +0200)
committerMichael Hanselmann <hansmi@google.com>
Thu, 11 Oct 2012 09:56:27 +0000 (11:56 +0200)
Needed for coming patches.

Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>

lib/backend.py
lib/bootstrap.py
lib/cmdlib.py
lib/constants.py
lib/pathutils.py
lib/utils/nodesetup.py

index aafe456..e1a410f 100644 (file)
@@ -200,7 +200,7 @@ def _BuildUploadFileList():
   """
   allowed_files = set([
     pathutils.CLUSTER_CONF_FILE,
-    constants.ETC_HOSTS,
+    pathutils.ETC_HOSTS,
     pathutils.SSH_KNOWN_HOSTS_FILE,
     pathutils.VNC_PASSWORD_FILE,
     pathutils.RAPI_CERT_FILE,
index dfd415b..5a784b9 100644 (file)
@@ -329,7 +329,7 @@ def InitCluster(cluster_name, mac_prefix, # pylint: disable=R0913, R0914
   if ipcls.IsLoopback(hostname.ip):
     raise errors.OpPrereqError("This host's IP (%s) resolves to a loopback"
                                " address. Please fix DNS or %s." %
-                               (hostname.ip, constants.ETC_HOSTS),
+                               (hostname.ip, pathutils.ETC_HOSTS),
                                errors.ECODE_ENVIRON)
 
   if not ipcls.Own(hostname.ip):
index 916f243..62ba065 100644 (file)
@@ -4311,7 +4311,7 @@ def _ComputeAncillaryFiles(cluster, redist):
     files_all.update(ssconf.SimpleStore().GetFileList())
 
   if cluster.modify_etc_hosts:
-    files_all.add(constants.ETC_HOSTS)
+    files_all.add(pathutils.ETC_HOSTS)
 
   if cluster.use_external_mip_script:
     files_all.add(pathutils.EXTERNAL_MASTER_SETUP_SCRIPT)
index f3e1870..86b4fd6 100644 (file)
@@ -136,7 +136,6 @@ RUN_DIRS_MODE = 0775
 SECURE_DIR_MODE = 0700
 SECURE_FILE_MODE = 0600
 ADOPTABLE_BLOCKDEV_ROOT = "/dev/disk/"
-ETC_HOSTS = "/etc/hosts"
 ENABLE_FILE_STORAGE = _autoconf.ENABLE_FILE_STORAGE
 ENABLE_SHARED_FILE_STORAGE = _autoconf.ENABLE_SHARED_FILE_STORAGE
 ENABLE_CONFD = _autoconf.ENABLE_CONFD
index 457b227..0efbc4c 100644 (file)
@@ -45,6 +45,7 @@ KVM_CONSOLE_WRAPPER = _autoconf.PKGLIBDIR + "/tools/kvm-console-wrapper"
 KVM_IFUP = _autoconf.PKGLIBDIR + "/kvm-ifup"
 SETUP_SSH = _autoconf.TOOLSDIR + "/setup-ssh"
 XM_CONSOLE_WRAPPER = _autoconf.PKGLIBDIR + "/tools/xm-console-wrapper"
+ETC_HOSTS = "/etc/hosts"
 
 # Top-level paths
 DATA_DIR = LOCALSTATEDIR + "/lib/ganeti"
index 5dfc621..d34741f 100644 (file)
@@ -24,7 +24,7 @@
 
 from cStringIO import StringIO
 
-from ganeti import constants
+from ganeti import pathutils
 
 from ganeti.utils import algo
 from ganeti.utils import io
@@ -71,12 +71,12 @@ def AddHostToEtcHosts(hostname, ip):
 
   @type hostname: str
   @param hostname: a hostname that will be resolved and added to
-      L{constants.ETC_HOSTS}
+      L{pathutils.ETC_HOSTS}
   @type ip: str
   @param ip: The ip address of the host
 
   """
-  SetEtcHostsEntry(constants.ETC_HOSTS, ip, hostname, [hostname.split(".")[0]])
+  SetEtcHostsEntry(pathutils.ETC_HOSTS, ip, hostname, [hostname.split(".")[0]])
 
 
 def RemoveEtcHostsEntry(file_name, hostname):
@@ -115,8 +115,8 @@ def RemoveHostFromEtcHosts(hostname):
   @type hostname: str
   @param hostname: hostname that will be resolved and its
       full and shot name will be removed from
-      L{constants.ETC_HOSTS}
+      L{pathutils.ETC_HOSTS}
 
   """
-  RemoveEtcHostsEntry(constants.ETC_HOSTS, hostname)
-  RemoveEtcHostsEntry(constants.ETC_HOSTS, hostname.split(".")[0])
+  RemoveEtcHostsEntry(pathutils.ETC_HOSTS, hostname)
+  RemoveEtcHostsEntry(pathutils.ETC_HOSTS, hostname.split(".")[0])