diff --git a/lib/hypervisor/hv_kvm.py b/lib/hypervisor/hv_kvm.py index 1fe502eb43ff0560ee6d610c5762aa2d62440fd0..26aacfdfadd31aa3f6858fbe438cafd932648d0f 100644 --- a/lib/hypervisor/hv_kvm.py +++ b/lib/hypervisor/hv_kvm.py @@ -1,7 +1,7 @@ # # -# Copyright (C) 2008, 2009, 2010, 2011, 2012 Google Inc. +# Copyright (C) 2008, 2009, 2010, 2011, 2012, 2013 Google Inc. # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -2034,14 +2034,17 @@ class KVMHypervisor(hv_base.BaseHypervisor): @return: Problem description if something is wrong, C{None} otherwise """ - # FIXME: this is the global kvm version, but the actual version can be - # customized as an hv parameter. we should use the nodegroup's default kvm - # path parameter here. + msgs = [] + # FIXME: this is the global kvm binary, but the actual path can be + # customized as an hv parameter; we should use the nodegroup's + # default kvm path parameter here. if not os.path.exists(constants.KVM_PATH): - return "The KVM binary ('%s') does not exist" % constants.KVM_PATH + msgs.append("The KVM binary ('%s') does not exist" % constants.KVM_PATH) if not os.path.exists(constants.SOCAT_PATH): - return "The socat binary ('%s') does not exist" % constants.SOCAT_PATH - return None + msgs.append("The socat binary ('%s') does not exist" % + constants.SOCAT_PATH) + + return self._FormatVerifyResults(msgs) @classmethod def CheckParameterSyntax(cls, hvparams):