While previous patches enabled QA tests of exports in instance moves,
this patch does the same for local exports.

Signed-off-by: Hrvoje Ribicic <r...@google.com>
---
 qa/ganeti-qa.py   | 3 +--
 qa/qa_instance.py | 9 ++++++---
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/qa/ganeti-qa.py b/qa/ganeti-qa.py
index 4c7ff2f..8572cd3 100755
--- a/qa/ganeti-qa.py
+++ b/qa/ganeti-qa.py
@@ -469,8 +469,7 @@ def RunExportImportTests(instance, inodes):
   # FIXME: export explicitly bails out on file based storage. other non-lvm
   # based storage types are untested, though. Also note that import could still
   # work, but is deeply embedded into the "export" case.
-  if (qa_config.TestEnabled("instance-export") and
-      instance.disk_template not in constants.DTS_FILEBASED):
+  if qa_config.TestEnabled("instance-export"):
     RunTest(qa_instance.TestInstanceExportNoTarget, instance)
 
     pnode = inodes[0]
diff --git a/qa/qa_instance.py b/qa/qa_instance.py
index e997701..18df485 100644
--- a/qa/qa_instance.py
+++ b/qa/qa_instance.py
@@ -1073,10 +1073,13 @@ def TestRecreateDisks(instance, inodes, othernodes):
 def TestInstanceExport(instance, node):
   """gnt-backup export -n ..."""
   name = instance.name
-  # Export does not work for file-based templates, thus we skip the test
+  options = ["gnt-backup", "export", "-n", node.primary]
+
+  # For files and shared files, the --long-sleep option should be used
   if instance.disk_template in [constants.DT_FILE, constants.DT_SHARED_FILE]:
-    return
-  AssertCommand(["gnt-backup", "export", "-n", node.primary, name])
+    options.append("--long-sleep")
+
+  AssertCommand(options + [name])
   return qa_utils.ResolveInstanceName(name)
 
 
-- 
2.2.0.rc0.207.ga3a616c

Reply via email to