From 8727d84f4268ddb7e8adbc615d353da905ca8daa Mon Sep 17 00:00:00 2001 From: Edison Su Date: Thu, 22 Sep 2011 14:49:33 -0700 Subject: [PATCH] Fixed: only the first snapshot got backedup to secondary storage, if multple snapshot taken. Reviewed-by: anthony --- .../agent/resource/computing/LibvirtComputingResource.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/agent/src/com/cloud/agent/resource/computing/LibvirtComputingResource.java b/agent/src/com/cloud/agent/resource/computing/LibvirtComputingResource.java index ddb4da7bd8d..290130010fd 100644 --- a/agent/src/com/cloud/agent/resource/computing/LibvirtComputingResource.java +++ b/agent/src/com/cloud/agent/resource/computing/LibvirtComputingResource.java @@ -1215,7 +1215,7 @@ public class LibvirtComputingResource extends ServerResourceBase implements Serv s_logger.debug("Failed to manage snapshot: " + e.toString()); return new ManageSnapshotAnswer(cmd, false, "Failed to manage snapshot: " + e.toString()); } - return new ManageSnapshotAnswer(cmd, cmd.getSnapshotId(), cmd.getVolumePath(), true, null); + return new ManageSnapshotAnswer(cmd, cmd.getSnapshotId(), cmd.getVolumePath() + File.separator + snapshotName, true, null); } protected BackupSnapshotAnswer execute(final BackupSnapshotCommand cmd) { @@ -1224,7 +1224,7 @@ public class LibvirtComputingResource extends ServerResourceBase implements Serv Long volumeId = cmd.getVolumeId(); String secondaryStoragePoolURL = cmd.getSecondaryStoragePoolURL(); String snapshotName = cmd.getSnapshotName(); - String snapshotPath = cmd.getSnapshotUuid(); + String snapshotPath = cmd.getVolumePath(); String snapshotDestPath = null; String vmName = cmd.getVmName();