On Fri, May 24, 2013 at 09:49:12AM +0000, Nitin Mehta wrote:
> Would this not be a problem for folks who are already on 4.0 ?

Corrected in 4.1 branch:

commit c559c40c3178873177b9f46c46e6c60b5598f2eb
Author: Chip Childers <chip.child...@gmail.com>
Date:   Fri May 24 17:06:02 2013 +0100

    CLOUDSTACK-528: Adding a corrective update to remove the extra space
    characters that exist after running older versions of
    schema-302to40.sql
        
    Signed-off-by: Chip Childers <chip.child...@gmail.com>

Cherry-picked into master as well.

> 
> On 13/05/13 9:22 PM, "Chip Childers" <chipchild...@apache.org> wrote:
> 
> >
> >-----------------------------------------------------------
> >This is an automatically generated e-mail. To reply, visit:
> >https://reviews.apache.org/r/11088/#review20489
> >-----------------------------------------------------------
> >
> >Ship it!
> >
> >
> >In 4.1:
> >
> >commit 5ccdf7dc4ad86145633124b9540aadfbfc423f6f
> >Author: Wei Zhou <w.z...@leaseweb.com>
> >Date:   Mon May 13 16:50:29 2013 +0100
> >
> >    CLOUDSTACK-528: Correct a bunch of mistakes in the 3.0.x to 4.0 db
> >    script
> >    Signed-off-by: Chip Childers <chip.child...@gmail.com>
> >
> >- Chip Childers
> >
> >
> >On May 13, 2013, 3:38 p.m., Wei Zhou wrote:
> >> 
> >> -----------------------------------------------------------
> >> This is an automatically generated e-mail. To reply, visit:
> >> https://reviews.apache.org/r/11088/
> >> -----------------------------------------------------------
> >> 
> >> (Updated May 13, 2013, 3:38 p.m.)
> >> 
> >> 
> >> Review request for cloudstack and Chip Childers.
> >> 
> >> 
> >> Description
> >> -------
> >> 
> >> There are some mistakes in configuration table.
> >> fixed them by this patch.
> >> 
> >> 
> >> This addresses bug CLOUDSTACK-528.
> >> 
> >> 
> >> Diffs
> >> -----
> >> 
> >>   setup/db/db/schema-302to40.sql a947ac1
> >> 
> >> Diff: https://reviews.apache.org/r/11088/diff/
> >> 
> >> 
> >> Testing
> >> -------
> >> 
> >> Nicolas has tested it.
> >> We also need an additional patch for the following new issue (Advanced
> >>Network with Security Groups).
> >> 
> >> 
> >> Thanks,
> >> 
> >> Wei Zhou
> >> 
> >>
> >
> 
> 

Reply via email to