----------------------------------------------------------- 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