Commit 1f4b9d39 authored by Iustin Pop's avatar Iustin Pop
Browse files

Switch KVM to multi-error verify results



This uses the new _FormatVerifyResults helper function to return
multiple errors.
Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
parent 0628a50a
#
#
# 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):
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment