You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Harikrishna Patnala <ha...@citrix.com> on 2013/06/04 06:23:07 UTC

Review Request: Fix for CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates

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

Review request for cloudstack and Kishan Kavala.


Description
-------

CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates 

Added fix to update systemvm templates for the hypervisors Xenserver, VMware, KVM, Hyperv, LXC. And also modifying image_data_store_id to 1 for all the SYSTEM/BUILT in templates including the newly registered system vm templates.


This addresses bug CLOUDSTACK-2728.


Diffs
-----

  engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 16ab05c 

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


Testing
-------

Tested locally by checking the upgrade from 4.1 to 4.2 with newly registered systemvm template.


Thanks,

Harikrishna Patnala


Re: Review Request: Fix for CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11618/#review21475
-----------------------------------------------------------


Commit 9fe7846d72e401720e1dcbce52d021e2646429f1 in branch refs/heads/master from Kishan Kavala <ki...@cloud.com>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=9fe7846 ]

CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates


- ASF Subversion and Git Services


On June 4, 2013, 4:23 a.m., Harikrishna Patnala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11618/
> -----------------------------------------------------------
> 
> (Updated June 4, 2013, 4:23 a.m.)
> 
> 
> Review request for cloudstack and Kishan Kavala.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates 
> 
> Added fix to update systemvm templates for the hypervisors Xenserver, VMware, KVM, Hyperv, LXC. And also modifying image_data_store_id to 1 for all the SYSTEM/BUILT in templates including the newly registered system vm templates.
> 
> 
> This addresses bug CLOUDSTACK-2728.
> 
> 
> Diffs
> -----
> 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 16ab05c 
> 
> Diff: https://reviews.apache.org/r/11618/diff/
> 
> 
> Testing
> -------
> 
> Tested locally by checking the upgrade from 4.1 to 4.2 with newly registered systemvm template.
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>


Re: Review Request: Fix for CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates

Posted by Kishan Kavala <Ki...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11618/#review21476
-----------------------------------------------------------

Ship it!


commit 9fe7846d72e401720e1dcbce52d021e2646429f1

- Kishan Kavala


On June 4, 2013, 4:23 a.m., Harikrishna Patnala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11618/
> -----------------------------------------------------------
> 
> (Updated June 4, 2013, 4:23 a.m.)
> 
> 
> Review request for cloudstack and Kishan Kavala.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates 
> 
> Added fix to update systemvm templates for the hypervisors Xenserver, VMware, KVM, Hyperv, LXC. And also modifying image_data_store_id to 1 for all the SYSTEM/BUILT in templates including the newly registered system vm templates.
> 
> 
> This addresses bug CLOUDSTACK-2728.
> 
> 
> Diffs
> -----
> 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 16ab05c 
> 
> Diff: https://reviews.apache.org/r/11618/diff/
> 
> 
> Testing
> -------
> 
> Tested locally by checking the upgrade from 4.1 to 4.2 with newly registered systemvm template.
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>