cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nitin Mehta" <nitin.me...@citrix.com>
Subject Re: Review Request 12937: CLOUDSTACK-3779: Support System vm scaling on VMWare
Date Mon, 29 Jul 2013 04:43:14 GMT

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



engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
<https://reviews.apache.org/r/12937/#comment47925>

    Why do u need the java code for this ? This could be done by the sql query right



server/src/com/cloud/server/ManagementServerImpl.java
<https://reviews.apache.org/r/12937/#comment47927>

    Why is this required ?


- Nitin Mehta


On July 26, 2013, 8:41 a.m., Harikrishna Patnala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12937/
> -----------------------------------------------------------
> 
> (Updated July 26, 2013, 8:41 a.m.)
> 
> 
> Review request for cloudstack and Nitin Mehta.
> 
> 
> Bugs: CLOUDSTACK-3637 and CLOUDSTACK-3779
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> CLOUDSTACK-3779: Support System vm scaling on VMWare
> 
> Marked the system template new system template as dynamicallyScalable
>    - handled upgrade case
>    - moved "dynamicallyScalable" flag to vm_instance table from user_vm_details to support
dynamic scaling of system vm
> 
> This patch depends on previous review request https://reviews.apache.org/r/12759/ . please
review and commit accordingly
> 
> 
> Diffs
> -----
> 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java d4b7b6d 
>   engine/schema/src/com/cloud/vm/VMInstanceVO.java e1d2892 
>   engine/schema/src/com/cloud/vm/dao/UserVmDao.java b4f9991 
>   engine/schema/src/com/cloud/vm/dao/UserVmDaoImpl.java 1c11563 
>   plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java
2cc592d 
>   server/src/com/cloud/hypervisor/HypervisorGuruBase.java 6d368bd 
>   server/src/com/cloud/server/ManagementServerImpl.java 7ac1254 
>   server/src/com/cloud/vm/UserVmManagerImpl.java ae12523 
>   server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 44a703d 
>   setup/db/db/schema-410to420.sql 82ca403 
> 
> Diff: https://reviews.apache.org/r/12937/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message