Ok. Vote cancelled. Wido, is the 4.1 branch corrected now? I'll be back near my laptop tonight. Just want to know if there is work to do, or can I just re-spin a forth round?
On May 26, 2013, at 6:01 AM, Wido den Hollander <w...@widodh.nl> wrote: > Hi Chip, > > I'm sorry, but I'm going to have to vote -1 on this one. > > It's my bad, but it seems that I made a mistake on the Debian packaging side. > See this commit: > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=dc822a83d77830281402175b4a57b25b7e3b180a > > I just verified, cloudstack-setup-agent has variables in it like > @AGENTSYSCONFDIR@ which would render the tool useless. > > I see this commit is already in the 4.1 branch, but it isn't in the commit > you are voting on (873c19). > > I also found a problem with the AWSAPI package, which is in master in commit > 28f7a216d8bf4da29a45cb76e5c28ee568ae1984 > > I already cherry-picked that one to 4.1 since it's only touching packaging > and not code. > > Other then the packaging I'm happy with this code. I obviously wasn't able to > do a full QA on my own, but the tests I've done all work, which include: > * Deploying instances > * Adding RBD storage > * Attaching RBD volumes > > With the packaging resolved I'd vote +1, but for now it's -1 (binding). > > Wido > > On 05/24/2013 07:41 PM, Chip Childers wrote: >> Hi All, >> >> I've created a 4.1.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.1 >> Commit: 873c1972c7bbe337cee2000a09451d14ebbcb728 >> >> List of changes: >> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.1 >> >> Source release (checksums and signatures are available at the same >> location): >> https://dist.apache.org/repos/dist/dev/cloudstack/4.1.0/ >> >> PGP release keys (signed using A99A5D58): >> https://dist.apache.org/repos/dist/release/cloudstack/KEYS >> >> Testing instructions are here: >> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure >> >> 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) >