Re: [VOTE] Apache CloudStack 4.1.0 (third round)

2013-05-26 Thread Wido den Hollander

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)



Re: [VOTE] Apache CloudStack 4.1.0 (third round)

2013-05-26 Thread Chip Childers
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)



Re: [VOTE] Apache CloudStack 4.1.0 (third round)

2013-05-26 Thread Chip Childers
On Sun, May 26, 2013 at 12:00:19PM +0200, Wido den Hollander 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

I got myself to my laptop.  I see that these corrections are in 4.1 now.
I'll respin a release candidate right now.


[VOTE] Apache CloudStack 4.1.0 (third round)

2013-05-24 Thread Chip Childers
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)