incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kishan Kavala" <Kishan.Kav...@citrix.com>
Subject Re: Review Request: Pick newer system VM template for vmware in latest cloudstack release.
Date Fri, 24 Aug 2012 09:38:45 GMT

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



server/src/com/cloud/upgrade/dao/Upgrade2214to30.java
<https://reviews.apache.org/r/6702/#comment23144>

    to get current version, query should be 
    select version from version order by id desc limit 1;



server/src/com/cloud/upgrade/dao/Upgrade2214to30.java
<https://reviews.apache.org/r/6702/#comment23145>

    Vmware template upgrade code should go into Upgrade302to40.
    In Upgrade2214to30 for Vmware case, template upgrade can be removed completely. Older
versions (< 302) won't have this code anyway. No need to selectively pick template.


- Kishan Kavala


On Aug. 21, 2012, 1:23 a.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6702/
> -----------------------------------------------------------
> 
> (Updated Aug. 21, 2012, 1:23 a.m.)
> 
> 
> Review request for cloudstack and Kishan Kavala.
> 
> 
> Description
> -------
> 
> Modifying reference to system VM template for vmware for the 2.2.14 to 3.0 upgrade path
so that it picks up the burbank template if upgrading from 2.2.14 to 3.0.5 or beyond, and
the acton template if upgrading from 2.2.14 to 3.0.[0-4].
> 
> 
> This addresses bug CS-16075.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/maint/Version.java 184b9a6 
>   server/src/com/cloud/upgrade/dao/Upgrade2214to30.java 43e2c00 
> 
> Diff: https://reviews.apache.org/r/6702/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>


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