Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 0065C114A8 for ; Mon, 5 May 2014 18:54:44 +0000 (UTC) Received: (qmail 29032 invoked by uid 500); 5 May 2014 18:54:42 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 28985 invoked by uid 500); 5 May 2014 18:54:42 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 28974 invoked by uid 99); 5 May 2014 18:54:42 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 05 May 2014 18:54:42 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id B28C11D752B; Mon, 5 May 2014 18:54:35 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============2672244006505020278==" MIME-Version: 1.0 Subject: Review Request 21087: Remove hard-coding for platform emulators in Xen Server From: "Amogh Vasekar" To: "Nitin Mehta" Cc: "Amogh Vasekar" , "cloudstack" Date: Mon, 05 May 2014 18:54:35 -0000 Message-ID: <20140505185435.1482.85624@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Amogh Vasekar" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/21087/ X-Sender: "Amogh Vasekar" Reply-To: "Amogh Vasekar" X-ReviewRequest-Repository: cloudstack-git --===============2672244006505020278== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21087/ ----------------------------------------------------------- Review request for cloudstack and Nitin Mehta. Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6358 https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6358 Repository: cloudstack-git Description ------- As a part of supporting dynamic guest OS defined by user, removing the hard-coded dependencies. This patch is for XenServer Diffs ----- api/src/com/cloud/agent/api/to/VirtualMachineTO.java bbd83852abf840ab5d93f53ad0c71e61491b2dda core/src/com/cloud/agent/api/VMSnapshotBaseCommand.java d2abd54e0407ab597636d9c9d5a596e55f987108 engine/storage/snapshot/src/org/apache/cloudstack/storage/vmsnapshot/DefaultVMSnapshotStrategy.java e86bd23b0500f25268a603a7c41e8e00d18957d2 engine/storage/snapshot/test/src/VMSnapshotStrategyTest.java 9eaa7e19c134caf0633bb9099c5f858075fddba7 engine/storage/src/org/apache/cloudstack/storage/helper/HypervisorHelperImpl.java b41c1fa14ac77cffa53ce5ae2aec6d9bea9cd649 plugins/hypervisors/xen/src/com/cloud/hypervisor/XenServerGuru.java 89e4ab5d94b61cc6acdefc8102da4a33dbbf23b5 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java ad3797306c92d80d929a1e09b46b471231baa6c0 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpOssResource.java 4a69d3aececd718cdb7257d3807225c429b73823 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpServer16Resource.java 8cb7997305f1344429c5b256bea35878137bb5a3 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpServerResource.java cc623bfab61f732a4310e3c6c93a51bc36a463b9 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java 3dc527e93d82733823f4e8fddb242d698c21629f plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56Resource.java a78ed819bdeb4878850989915a7280b0639e3ab6 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56SP2Resource.java 9e7d5dd09d0c1a1d18c6c348ce098218933d18ed plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer600Resource.java 021b9cbd4dc7ed4a5041c69aa55f29daa732f52b plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer602Resource.java 5c9bfbb5952008b332900cd7af21671f7dc465a5 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer610Resource.java 0c3e1e9660abdbfee1dcfe58c62145cbe0e06c9b plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620Resource.java c10dc1e90cbce2ff41cc62e2120f5e23931a4207 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer620SP1Resource.java 4f04961e35742d7e846cec47725551d67b704bfc plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java 40143646f7744e05e2ca009d99d53973ceaf58ee Diff: https://reviews.apache.org/r/21087/diff/ Testing ------- 1. Local testing on XenServer 6.0.2 2. Successfully got up system VMs 3. Successfully created a CentOS VM 4. Created VM snapshot and reverted to snapshot 5. Manipulated guest_os_hypervisor table for mapping, and checked for correct propagation of platform emulator. Thanks, Amogh Vasekar --===============2672244006505020278==--