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



server/src/com/cloud/vm/UserVmManagerImpl.java
<https://reviews.apache.org/r/9439/#comment35291>

    Better to rename it to start with Capital letter as this is name of data 
type.


- Sateesh Chodapuneedi


On Feb. 15, 2013, 12:33 a.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9439/
> -----------------------------------------------------------
> 
> (Updated Feb. 15, 2013, 12:33 a.m.)
> 
> 
> Review request for cloudstack and Kelven Yang.
> 
> 
> 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 the 
> earliest.
> 
> 
> This addresses bug CS-670.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/vm/UserVmManagerImpl.java 19887ff 
>   setup/db/db/schema-40to410.sql 7f00441 
> 
> Diff: https://reviews.apache.org/r/9439/diff/
> 
> 
> Testing
> -------
> 
> Manual Testing
> ==============
> 
> 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 
> cloud schema to reflect the clone type of the VMs.
> Linked clones and full clones can reside together in the same cluster and 
> work as expected.
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>

Reply via email to