51d9a2
From 4c301176e317d6809a8a39ab0ad971f5d2e1ff65 Mon Sep 17 00:00:00 2001
51d9a2
Message-Id: <4c301176e317d6809a8a39ab0ad971f5d2e1ff65@dist-git>
51d9a2
From: John Ferlan <jferlan@redhat.com>
51d9a2
Date: Mon, 27 Aug 2018 08:23:52 -0400
51d9a2
Subject: [PATCH] storage: Add --shrink to qemu-img command when shrinking vol
51d9a2
MIME-Version: 1.0
51d9a2
Content-Type: text/plain; charset=UTF-8
51d9a2
Content-Transfer-Encoding: 8bit
51d9a2
51d9a2
https://bugzilla.redhat.com/show_bug.cgi?id=1613746
51d9a2
51d9a2
When shrinking the capacity of a qcow2 or luks volume using
51d9a2
the qemu-img program, the --shrink qualifier must be added.
51d9a2
51d9a2
Signed-off-by: John Ferlan <jferlan@redhat.com>
51d9a2
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
51d9a2
(cherry picked from commit b04d1b6a35f9fb826f7bb83fdac364a127ec75ca)
51d9a2
https: //bugzilla.redhat.com/show_bug.cgi?id=1622534
51d9a2
Reviewed-by: Erik Skultety <eskultet@redhat.com>
51d9a2
---
51d9a2
 src/storage/storage_util.c | 8 ++++----
51d9a2
 1 file changed, 4 insertions(+), 4 deletions(-)
51d9a2
51d9a2
diff --git a/src/storage/storage_util.c b/src/storage/storage_util.c
51d9a2
index a701a75702..5dc22d3182 100644
51d9a2
--- a/src/storage/storage_util.c
51d9a2
+++ b/src/storage/storage_util.c
51d9a2
@@ -2287,12 +2287,12 @@ storageBackendResizeQemuImg(virStoragePoolObjPtr pool,
51d9a2
      * a multiple of 512 */
51d9a2
     capacity = VIR_ROUND_UP(capacity, 512);
51d9a2
 
51d9a2
-    cmd = virCommandNew(img_tool);
51d9a2
+    cmd = virCommandNewArgList(img_tool, "resize", NULL);
51d9a2
+    if (capacity < vol->target.capacity)
51d9a2
+        virCommandAddArg(cmd, "--shrink");
51d9a2
     if (!vol->target.encryption) {
51d9a2
-        virCommandAddArgList(cmd, "resize", vol->target.path, NULL);
51d9a2
+        virCommandAddArg(cmd, vol->target.path);
51d9a2
     } else {
51d9a2
-        virCommandAddArg(cmd, "resize");
51d9a2
-
51d9a2
         if (storageBackendCreateQemuImgSecretObject(cmd, secretPath,
51d9a2
                                                     secretAlias) < 0)
51d9a2
             goto cleanup;
51d9a2
-- 
51d9a2
2.18.0
51d9a2