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

Review request for cloudstack.


Description
-------

merge VM Snapshot feature from branch vm-snapshot to master 
KVM support is temporarily not uploaded, which is in 
LibvirtComputingResource.java, it needs latest libvirt.jar to compile.


This addresses bug CLOUDSTACK-684.


Diffs
-----

  api/src/com/cloud/agent/api/CreateVMSnapshotAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/CreateVMSnapshotCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/CreateVolumeFromVMSnapshotAnswer.java 
PRE-CREATION 
  api/src/com/cloud/agent/api/CreateVolumeFromVMSnapshotCommand.java 
PRE-CREATION 
  api/src/com/cloud/agent/api/DeleteVMSnapshotAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/DeleteVMSnapshotCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/RevertToVMSnapshotAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/RevertToVMSnapshotCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/VMSnapshotBaseCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/VMSnapshotTO.java PRE-CREATION 
  api/src/com/cloud/agent/api/to/VolumeTO.java ff739c5 
  api/src/com/cloud/event/EventTypes.java 87eddca 
  api/src/com/cloud/server/ResourceTag.java 5ec9f01 
  api/src/com/cloud/vm/VirtualMachine.java cd30579 
  api/src/com/cloud/vm/snapshot/VMSnapshot.java PRE-CREATION 
  api/src/com/cloud/vm/snapshot/VMSnapshotService.java PRE-CREATION 
  api/src/org/apache/cloudstack/api/ApiConstants.java 58a7831 
  api/src/org/apache/cloudstack/api/BaseCmd.java ef72af9 
  api/src/org/apache/cloudstack/api/ResponseGenerator.java 63df4dc 
  
api/src/org/apache/cloudstack/api/command/user/vmsnapshot/CreateVMSnapshotCmd.java
 PRE-CREATION 
  
api/src/org/apache/cloudstack/api/command/user/vmsnapshot/DeleteVMSnapshotCmd.java
 PRE-CREATION 
  
api/src/org/apache/cloudstack/api/command/user/vmsnapshot/ListVMSnapshotCmd.java
 PRE-CREATION 
  
api/src/org/apache/cloudstack/api/command/user/vmsnapshot/RevertToSnapshotCmd.java
 PRE-CREATION 
  api/src/org/apache/cloudstack/api/response/VMSnapshotResponse.java 
PRE-CREATION 
  client/WEB-INF/classes/resources/messages.properties 390738f 
  client/tomcatconf/commands.properties.in 3740fb0 
  core/src/com/cloud/vm/snapshot/VMSnapshotVO.java PRE-CREATION 
  
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManager.java
 f27e026 
  
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java
 8650274 
  
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
 dd917f7 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 065d3be 
  scripts/vm/hypervisor/xenserver/vmopsSnapshot 80e21f8 
  server/src/com/cloud/api/ApiDBUtils.java 0b08b26 
  server/src/com/cloud/api/ApiResponseHelper.java 1c8849a 
  server/src/com/cloud/capacity/CapacityManagerImpl.java c410109 
  server/src/com/cloud/configuration/Config.java 4ae144e 
  server/src/com/cloud/configuration/DefaultComponentLibrary.java 98da7ad 
  server/src/com/cloud/resource/ResourceManagerImpl.java f82424a 
  server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java 15d8c53 
  server/src/com/cloud/tags/TaggedResourceManagerImpl.java 42d3c83 
  server/src/com/cloud/vm/UserVmManagerImpl.java 5891056 
  server/src/com/cloud/vm/VirtualMachineManagerImpl.java e0647bb 
  server/src/com/cloud/vm/snapshot/VMSnapshotManager.java PRE-CREATION 
  server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java PRE-CREATION 
  server/src/com/cloud/vm/snapshot/dao/VMSnapshotDao.java PRE-CREATION 
  server/src/com/cloud/vm/snapshot/dao/VMSnapshotDaoImpl.java PRE-CREATION 
  server/test/com/cloud/vm/snapshot/VMSnapshotManagerTest.java PRE-CREATION 
  setup/db/create-schema.sql ead98a5 
  ui/css/cloudstack3.css 6519ed0 
  ui/dictionary.jsp 203c9da 
  ui/index.jsp 5481da8 
  ui/scripts/instances.js 5b5ed18 
  ui/scripts/ui/widgets/detailView.js 4b4fbd5 
  ui/scripts/vm_snapshots.js PRE-CREATION 
  vmware-base/src/com/cloud/hypervisor/vmware/mo/HostMO.java a765b42 
  vmware-base/src/com/cloud/hypervisor/vmware/mo/VirtualMachineMO.java cd54127 

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


Testing
-------

simple scenarios including VM Snapshot creation, delete, list and revert.


Thanks,

mice xia

Reply via email to