From 83df80b715c76bf4db38d20eac9bff8b3f27eacb Mon Sep 17 00:00:00 2001 From: Kishan Kavala Date: Thu, 10 Apr 2014 12:59:04 +0530 Subject: [PATCH] CLOUDSTACK-6371: Set snapshot size in copycommand answer during snapshot backup (cherry picked from commit 295fa84d4d3bb100055d7124bb0d0d68491328fc) Signed-off-by: Rohit Yadav --- .../cloud/hypervisor/kvm/storage/KVMStorageProcessor.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java index 8d237cc0c4d..3d1ca55b572 100644 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java @@ -689,6 +689,7 @@ public class KVMStorageProcessor implements StorageProcessor { primaryStore.getUuid(), volumePath); primaryPool = snapshotDisk.getPool(); + long size = 0; /** * RBD snapshots can't be copied using qemu-img, so we have to use * the Java bindings for librbd here. @@ -733,6 +734,7 @@ public class KVMStorageProcessor implements StorageProcessor { offset += bytes; } s_logger.debug("Completed backing up RBD snapshot " + snapshotName + " to " + snapFile.getAbsolutePath() + ". Bytes written: " + offset); + size = offset; bos.close(); s_logger.debug("Attempting to remove snapshot RBD " + snapshotName + " from image " + snapshotDisk.getName()); @@ -763,10 +765,15 @@ public class KVMStorageProcessor implements StorageProcessor { s_logger.debug("Failed to backup snaptshot: " + result); return new CopyCmdAnswer(result); } + File snapFile = new File(snapshotDestPath + "/" + snapshotName); + if(snapFile.exists()){ + size = snapFile.length(); + } } SnapshotObjectTO newSnapshot = new SnapshotObjectTO(); newSnapshot.setPath(snapshotRelPath + File.separator + snapshotName); + newSnapshot.setPhysicalSize(size); return new CopyCmdAnswer(newSnapshot); } catch (LibvirtException e) { s_logger.debug("Failed to backup snapshot: " + e.toString());