incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasanna Santhanam" <>
Subject Re: Review Request: CLOUDSTACK-649: KVM storage gc does not delete the invalid snapshots from physical storage
Date Mon, 17 Dec 2012 04:15:29 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Dec. 17, 2012, 4:15 a.m.)

Review request for cloudstack, Nitin Mehta, edison su, and Marcus Sorensen.


Including more reviewers


When CleanupSnapshotBackupCommand is sent to the SSVM agent because it needs to perform storage
GC - outdated snapshots remain undeleted. But the answer returned to mgmt server is success.
This is related to CLOUDSTACK-643 where KVM uses a different path to its snapshot backups
that includes the zoneid in the path.

KVM uses /snapshots/zoneid/accountid/volumeid for its snapshot path.  The
    cleanupbackupsnapshotcommand which performs GC interprets an invalid path (as
    /snapshots/accountid/volumeid) to the snapshot and fails to delete them from
    sec.  storage.
    Now GC path is corrected so snapshots removed fmor DB are also removed from
    secondary storage.

This addresses bug CLOUDSTACK-649.


  core/src/com/cloud/storage/resource/ d8fdc3a 



1. Tested create snapshot, delete snapshot
2. Tested recurring snapshot creation , deletion via storage gc

Both scenarios pass


Prasanna Santhanam

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message