Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============5833724572659107978=="
MIME-Version: 1.0
Subject: Re: Review Request: Support for full clones on ESX for Cloudstack
 (CS-670)
From: "Venkata Siva Vijayendra Bhamidipati"
 <vijayendra.bhamidipati@citrix.com>
To: "Kelven Yang" <kelven.yang@citrix.com>
Cc: "Sateesh Chodapuneedi" <sateesh.chodapuneedi@citrix.com>,
 "cloudstack" <cloudstack-dev@incubator.apache.org>,
 "Venkata Siva Vijayendra Bhamidipati" <vijayendra.bhamidipati@citrix.com>,
 "Frank Zhang" <frank.zhang@citrix.com>
Date: Mon, 25 Feb 2013 22:48:28 -0000
Message-ID: <20130225224828.1281.8678@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Venkata Siva Vijayendra Bhamidipati" <noreply@reviews.apache.org>
References: <20130223022339.18986.24540@reviews.apache.org>
In-Reply-To: <20130223022339.18986.24540@reviews.apache.org>
Reply-To: "Venkata Siva Vijayendra Bhamidipati"
 <vijayendra.bhamidipati@citrix.com>

--===============5833724572659107978==
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/9439/
-----------------------------------------------------------

(Updated Feb. 25, 2013, 10:48 p.m.)


Review request for cloudstack and Kelven Yang.


Changes
-------

Uploading latest consolidated diff that applies on master.


Description
-------

Please find the diffs for full clone guest VM support for ESX on Cloudstack=
. The diffs do not include unit tests yet. The tests will be uploaded at th=
e earliest.


This addresses bug CS-670.


Diffs (updated)
-----

  core/src/com/cloud/vm/UserVmCloneSettingVO.java PRE-CREATION =

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/Vmware=
Manager.java 445b2f0 =

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/Vmware=
ManagerImpl.java 6b6bf19 =

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/Vmwar=
eResource.java f754c58 =

  server/conf/migration-components.xml 90fbafa =

  server/src/com/cloud/configuration/Config.java 1cea9aa =

  server/src/com/cloud/vm/UserVmManagerImpl.java 6cb3f1a =

  server/src/com/cloud/vm/dao/UserVmCloneSettingDao.java PRE-CREATION =

  server/src/com/cloud/vm/dao/UserVmCloneSettingDaoImpl.java PRE-CREATION =

  server/test/com/cloud/vm/dao/UserVmCloneSettingDaoImplTest.java PRE-CREAT=
ION =

  server/test/com/cloud/vm/dao/UserVmCloneSettingDaoTestConfiguration.java =
PRE-CREATION =

  server/test/resources/CloneSettingDaoTestContext.xml PRE-CREATION =

  setup/db/db/schema-410to420.sql be8e7f5 =


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


Testing
-------

Manual Testing
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D

Creation of guest VMs when the global flag vmware.create.full.clone is set =
to its default value of false will create those guest VMs as linked clones =
(current default behavior without these changes).
Creation of guest VMs when the global flag vmware.create.full.clone is set =
to true will create those guest VMs as full clones.
Appropriate entries are created in the user_vm_clone_setting table of the c=
loud schema to reflect the clone type of the VMs.
Linked clones and full clones can reside together in the same cluster and w=
ork as expected.


Thanks,

Venkata Siva Vijayendra Bhamidipati


--===============5833724572659107978==--