Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
MIME-Version: 1.0
In-Reply-To: <CAC2panpu3WZ06DVfwyU03JB+s13zdOfaAdGDmZUm2PXhf=ZSwA@mail.gmail.com>
References: <CAC2panr-YrsLVWJPFTQHQP4uovRJeWyxUkYz9bA-4Cem5+SWoA@mail.gmail.com>
 <5a7fb1431d2b4781ab89ad26fd0c66dc@IPPEXCH13MB1.ipp.corp> <422068395.800.1468306101305@ox.pcextreme.nl>
 <e2adb450b7ff4c48a08f5e7afecdcccb@IPPEXCH13MB1.ipp.corp> <CAC2panpA+WKeExAJHA0w4EBxb-77RLkDTwJeM2=_Abx6qEatpA@mail.gmail.com>
 <1696729248.855.1468343235268@ox.pcextreme.nl> <CAC2panrYH63vm9YeFNm3jvJOVg=PrG0iF3Yf6FsGzYYs12Hb1Q@mail.gmail.com>
 <5785E5A8.8010906@apache.org> <CAC2panpu3WZ06DVfwyU03JB+s13zdOfaAdGDmZUm2PXhf=ZSwA@mail.gmail.com>
From: Patrick Dube <patrickdube91@gmail.com>
Date: Wed, 20 Jul 2016 10:34:13 -0400
Message-ID: <CA+6jLrS8Y_+j22QXMqSYzELhD9ezx7zY9iswmmppc7-tXsCQ0A@mail.gmail.com>
Subject: Re: [VOTE] Apache Cloudstack 4.9.0 RC1
To: dev@cloudstack.apache.org
Content-Type: multipart/alternative; boundary=94eb2c046c2826ba0c0538121ae5
archived-at: Wed, 20 Jul 2016 14:34:53 -0000

--94eb2c046c2826ba0c0538121ae5
Content-Type: text/plain; charset=UTF-8

Hello all,

