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 4FB38103EC for ; Thu, 20 Jun 2013 05:19:48 +0000 (UTC) Received: (qmail 25743 invoked by uid 500); 20 Jun 2013 05:19:47 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 25694 invoked by uid 500); 20 Jun 2013 05:19:47 -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 25682 invoked by uid 500); 20 Jun 2013 05:19:46 -0000 Delivered-To: apmail-incubator-cloudstack-dev@incubator.apache.org Received: (qmail 25672 invoked by uid 99); 20 Jun 2013 05:19:45 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 20 Jun 2013 05:19:45 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id E5B841CCD3A; Thu, 20 Jun 2013 05:19:37 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4115348428809672072==" MIME-Version: 1.0 Subject: Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm From: "Harikrishna Patnala" To: "Abhinandan Prateek" , "Nitin Mehta" Cc: "cloudstack" , "Harikrishna Patnala" , "Prasanna Santhanam" Date: Thu, 20 Jun 2013 05:19:37 -0000 Message-ID: <20130620051937.29623.80065@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Harikrishna Patnala" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/11910/ X-Sender: "Harikrishna Patnala" References: <20130619110102.29623.55231@reviews.apache.org> In-Reply-To: <20130619110102.29623.55231@reviews.apache.org> Reply-To: "Harikrishna Patnala" --===============4115348428809672072== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11910/ ----------------------------------------------------------- (Updated June 20, 2013, 5:19 a.m.) Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. Changes ------- Updated patch after resolving merge conflicts Description ------- CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dy= namic scaling of vm = CLOUDSTACK-3042 - handle Scaling up of vm memory/CPU based on the presence = of XS tools in the template This should also take care of updation of VM after XS tools are installed i= n the vm and set memory values accordingly to support dynamic scaling after= stop start of VM This addresses bugs CLOUDSTACK-2987 and CLOUDSTACK-3042. Diffs (updated) ----- api/src/com/cloud/template/VirtualMachineTemplate.java cedc793 = api/src/com/cloud/vm/VirtualMachine.java ce9add6 = api/src/org/apache/cloudstack/api/ApiConstants.java 12e5097 = api/src/org/apache/cloudstack/api/BaseUpdateTemplateOrIsoCmd.java 6fd9773 = api/src/org/apache/cloudstack/api/command/user/iso/RegisterIsoCmd.java 28= 4d553 = api/src/org/apache/cloudstack/api/command/user/template/RegisterTemplateC= md.java c9da0c2 = api/src/org/apache/cloudstack/api/command/user/vm/UpdateVMCmd.java 286028= 3 = api/src/org/apache/cloudstack/api/response/TemplateResponse.java ed933ff = api/src/org/apache/cloudstack/api/response/UserVmResponse.java 5b71ba2 = core/src/com/cloud/agent/api/ScaleVmCommand.java b361485 = engine/schema/src/com/cloud/storage/VMTemplateVO.java e643d75 = engine/schema/src/com/cloud/vm/VMInstanceVO.java fbe03dc = engine/storage/src/org/apache/cloudstack/storage/image/TemplateEntityImpl= .java 4d162bb = plugins/hypervisors/xen/src/com/cloud/hypervisor/XenServerGuru.java 8c38a= 69 = plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResou= rceBase.java 5e8283a = plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56= FP1Resource.java 8e37809 = server/src/com/cloud/api/ApiResponseHelper.java 7ffa30f = server/src/com/cloud/api/query/dao/UserVmJoinDaoImpl.java f9877ab = server/src/com/cloud/api/query/vo/UserVmJoinVO.java c97d71a = server/src/com/cloud/hypervisor/HypervisorGuruBase.java 1ad9a1f = server/src/com/cloud/server/ManagementServerImpl.java cfc8333 = server/src/com/cloud/storage/TemplateProfile.java 0b55f1f = server/src/com/cloud/template/TemplateAdapter.java 9a2d877 = server/src/com/cloud/template/TemplateAdapterBase.java 0940d3e = server/src/com/cloud/vm/UserVmManagerImpl.java e8ea024 = server/src/com/cloud/vm/VirtualMachineManagerImpl.java 5814075 = server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 8715c9e = setup/db/db/schema-410to420.sql c782234 = Diff: https://reviews.apache.org/r/11910/diff/ Testing ------- Tested locally Thanks, Harikrishna Patnala --===============4115348428809672072==--