Master not running

2013-07-29 Thread Donal Lafferty
Build master from scratch using pull from this morning. Master builds, but does not run using 'mvn -pl :cloud-client-ui jetty:run' due using to the following issues: https://issues.apache.org/jira/browse/CLOUDSTACK-3919 https://issues.apache.org/jira/browse/CLOUDSTACK-3918 (work around: delet

[VOTE] Whitespace changes to bylaws.mdtext

2013-07-29 Thread Noah Slater
Hi, I'd like to make several changes to our by-laws, but before I continue, I've prepared a changset that tidies up whitespace and hard wraps. This will make it easier to edit. The only non-whitespace change my patch makes is to correct two spelling errors: Transparancy -> Transparency desicion

Re: [VOTE] Whitespace changes to bylaws.mdtext

2013-07-29 Thread David Nalley
On Mon, Jul 29, 2013 at 5:55 PM, Noah Slater wrote: > Hi, > > I'd like to make several changes to our by-laws, but before I continue, > I've prepared a changset that tidies up whitespace and hard wraps. This > will make it easier to edit. > > The only non-whitespace change my patch makes is to cor

Re: Review Request 12905: Multiple VLAN range API need to accept a list rather than "add" or "remove" per command

2013-07-29 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12905/#review24182 --- Also the response format is incorrect. The vlan ranges have to be se

RE: [VOTE] Whitespace changes to bylaws.mdtext

2013-07-29 Thread Alex Huang
+1 > -Original Message- > From: Noah Slater [mailto:nsla...@apache.org] > Sent: Monday, July 29, 2013 2:55 PM > To: CloudStack, Dev > Subject: [VOTE] Whitespace changes to bylaws.mdtext > > Hi, > > I'd like to make several changes to our by-laws, but before I continue, I've > prepared a

Review Request 13047: VMSnapshot chainSize check

2013-07-29 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13047/ --- Review request for cloudstack and mice xia. Bugs: https://issues.apache.org/jir

Re: help submitting to review board.

2013-07-29 Thread Thomas O'Dowd
Hi all, I'm wondering what the next steps on this[1] review is. I think someone needs to merge the patch in before I close but I'm not sure what the flow is. Maybe I need to close as submitted first? Can someone advise? Thanks, Tom. [1] https://reviews.apache.org/r/12963/ On Fri, 2013-07-26 at

Re: help submitting to review board.

2013-07-29 Thread Min Chen
The person who reviewed and approved should commit the patch for you. Thanks -min On 7/29/13 5:40 PM, "Thomas O'Dowd" wrote: >Hi all, > >I'm wondering what the next steps on this[1] review is. I think someone >needs to merge the patch in before I close but I'm not sure what the >flow is. Maybe

RE: [DISCUSS] Compatibility issue between network plugins and hypervisors

2013-07-29 Thread Toshiaki Hatano
Daan, In my opinion, it's responsibility of the developer to provide information and to stop Ops from failing over cliff. Obviously Dev knows how code works (sometime not :), but Ops doesn't. So, if Dev deny to help Ops, how can Ops setup the cloud in proper way? When someone enhance hypervisor a

Re: Review Request 13047: VMSnapshot chainSize check

2013-07-29 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13047/#review24186 --- Review 13047 failed the build test : FAILURE The url of build clouds

Re: Review Request 13004: Bug:advance zone, create public Network with vlan id specified, but the portgroup is cloud.public.untagged.0.1-vSwitch0 not cloud.public.[vlanid].0.1-vSwitch0

2013-07-29 Thread Jijun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13004/ --- (Updated July 30, 2013, 12:58 a.m.) Review request for cloudstack, daan Hooglan

Re: Network Problem

2013-07-29 Thread Jijun
Hi , Basic Network will setup the security group, and it will deny all the inbound traffic and allow outbound traffic. you should manually add some ingress rules. offical document: http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.2/html/Installation_Guide/security-groups.html O

RE: Physical Network / Guest Network / Service Offering Interactions for Basic Networking

2013-07-29 Thread Soheil Eizadi
Thanks for the replies, I posted the message as I was going to fix it and submit a patch but wanted to know if there were any issues I should be aware of in this area. Actually the problem is not with ConfigurationServerImpl, but with the call to CreateGuestNetwork(). I am actually a little con

Re: Review Request 13047: VMSnapshot chainSize check

2013-07-29 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13047/#review24200 --- the chainSize for a volume is the total size of all nodes, not a sin

Re: Review Request 13008: Fix usage of vm.instancename.flag when generating VM names on ESX hypervisor

2013-07-29 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13008/ --- (Updated July 30, 2013, 3:46 a.m.) Review request for cloudstack, Alex Huang, C

Re: Review Request 12883: DNS value in Zone is not getting set in the Network created for the Zone

2013-07-29 Thread Soheil Eizadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12883/#review24205 --- In researching this more look like there is a code in NetworkManager

Re: Review Request 13025: Bugfix CLOUDSTACK-3915: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/#review24207 --- Commit 238815759664eb2a0e8c000389c976a8b1c5b269 in branch refs/heads

Re: Review Request 13025: Bugfix CLOUDSTACK-3915: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/#review24208 --- Ship it! - Prasanna Santhanam On July 29, 2013, 7:20 p.m., Jonath

Re: Review Request 13025: Bugfix CLOUDSTACK-3915: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/#review24206 --- Commit 650de7998d989c1bc4a6b5a0a98e3fad49bf2247 in branch refs/heads

Re: Review Request 12963: Fix some simple typos in ConfigurationManager

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12963/#review24212 --- Ship it! commit 5552d26872a13e7eab584412f787ae99551df23e (master) c

Re: Review Request 13004: Bug:advance zone, create public Network with vlan id specified, but the portgroup is cloud.public.untagged.0.1-vSwitch0 not cloud.public.[vlanid].0.1-vSwitch0

2013-07-29 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13004/#review24213 --- Ship it! Ship It! - daan Hoogland On July 30, 2013, 12:58 a.m.,

Re: 4.2 and 4.1 non-oss

2013-07-29 Thread Ian Duffy
Cool will add it to my to do list. On 29 July 2013 20:15, David Nalley wrote: > Bah. Autocorrect. Make. > On Jul 29, 2013 3:15 PM, "David Nalley" wrote: > >> Not that I am aware of. Feel free to male it so. >> On Jul 29, 2013 2:11 PM, "Ian Duffy" wrote: >> >>> Hi, >>> >>> Just wondering, is the

Re: Open up ports beyond 80/443/8080 for downloading templates

2013-07-29 Thread Thomas O'Dowd
Thanks Ian. I had a look at this file. It's an easy fix to remove the check from here but it's a general utility function so will also affect other uris... If there is no reason to limit any uri to those ports then I'd like to remove this check and open them up. Interested to hear opinions, Tom.

Re: Open up ports beyond 80/443/8080 for downloading templates

2013-07-29 Thread Prasanna Santhanam
On Tue, Jul 30, 2013 at 03:37:39PM +0900, Thomas O'Dowd wrote: > Thanks Ian. I had a look at this file. It's an easy fix to remove the > check from here but it's a general utility function so will also affect > other uris... If there is no reason to limit any uri to those ports then > I'd like to r

Review Request 13055: Fix to select default vpc offering with internal lb

2013-07-29 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13055/ --- Review request for cloudstack, venkata swamy babu budumuru and Prasanna Santhan

<    1   2