X-Git-Url: https://code.grnet.gr/git/ganeti-local/blobdiff_plain/7910e7a55004a75a909de7dd9a1b657ab09a6875..bcd87e22bb462c87c50fc07a2cca14a977e79253:/qa/qa_instance.py diff --git a/qa/qa_instance.py b/qa/qa_instance.py index e9ea788..ab2a0f4 100644 --- a/qa/qa_instance.py +++ b/qa/qa_instance.py @@ -1,7 +1,7 @@ # # -# Copyright (C) 2007 Google Inc. +# Copyright (C) 2007, 2011 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 @@ -28,12 +28,13 @@ import time from ganeti import utils from ganeti import constants +from ganeti import query import qa_config import qa_utils import qa_error -from qa_utils import AssertEqual, AssertNotEqual, StartSSH +from qa_utils import AssertIn, AssertCommand, AssertEqual def _GetDiskStatePath(disk): @@ -41,14 +42,13 @@ def _GetDiskStatePath(disk): def _GetGenericAddParameters(): - return ['--os-size=%s' % qa_config.get('os-size'), - '--swap-size=%s' % qa_config.get('swap-size'), - '--memory=%s' % qa_config.get('mem')] + params = ['-B', '%s=%s' % (constants.BE_MEMORY, qa_config.get('mem'))] + for idx, size in enumerate(qa_config.get('disk')): + params.extend(["--disk", "%s:size=%s" % (idx, size)]) + return params def _DiskTest(node, disk_template): - master = qa_config.GetMasterNode() - instance = qa_config.AcquireInstance() try: cmd = (['gnt-instance', 'add', @@ -58,208 +58,260 @@ def _DiskTest(node, disk_template): _GetGenericAddParameters()) cmd.append(instance['name']) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(cmd) + + _CheckSsconfInstanceList(instance["name"]) + return instance except: qa_config.ReleaseInstance(instance) raise -@qa_utils.DefineHook('instance-add-plain-disk') def TestInstanceAddWithPlainDisk(node): """gnt-instance add -t plain""" return _DiskTest(node['primary'], 'plain') -@qa_utils.DefineHook('instance-add-drbd-disk') def TestInstanceAddWithDrbdDisk(node, node2): """gnt-instance add -t drbd""" return _DiskTest("%s:%s" % (node['primary'], node2['primary']), 'drbd') -@qa_utils.DefineHook('instance-remove') def TestInstanceRemove(instance): """gnt-instance remove""" - master = qa_config.GetMasterNode() - - cmd = ['gnt-instance', 'remove', '-f', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "remove", "-f", instance["name"]]) qa_config.ReleaseInstance(instance) -@qa_utils.DefineHook('instance-startup') def TestInstanceStartup(instance): """gnt-instance startup""" - master = qa_config.GetMasterNode() - - cmd = ['gnt-instance', 'startup', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "startup", instance["name"]]) -@qa_utils.DefineHook('instance-shutdown') def TestInstanceShutdown(instance): """gnt-instance shutdown""" - master = qa_config.GetMasterNode() + AssertCommand(["gnt-instance", "shutdown", instance["name"]]) + + +def TestInstanceReboot(instance): + """gnt-instance reboot""" + options = qa_config.get('options', {}) + reboot_types = options.get("reboot-types", constants.REBOOT_TYPES) + name = instance["name"] + for rtype in reboot_types: + AssertCommand(["gnt-instance", "reboot", "--type=%s" % rtype, name]) + + AssertCommand(["gnt-instance", "shutdown", name]) + AssertCommand(["gnt-instance", "reboot", name]) - cmd = ['gnt-instance', 'shutdown', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + master = qa_config.GetMasterNode() + cmd = ["gnt-instance", "list", "--no-header", "-o", "status", name] + result_output = qa_utils.GetCommandOutput(master["primary"], + utils.ShellQuoteArgs(cmd)) + AssertEqual(result_output.strip(), constants.INSTST_RUNNING) -@qa_utils.DefineHook('instance-reinstall') def TestInstanceReinstall(instance): """gnt-instance reinstall""" + AssertCommand(["gnt-instance", "reinstall", "-f", instance["name"]]) + + +def _ReadSsconfInstanceList(): + """Reads ssconf_instance_list from the master node. + + """ master = qa_config.GetMasterNode() - cmd = ['gnt-instance', 'reinstall', '-f', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + cmd = ["cat", utils.PathJoin(constants.DATA_DIR, + "ssconf_%s" % constants.SS_INSTANCE_LIST)] + + return qa_utils.GetCommandOutput(master["primary"], + utils.ShellQuoteArgs(cmd)).splitlines() + + +def _CheckSsconfInstanceList(instance): + """Checks if a certain instance is in the ssconf instance list. + + @type instance: string + @param instance: Instance name + + """ + AssertIn(qa_utils.ResolveInstanceName(instance), + _ReadSsconfInstanceList()) + + +def TestInstanceRename(rename_source, rename_target): + """gnt-instance rename""" + _CheckSsconfInstanceList(rename_source) + AssertCommand(["gnt-instance", "rename", rename_source, rename_target]) + _CheckSsconfInstanceList(rename_target) + AssertCommand(["gnt-instance", "rename", rename_target, rename_source]) + _CheckSsconfInstanceList(rename_source) + qa_utils.AddToEtcHosts(["meeeeh-not-exists", rename_target]) + try: + AssertCommand(["gnt-instance", "rename", rename_source, rename_target], + fail=True) + _CheckSsconfInstanceList(rename_source) + finally: + qa_utils.RemoveFromEtcHosts(["meeeeh-not-exists", rename_target]) + AssertCommand(["gnt-instance", "rename", rename_source, rename_target]) + _CheckSsconfInstanceList(rename_target) -@qa_utils.DefineHook('instance-failover') def TestInstanceFailover(instance): """gnt-instance failover""" - master = qa_config.GetMasterNode() - cmd = ['gnt-instance', 'failover', '--force', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + # failover ... + AssertCommand(cmd) + # ... and back + AssertCommand(cmd) + + +def TestInstanceMigrate(instance): + """gnt-instance migrate""" + cmd = ["gnt-instance", "migrate", "--force", instance["name"]] + # migrate ... + AssertCommand(cmd) + # ... and back + AssertCommand(cmd) + AssertCommand(["gnt-instance", "shutdown", instance["name"]]) + AssertCommand(cmd, fail=True) + AssertCommand(["gnt-instance", "migrate", "--force", "--allow-failover", + instance["name"]]) + AssertCommand(["gnt-instance", "start", instance["name"]]) + AssertCommand(cmd) -@qa_utils.DefineHook('instance-info') def TestInstanceInfo(instance): """gnt-instance info""" - master = qa_config.GetMasterNode() - - cmd = ['gnt-instance', 'info', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "info", instance["name"]]) -@qa_utils.DefineHook('instance-modify') def TestInstanceModify(instance): """gnt-instance modify""" - master = qa_config.GetMasterNode() + # Assume /sbin/init exists on all systems + test_kernel = "/sbin/init" + test_initrd = test_kernel orig_memory = qa_config.get('mem') - orig_bridge = qa_config.get('bridge', 'xen-br0') + #orig_bridge = qa_config.get("bridge", "xen-br0") args = [ - ["--memory", "128"], - ["--memory", str(orig_memory)], - ["--cpu", "2"], - ["--cpu", "1"], - ["--bridge", "xen-br1"], - ["--bridge", orig_bridge], - ["--kernel", "/dev/null"], - ["--kernel", "default"], - ["--initrd", "/dev/null"], - ["--initrd", "none"], - ["--initrd", "default"], - ["--hvm-boot-order", "acn"], - ["--hvm-boot-order", "default"], + ["-B", "%s=128" % constants.BE_MEMORY], + ["-B", "%s=%s" % (constants.BE_MEMORY, orig_memory)], + ["-B", "%s=2" % constants.BE_VCPUS], + ["-B", "%s=1" % constants.BE_VCPUS], + ["-B", "%s=%s" % (constants.BE_VCPUS, constants.VALUE_DEFAULT)], + + ["-H", "%s=%s" % (constants.HV_KERNEL_PATH, test_kernel)], + ["-H", "%s=%s" % (constants.HV_KERNEL_PATH, constants.VALUE_DEFAULT)], + ["-H", "%s=%s" % (constants.HV_INITRD_PATH, test_initrd)], + ["-H", "no_%s" % (constants.HV_INITRD_PATH, )], + ["-H", "%s=%s" % (constants.HV_INITRD_PATH, constants.VALUE_DEFAULT)], + + # TODO: bridge tests + #["--bridge", "xen-br1"], + #["--bridge", orig_bridge], + + # TODO: Do these tests only with xen-hvm + #["-H", "%s=acn" % constants.HV_BOOT_ORDER], + #["-H", "%s=%s" % (constants.HV_BOOT_ORDER, constants.VALUE_DEFAULT)], ] for alist in args: - cmd = ['gnt-instance', 'modify'] + alist + [instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "modify"] + alist + [instance["name"]]) # check no-modify - cmd = ['gnt-instance', 'modify', instance['name']] - AssertNotEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "modify", instance["name"]], fail=True) + + +def TestInstanceConvertDisk(instance, snode): + """gnt-instance modify -t""" + name = instance["name"] + AssertCommand(["gnt-instance", "modify", "-t", "plain", name]) + AssertCommand(["gnt-instance", "modify", "-t", "drbd", + "-n", snode["primary"], name]) -@qa_utils.DefineHook('instance-list') def TestInstanceList(): """gnt-instance list""" - master = qa_config.GetMasterNode() + qa_utils.GenericQueryTest("gnt-instance", query.INSTANCE_FIELDS.keys()) + - cmd = ['gnt-instance', 'list'] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) +def TestInstanceListFields(): + """gnt-instance list-fields""" + qa_utils.GenericQueryFieldsTest("gnt-instance", query.INSTANCE_FIELDS.keys()) -@qa_utils.DefineHook('instance-console') def TestInstanceConsole(instance): """gnt-instance console""" - master = qa_config.GetMasterNode() + AssertCommand(["gnt-instance", "console", "--show-cmd", instance["name"]]) - cmd = ['gnt-instance', 'console', '--show-cmd', instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) - -@qa_utils.DefineHook('instance-replace-disks') def TestReplaceDisks(instance, pnode, snode, othernode): """gnt-instance replace-disks""" - master = qa_config.GetMasterNode() - + # pylint: disable-msg=W0613 + # due to unused pnode arg + # FIXME: should be removed from the function completely def buildcmd(args): cmd = ['gnt-instance', 'replace-disks'] cmd.extend(args) cmd.append(instance["name"]) return cmd - cmd = buildcmd(["-p"]) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + for data in [ + ["-p"], + ["-s"], + ["--new-secondary=%s" % othernode["primary"]], + # and restore + ["--new-secondary=%s" % snode["primary"]], + ]: + AssertCommand(buildcmd(data)) - cmd = buildcmd(["-s"]) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(buildcmd(["-a"])) + AssertCommand(["gnt-instance", "stop", instance["name"]]) + AssertCommand(buildcmd(["-a"]), fail=True) + AssertCommand(["gnt-instance", "activate-disks", instance["name"]]) + AssertCommand(buildcmd(["-a"])) + AssertCommand(["gnt-instance", "start", instance["name"]]) - cmd = buildcmd(["--new-secondary=%s" % othernode["primary"]]) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) - # Restore - cmd = buildcmd(["--new-secondary=%s" % snode["primary"]]) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) +def TestInstanceExport(instance, node): + """gnt-backup export -n ...""" + name = instance["name"] + AssertCommand(["gnt-backup", "export", "-n", node["primary"], name]) + return qa_utils.ResolveInstanceName(name) -@qa_utils.DefineHook('backup-export') -def TestInstanceExport(instance, node): - """gnt-backup export""" - master = qa_config.GetMasterNode() +def TestInstanceExportWithRemove(instance, node): + """gnt-backup export --remove-instance""" + AssertCommand(["gnt-backup", "export", "-n", node["primary"], + "--remove-instance", instance["name"]]) - cmd = ['gnt-backup', 'export', '-n', node['primary'], instance['name']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) - return qa_utils.ResolveInstanceName(instance) +def TestInstanceExportNoTarget(instance): + """gnt-backup export (without target node, should fail)""" + AssertCommand(["gnt-backup", "export", instance["name"]], fail=True) -@qa_utils.DefineHook('backup-import') def TestInstanceImport(node, newinst, expnode, name): """gnt-backup import""" - master = qa_config.GetMasterNode() - cmd = (['gnt-backup', 'import', '--disk-template=plain', '--no-ip-check', + '--net', '0:mac=generate', '--src-node=%s' % expnode['primary'], '--src-dir=%s/%s' % (constants.EXPORT_DIR, name), '--node=%s' % node['primary']] + _GetGenericAddParameters()) cmd.append(newinst['name']) - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(cmd) -@qa_utils.DefineHook('backup-list') def TestBackupList(expnode): """gnt-backup list""" - master = qa_config.GetMasterNode() - - cmd = ['gnt-backup', 'list', '--node=%s' % expnode['primary']] - AssertEqual(StartSSH(master['primary'], - utils.ShellQuoteArgs(cmd)).wait(), 0) + AssertCommand(["gnt-backup", "list", "--node=%s" % expnode["primary"]]) def _TestInstanceDiskFailure(instance, node, node2, onmaster): @@ -267,7 +319,7 @@ def _TestInstanceDiskFailure(instance, node, node2, onmaster): master = qa_config.GetMasterNode() sq = utils.ShellQuoteArgs - instance_full = qa_utils.ResolveInstanceName(instance) + instance_full = qa_utils.ResolveInstanceName(instance["name"]) node_full = qa_utils.ResolveNodeName(node) node2_full = qa_utils.ResolveNodeName(node2) @@ -288,7 +340,7 @@ def _TestInstanceDiskFailure(instance, node, node2, onmaster): m = re_disk.match(phys) if not m: - raise qa_error.Error("Unknown disk name format: %s" % disk) + raise qa_error.Error("Unknown disk name format: %s" % phys) name = m.group(1) if name not in node2disk[node_name]: @@ -304,7 +356,7 @@ def _TestInstanceDiskFailure(instance, node, node2, onmaster): cmds = [] for disk in disks: cmds.append(sq(["test", "-f", _GetDiskStatePath(disk)])) - AssertEqual(StartSSH(node_name, ' && '.join(cmds)).wait(), 0) + AssertCommand(" && ".join(cmds), node=node_name) print qa_utils.FormatInfo("Getting device paths") cmd = ['gnt-instance', 'activate-disks', instance['name']] @@ -330,8 +382,7 @@ def _TestInstanceDiskFailure(instance, node, node2, onmaster): for name in node2disk[[node2_full, node_full][int(onmaster)]]: halted_disks.append(name) cmds.append(sq(["echo", "offline"]) + " >%s" % _GetDiskStatePath(name)) - AssertEqual(StartSSH([node2, node][int(onmaster)]['primary'], - ' && '.join(cmds)).wait(), 0) + AssertCommand(" && ".join(cmds), node=[node2, node][int(onmaster)]) print qa_utils.FormatInfo("Write to disks and give some time to notice" " to notice the problem") @@ -340,56 +391,46 @@ def _TestInstanceDiskFailure(instance, node, node2, onmaster): cmds.append(sq(["dd", "count=1", "bs=512", "conv=notrunc", "if=%s" % disk, "of=%s" % disk])) for _ in (0, 1, 2): - AssertEqual(StartSSH(node['primary'], ' && '.join(cmds)).wait(), 0) + AssertCommand(" && ".join(cmds), node=node) time.sleep(3) print qa_utils.FormatInfo("Debugging info") for name in drbddevs: - cmd = ['drbdsetup', name, 'show'] - AssertEqual(StartSSH(node['primary'], sq(cmd)).wait(), 0) + AssertCommand(["drbdsetup", name, "show"], node=node) - cmd = ['gnt-instance', 'info', instance['name']] - AssertEqual(StartSSH(master['primary'], sq(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "info", instance["name"]]) finally: print qa_utils.FormatInfo("Activating disks again") cmds = [] for name in halted_disks: cmds.append(sq(["echo", "running"]) + " >%s" % _GetDiskStatePath(name)) - AssertEqual(StartSSH([node2, node][int(onmaster)]['primary'], - '; '.join(cmds)).wait(), 0) + AssertCommand("; ".join(cmds), node=[node2, node][int(onmaster)]) if onmaster: for name in drbddevs: - cmd = ['drbdsetup', name, 'detach'] - AssertEqual(StartSSH(node['primary'], sq(cmd)).wait(), 0) + AssertCommand(["drbdsetup", name, "detach"], node=node) else: for name in drbddevs: - cmd = ['drbdsetup', name, 'disconnect'] - AssertEqual(StartSSH(node2['primary'], sq(cmd)).wait(), 0) + AssertCommand(["drbdsetup", name, "disconnect"], node=node2) # TODO - #cmd = ['vgs'] - #AssertEqual(StartSSH([node2, node][int(onmaster)]['primary'], - # sq(cmd)).wait(), 0) + #AssertCommand(["vgs"], [node2, node][int(onmaster)]) print qa_utils.FormatInfo("Making sure disks are up again") - cmd = ['gnt-instance', 'replace-disks', instance['name']] - AssertEqual(StartSSH(master['primary'], sq(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "replace-disks", instance["name"]]) print qa_utils.FormatInfo("Restarting instance") - cmd = ['gnt-instance', 'shutdown', instance['name']] - AssertEqual(StartSSH(master['primary'], sq(cmd)).wait(), 0) - - cmd = ['gnt-instance', 'startup', instance['name']] - AssertEqual(StartSSH(master['primary'], sq(cmd)).wait(), 0) + AssertCommand(["gnt-instance", "shutdown", instance["name"]]) + AssertCommand(["gnt-instance", "startup", instance["name"]]) - cmd = ['gnt-cluster', 'verify'] - AssertEqual(StartSSH(master['primary'], sq(cmd)).wait(), 0) + AssertCommand(["gnt-cluster", "verify"]) def TestInstanceMasterDiskFailure(instance, node, node2): """Testing disk failure on master node.""" + # pylint: disable-msg=W0613 + # due to unused args print qa_utils.FormatError("Disk failure on primary node cannot be" " tested due to potential crashes.") # The following can cause crashes, thus it's disabled until fixed