qa: disable inter-cluster-instance-move for file
authorGuido Trotter <ultrotter@google.com>
Sun, 7 Apr 2013 05:53:37 +0000 (07:53 +0200)
committerGuido Trotter <ultrotter@google.com>
Sun, 7 Apr 2013 11:59:23 +0000 (13:59 +0200)
This actually hides a real issue that was discovered by enabling
file-based qa, so bug was filed for it. On the other hand it's better
than keeping the qa broken, since this functionality was broken anyway.

Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Michele Tartara <mtartara@google.com>

qa/ganeti-qa.py

index 1ea0b25..571b05a 100755 (executable)
@@ -391,7 +391,10 @@ def RunExportImportTests(instance, inodes):
     finally:
       expnode.Release()
 
-  if qa_config.TestEnabled([qa_rapi.Enabled, "inter-cluster-instance-move"]):
+  # FIXME: inter-cluster-instance-move crashes on file based instances :/
+  # See Issue 414.
+  if (qa_config.TestEnabled([qa_rapi.Enabled, "inter-cluster-instance-move"])
+      and instance.disk_template != constants.DT_FILE):
     newinst = qa_config.AcquireInstance()
     try:
       tnode = qa_config.AcquireNode(exclude=inodes)