-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7594/
-----------------------------------------------------------

Review request for cloudstack, Nitin Mehta and anthony xu.


Description
-------

Fixed CS-16263
The issue is happening because of the multiple secondary storages, the 
secondary storage which is used first is getting mounted on the host resulting 
in failure of other snapshot using different secondary storage as it is not 
getting mounted on the host. 
For eg:
1. When first incremental snapshot (eg Ssec1-1) is taken, secondary storage 
(sec1) is mounted on local path “/var/run/sr-mount/ on the host. 
2. Second incremental snapshot (Ssec1-2) is created, to get the parent vhd, the 
secondary storage is mounted at mount path 
“/var/run/cloud_mount/dc_id/snapshots”.
3. If other snapshot (for different volume) is created on different secondary 
storage (sec2), sec2 is not getting mounted on 
"/var/run/cloud_mount/dc_id/snapshots" path.                              

The fix is to mount other secondary storages as well on the host. The local 
mount path in vmopsSnapshot.py is modified, instead of mounting on 
“/var/run/cloud_mount/dc_id/snapshots”, 
“/var/run/cloud_mount/dc_id/snapshots/secHostid” mount path is used.


This addresses bug CS-16263.


Diffs
-----

  api/src/com/cloud/agent/api/BackupSnapshotCommand.java 9476d7d 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 d2db85c 
  scripts/vm/hypervisor/xenserver/vmopsSnapshot 80e21f8 
  server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java d89a6d9 

Diff: https://reviews.apache.org/r/7594/diff/


Testing
-------

Tested for following cases:
1. Snapshot Creation: 
Host 1 has a vm i-1, volume ROOT-1 DATA-1, Host 2 has a vm i-2, volume ROOT-2 
DATA-2, Taken recurring snapshot for 4 volumes.
Verified: multiple secondary storages are getting mounted on host. Snapshots 
creation successful
2. Template Creation: 
Verified: templates successfully created from the above snapshots
3. Volume creation:
Verified: volumes successfully created from the above snapshots
4. VM Migration
Migrate i-1  from Host-1 to Host-2. 
Verified: Secondary mount point successfully created on Host-2.  
 


Thanks,

deepti dohare

Reply via email to