From: Guido Trotter <ultrotter@google.com>
Date: Wed, 11 Feb 2009 10:19:49 +0000 (+0000)
Subject: HTS_USE_VNC, rename and remove KVM
X-Git-Tag: v2.0.0beta2~47
X-Git-Url: https://code.grnet.gr/git/ganeti-local/commitdiff_plain/ccd905ac8a44b286f6381dc96bb9a1a8af6cf0ff

HTS_USE_VNC, rename and remove KVM

Currently we use the HTS_USE_VNC constant only to copy the vnc password
file. While KVM uses vnc it currently has no password support, nor we'll
be on time making one for 2.0, so renaming the constant to
HTS_COPY_VNC_PASSWORD and only putting Xen HVM in it. In the future
(2.1) password handling will need to be reworked anyway.

Reviewed-by: iustinp
---

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index 0f3e517..58b4e03 100644
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -2209,7 +2209,7 @@ class LUAddNode(LogicalUnit):
 
     to_copy = []
     enabled_hypervisors = self.cfg.GetClusterInfo().enabled_hypervisors
-    if constants.HTS_USE_VNC.intersection(enabled_hypervisors):
+    if constants.HTS_COPY_VNC_PASSWORD.intersection(enabled_hypervisors):
       to_copy.append(constants.VNC_PASSWORD_FILE)
 
     for fname in to_copy:
diff --git a/lib/constants.py b/lib/constants.py
index 04e63e5..a50526e 100644
--- a/lib/constants.py
+++ b/lib/constants.py
@@ -348,7 +348,7 @@ HT_XEN_HVM = "xen-hvm"
 HT_KVM = "kvm"
 HYPER_TYPES = frozenset([HT_XEN_PVM, HT_FAKE, HT_XEN_HVM, HT_KVM])
 HTS_REQ_PORT = frozenset([HT_XEN_HVM, HT_KVM])
-HTS_USE_VNC = frozenset([HT_XEN_HVM, HT_KVM])
+HTS_COPY_VNC_PASSWORD = frozenset([HT_XEN_HVM])
 
 VNC_BASE_PORT = 5900
 VNC_PASSWORD_FILE = _autoconf.SYSCONFDIR + "/ganeti/vnc-cluster-password"