Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-12 Thread Pradeep Soundararajan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/ --- (Updated March 12, 2013, 6:27 a.m.) Review request for cloudstack, Chip

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-12 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17735 --- Ship it! Applied to 4.1: commit

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-07 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17569 --- I'm waiting for resolution on the question asked before applying

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-07 Thread Chiradeep Vittal
+1 on being part of default cloudstack install. On 3/7/13 12:07 PM, Chip Childers chip.child...@sungard.com wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17569

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17472 --- But awsapi should be an optional component to install, right? -

RE: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread Pradeep Soundararajan
I have also noticed cloudstack-awsapi*.rpm is getting generated as part of packaging which is irrespective of awsapi build. That is the reason I have included the same as part of MS install. “mvn -P awsapi package -Dsystemvm” Thanks, Pradeep S From: Prasanna Santhanam

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread Chip Childers
On Wed, Mar 06, 2013 at 12:23:09PM +, Prasanna Santhanam wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17472

RE: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread Hugo Trippaers
I automatically assumed it's an optional component, but I don't think we actually discussed this. It's also optional in the maven compile/build I've noticed. I'm certainly not against it being part of the main management server, but the dependencies are not really aligned yet. So it's going to

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread Chip Childers
On Wed, Mar 06, 2013 at 03:35:17PM +, Hugo Trippaers wrote: I automatically assumed it's an optional component, but I don't think we actually discussed this. It's also optional in the maven compile/build I've noticed. I'm certainly not against it being part of the main management

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-06 Thread prasanna
On 6 March 2013 21:13, Chip Childers chip.child...@sungard.com wrote: On Wed, Mar 06, 2013 at 03:35:17PM +, Hugo Trippaers wrote: I automatically assumed it's an optional component, but I don't think we actually discussed this. It's also optional in the maven compile/build I've noticed.