[PATCH] tests: be more explicit on qcow2 versions in virstoragetest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



While working on v1.0.5-maint (the branch in use on Fedora 19)
with the host at Fedora 20, I got a failure in virstoragetest.
I traced it to the fact that we were using qemu-img to create a
qcow2 file, but qemu-img changed from creating v2 files by
default in F19 to creating v3 files in F20.  Rather than leaving
it up to qemu-img, it is better to write the test to force
testing of BOTH file formats (better code coverage and all).

This patch alone does not fix all the failures in v1.0.5-maint;
for that, we must decide to either teach the older branch to
understand v3 files, or to reject them outright as unsupported.
But for upstream, making the test less dependent on changing
qemu-img defaults is always a good thing.

* tests/virstoragetest.c (testPrepImages): Simplify creation of
raw file; check if qemu supports compat and if so use it.

Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>
---
 tests/virstoragetest.c | 32 +++++++++++++++++++++++---------
 1 file changed, 23 insertions(+), 9 deletions(-)

diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c
index e5c73f5..db0cf1c 100644
--- a/tests/virstoragetest.c
+++ b/tests/virstoragetest.c
@@ -87,6 +87,8 @@ testPrepImages(void)
 {
     int ret = EXIT_FAILURE;
     virCommandPtr cmd = NULL;
+    char *buf = NULL;
+    bool compat = false;

     qemuimg = virFindFileInPath("kvm-img");
     if (!qemuimg)
@@ -94,6 +96,18 @@ testPrepImages(void)
     if (!qemuimg)
         goto skip;

+    /* See if qemu-img supports '-o compat=xxx'.  If so, we force the
+     * use of both v2 and v3 files; if not, it is v2 only but the test
+     * still works. */
+    cmd = virCommandNewArgList(qemuimg, "create", "-f", "qcow2",
+                               "-o?", "/dev/null", NULL);
+    virCommandSetOutputBuffer(cmd, &buf);
+    if (virCommandRun(cmd, NULL) < 0)
+        goto skip;
+    if (strstr(buf, "compat "))
+        compat = true;
+    VIR_FREE(buf);
+
     if (virAsprintf(&absraw, "%s/raw", datadir) < 0 ||
         virAsprintf(&absqcow2, "%s/qcow2", datadir) < 0 ||
         virAsprintf(&abswrap, "%s/wrap", datadir) < 0 ||
@@ -111,10 +125,8 @@ testPrepImages(void)
         goto cleanup;
     }

-    /* I'm lazy enough to use a shell one-liner instead of open/write/close */
-    virCommandFree(cmd);
-    cmd = virCommandNewArgList("sh", "-c", "printf %1024d 0 > raw", NULL);
-    if (virCommandRun(cmd, NULL) < 0) {
+    if (virAsprintf(&buf, "%1024d", 0) < 0 ||
+        virFileWriteStr("raw", buf, 0600) < 0) {
         fprintf(stderr, "unable to create raw file\n");
         goto cleanup;
     }
@@ -126,9 +138,10 @@ testPrepImages(void)
     /* Create a qcow2 wrapping relative raw; later on, we modify its
      * metadata to test other configurations */
     virCommandFree(cmd);
-    cmd = virCommandNewArgList(qemuimg, "create", "-f", "qcow2",
-                               "-obacking_file=raw,backing_fmt=raw", "qcow2",
-                               NULL);
+    cmd = virCommandNewArgList(qemuimg, "create", "-f", "qcow2", NULL);
+    virCommandAddArgFormat(cmd, "-obacking_file=raw,backing_fmt=raw%s",
+                           compat ? ",compat=0.10" : "");
+    virCommandAddArg(cmd, "qcow2");
     if (virCommandRun(cmd, NULL) < 0)
         goto skip;
     /* Make sure our later uses of 'qemu-img rebase' will work */
@@ -146,8 +159,8 @@ testPrepImages(void)
      * can correctly avoid insecure probing.  */
     virCommandFree(cmd);
     cmd = virCommandNewArgList(qemuimg, "create", "-f", "qcow2", NULL);
-    virCommandAddArgFormat(cmd, "-obacking_file=%s,backing_fmt=qcow2",
-                           absqcow2);
+    virCommandAddArgFormat(cmd, "-obacking_file=%s,backing_fmt=qcow2%s",
+                           absqcow2, compat ? ",compat=1.1" : "");
     virCommandAddArg(cmd, "wrap");
     if (virCommandRun(cmd, NULL) < 0)
         goto skip;
@@ -172,6 +185,7 @@ testPrepImages(void)

     ret = 0;
 cleanup:
+    VIR_FREE(buf);
     virCommandFree(cmd);
     if (ret)
         testCleanupImages();
-- 
1.8.4.2

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]