----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8432/ -----------------------------------------------------------
Review request for cloudstack, Murali Reddy and Kelven Yang. Description ------- Issue seen during system vm template upgrade and restoreVM command scenarios for vmware. In these cases CS tries to recreate root disk with same name as the existing one, in case of vmware this results in creation of vmdk file with same name for both existing and new root volume. This results in undesired behavior when storage cleanup thread tries to cleanup old volume. Made the vmdk file name unique by adding the volume id to it. This will ensure that during volume recreation in the scenarios mentioned vmdk will get created with a new name and there will be no undesired side effects of running the storage cleanup thread. This addresses bug CLOUDSTACK-595. Diffs ----- plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java 0b0f285 server/src/com/cloud/storage/StorageManagerImpl.java e252633 Diff: https://reviews.apache.org/r/8432/diff/ Testing ------- Enabled storage cleanup (check storage.cleanup.* global settings) Verified the following: - On the primary storage verify that root vmdk file is created with a name like ROOT-<vm_id>-<volume_id>.vmdk - After system vm template upgrade for vmware verified that the vm continues to run even after storage cleanup thread has executed. And the new vmdk file for root volume exists on primary storage - After running restoreVirtualMachine command verify that a new vmdk gets created. Thanks, Koushik Das