I am have found that the VPC router rules are not reapplied correctly if
the router is restarted, which severely breaks the VPC (e.g. you can't
deploy a VM if it's password is set via cloud-init). I have a PR to fix the
restart that I will open today. More details in the PR. Also, if the
userData is not padded correctly, this will prevent subsequent deployments
in the VPC

Regards,


On Mon, Jul 18, 2016 at 2:18 PM, Will Stevens <wstevens@cloudops.com> wrote:

> I have merged the following PRs in preparation for RC2:
> https://github.com/apache/cloudstack/pull/1610
> https://github.com/apache/cloudstack/pull/1609
> https://github.com/apache/cloudstack/pull/1608
> https://github.com/apache/cloudstack/pull/1601
> https://github.com/apache/cloudstack/pull/1595
> https://github.com/apache/cloudstack/pull/1480
> https://github.com/apache/cloudstack/pull/1455
>
> I am doing a 3rd round of tests on the following PRs since they both showed
> the same error on two different CI environments on two consecutive runs:
> https://github.com/apache/cloudstack/pull/1594
> https://github.com/apache/cloudstack/pull/1583
>
> Any feedback on these last two would be appreciated.  I would like to merge
> them before I push the RC2 if possible.
>
> I would like to cut RC2 tonight.
>
> Cheers,
>
> *Will STEVENS*
> Lead Developer
>
> *CloudOps* *| *Cloud Solutions Experts
> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> w cloudops.com *|* tw @CloudOps_
>
> On Wed, Jul 13, 2016 at 2:54 AM, Milamber <milamber@apache.org> wrote:
>
> >
> > Please add this PR on your list. That will fix the spanish localization
> > display. Thanks
> >
> > https://github.com/apache/cloudstack/pull/1583
> >
> > On 12/07/2016 19:17, Will Stevens wrote:
> >
> >> Thank you gentlemen.  I have been following the 1608 PR from afar and I
> >> will review your list Wido.  I will get my CI chugging with some of
> those
> >> PRs today.
> >>
> >> Thx,
> >>
> >> *Will STEVENS*
> >> Lead Developer
> >>
> >> *CloudOps* *| *Cloud Solutions Experts
> >> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> >> w cloudops.com *|* tw @CloudOps_
> >>
> >> On Tue, Jul 12, 2016 at 1:07 PM, Wido den Hollander <wido@widodh.nl>
> >> wrote:
> >>
> >> Hi Will,
> >>>
> >>> These are on my list:
> >>>
> >>> - https://github.com/apache/cloudstack/pull/1610
> >>> - https://github.com/apache/cloudstack/pull/1480
> >>> - https://github.com/apache/cloudstack/pull/1608
> >>>
> >>> Those 3 should fix all the -1 votes and discussion on the list.
> >>>
> >>> Wido
> >>>
> >>> Op 12 juli 2016 om 18:14 schreef Will Stevens <wstevens@cloudops.com>:
> >>>>
> >>>>
> >>>> Thank you, so this RC has officially failed.  Can you guys please
> review
> >>>> those PRs and give me code review so I can be confident with those
> PRs.
> >>>>
> >>> I
> >>>
> >>>> will run testing on those two PRs to make sure things don't break with
> >>>>
> >>> them.
> >>>
> >>>> Is there anything else that needs to be added to the next RC, please
> >>>>
> >>> speak
> >>>
> >>>> now so we can streamline this next RC.
> >>>>
> >>>> Thanks,
> >>>>
> >>>> *Will STEVENS*
> >>>> Lead Developer
> >>>>
> >>>> *CloudOps* *| *Cloud Solutions Experts
> >>>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> >>>> w cloudops.com *|* tw @CloudOps_
> >>>>
> >>>> On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair <slair@ippathways.com>
> >>>>
> >>> wrote:
> >>>
> >>>> Hi all, I vote -1 and would like to see the jdbc:mysql and
> site-to-site
> >>>>> vpn fixed in 4.9.
> >>>>>
> >>>>> https://github.com/apache/cloudstack/pull/1610
> >>>>> https://github.com/apache/cloudstack/pull/1480
> >>>>>
> >>>>> Thanks!
> >>>>> Sean
> >>>>>
> >>>>> -----Original Message-----
> >>>>> From: Wido den Hollander [mailto:wido@widodh.nl]
> >>>>> Sent: Tuesday, July 12, 2016 1:48 AM
> >>>>> To: Sean Lair <slair@ippathways.com>; dev@cloudstack.apache.org
> >>>>> Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1
> >>>>>
> >>>>>
> >>>>> Op 11 juli 2016 om 22:40 schreef Sean Lair <slair@ippathways.com>:
> >>>>>>
> >>>>>>
> >>>>>> Hi all,
> >>>>>>
> >>>>>> One small comment since strongSwan didn't make it into 4.9.  There
> is
> >>>>>>
> >>>>> still a very simple bug in enabling PFS for site-to-site VPNs.  The
> >>>>>
> >>>> code
> >>>
> >>>> checks the Dead Peer Detection (DPD) variable instead of the PFS
> >>>>>
> >>>> variable
> >>>
> >>>> when determining whether or not to enable PFS for the site-to-site
> VPN.
> >>>>>
> >>>>>> Here is the 1-line of code that is broken.  You can see how it
> >>>>>>
> >>>>> refers to
> >>>
> >>>> dpd to set pfs.
> >>>>>
> >>>>>> file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> >>>>>>
> >>>>>> This pull request fixes the issue, but was not merged since we were
> >>>>>>
> >>>>> going to strongSwan.  It would be nice if this bug fix was put into
> >>>>>
> >>>> 4.9.0
> >>>
> >>>> https://github.com/apache/cloudstack/pull/1480
> >>>>>>
> >>>>>> Would it make you a -1 for you without this PR? If so, please vote
> -1
> >>>>>
> >>>> :)
> >>>
> >>>> Wido
> >>>>>
> >>>>> Thanks!
> >>>>>> Sean
> >>>>>>
> >>>>>> -----Original Message-----
> >>>>>> From: Will Stevens [mailto:williamstevens@gmail.com]
> >>>>>> Sent: Wednesday, July 6, 2016 3:52 PM
> >>>>>> To: dev@cloudstack.apache.org
> >>>>>> Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> >>>>>>
> >>>>>> Hi All,
> >>>>>>
> >>>>>> I've created a 4.9.0 release, with the following artifacts up for a
> >>>>>>
> >>>>> vote:
> >>>
> >>>> Git Branch and Commit SH:
> >>>>>>
> >>>>>>
> >>>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> >>>
> >>>> Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> >>>>>>
> >>>>>> I will be updating the Release Notes with the changes in this
> release
> >>>>>>
> >>>>> tomorrow.  If the RC changes, I can adapt the release notes after.
> >>>>>
> >>>>>> Source release (checksums and signatures are available at the same
> >>>>>> location):
> >>>>>> https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> >>>>>>
> >>>>>> PGP release keys (signed using CB818F64):
> >>>>>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> >>>>>>
> >>>>>> Vote will be open for 72 hours.
> >>>>>>
> >>>>>> For sanity in tallying the vote, can PMC members please be sure to
> >>>>>>
> >>>>> indicate "(binding)" with their vote?
> >>>>>
> >>>>>> [ ] +1  approve
> >>>>>> [ ] +0  no opinion
> >>>>>> [ ] -1  disapprove (and reason why)
> >>>>>>
> >>>>>> Thanks,
> >>>>>>
> >>>>>> Will
> >>>>>>
> >>>>>
> >
>

--94eb2c046c2826ba0c0538121ae5--