PR #27 CLOUDSTACK-7767

yadvr · yadvr · commit 448bb797404c · 2014-10-30T15:10:35.000+05:30
Fixed events are not generated for snapshot creation #27 This closes #27
diff --git a/server/src/com/cloud/storage/VolumeApiServiceImpl.java b/server/src/com/cloud/storage/VolumeApiServiceImpl.java
@@ -1828,6 +1828,7 @@ protected Volume liveMigrateVolume(Volume volume, StoragePool destPool) {
     }
 
     @Override
+    @ActionEvent(eventType = EventTypes.EVENT_SNAPSHOT_CREATE, eventDescription = "taking snapshot", async = true)
     public Snapshot takeSnapshot(Long volumeId, Long policyId, Long snapshotId, Account account, boolean quiescevm) throws ResourceAllocationException {
 
         VolumeInfo volume = volFactory.getVolume(volumeId);
@@ -1914,6 +1915,7 @@ private Snapshot orchestrateTakeVolumeSnapshot(Long volumeId, Long policyId, Lon
     }
 
     @Override
+    @ActionEvent(eventType = EventTypes.EVENT_SNAPSHOT_CREATE, eventDescription = "allocating snapshot", create = true)
     public Snapshot allocSnapshot(Long volumeId, Long policyId) throws ResourceAllocationException {
         Account caller = CallContext.current().getCallingAccount();
 

Original file line number	Diff line number	Diff line change
`@@ -1828,6 +1828,7 @@ protected Volume liveMigrateVolume(Volume volume, StoragePool destPool) {`
`1828`	`1828`	`}`
`1829`	`1829`
`1830`	`1830`	`@Override`
	`1831`	`+ @ActionEvent(eventType = EventTypes.EVENT_SNAPSHOT_CREATE, eventDescription = "taking snapshot", async = true)`
`1831`	`1832`	`public Snapshot takeSnapshot(Long volumeId, Long policyId, Long snapshotId, Account account, boolean quiescevm) throws ResourceAllocationException {`
`1832`	`1833`
`1833`	`1834`	`VolumeInfo volume = volFactory.getVolume(volumeId);`
`@@ -1914,6 +1915,7 @@ private Snapshot orchestrateTakeVolumeSnapshot(Long volumeId, Long policyId, Lon`
`1914`	`1915`	`}`
`1915`	`1916`
`1916`	`1917`	`@Override`
	`1918`	`+ @ActionEvent(eventType = EventTypes.EVENT_SNAPSHOT_CREATE, eventDescription = "allocating snapshot", create = true)`
`1917`	`1919`	`public Snapshot allocSnapshot(Long volumeId, Long policyId) throws ResourceAllocationException {`
`1918`	`1920`	`Account caller = CallContext.current().getCallingAccount();`
`1919`	`1921`