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

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

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: 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: 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: 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 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 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 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 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 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: 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: 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: 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: 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: [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: 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: 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

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: [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

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 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

[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

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

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
> On July 29, 2013, 3:59 a.m., Jenkins Cloudstack.org wrote: > > Review 13008 failed the build test : FAILURE > > The url of build cloudstack-master-with-patch #65 is : > > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/65/ > > Venkata Siva Vijayendra Bhamidipati wrote: > Th

Re: deploydb jdbc/mysql connector error?

2013-07-29 Thread Ian Duffy
Removing the scope line from 183 of the root pom.xml also seemed to get it going again. On 29 July 2013 21:41, Vijayendra Bhamidipati wrote: > Thanks Ian! I'll give that a try. > > Regards, > Vijay > > -Original Message- > From: Ian Duffy [mailto:i...@ianduffy.ie] > Sent: Monday, July 29,

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/#review24171 --- 1) DataCenterVnetDaoImpl.java deleteRange method. List vnets - thi

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

2013-07-29 Thread Kelven Yang
> On July 29, 2013, 3:59 a.m., Jenkins Cloudstack.org wrote: > > Review 13008 failed the build test : FAILURE > > The url of build cloudstack-master-with-patch #65 is : > > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/65/ > > Venkata Siva Vijayendra Bhamidipati wrote: > Th

RE: [DISCUSS] maven dependencies...

2013-07-29 Thread Donal Lafferty
Could you also have a look at ./client/tomcatconf/log4j-cloud.xml.in? It looks like it was updated in the refactor. AFAIK, in '', {1.} should be {1}. See "Conversion Character" chart under http://logging.apache.org/log4j/1.2/apidocs/org/apache/log4j/PatternLayout.html There's a lot of paper w

RE: deploydb jdbc/mysql connector error?

2013-07-29 Thread Vijayendra Bhamidipati
Thanks Ian! I'll give that a try. Regards, Vijay -Original Message- From: Ian Duffy [mailto:i...@ianduffy.ie] Sent: Monday, July 29, 2013 1:27 PM To: dev@cloudstack.apache.org Subject: Re: deploydb jdbc/mysql connector error? 1f64354ec830db6e30c17ec6e4a6d8c33f2e10da few days old. On 29

Re: Review Request 12912: Fix for packaging defect CLOUDSTACK-2133

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

Re: Review Request 12912: Fix for packaging defect CLOUDSTACK-2133

2013-07-29 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12912/#review24168 --- Ship it! Ship It! - edison su On July 29, 2013, 6:19 p.m., Rayee

Re: Review Request 13003: Adding packaging support for RHEL 5 and 6.2

2013-07-29 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13003/#review24167 --- Ship it! Ship It! - edison su On July 28, 2013, 1:14 a.m., Rayee

Re: Review Request 12912: Fix for packaging defect CLOUDSTACK-2133

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

Re: deploydb jdbc/mysql connector error?

2013-07-29 Thread Ian Duffy
1f64354ec830db6e30c17ec6e4a6d8c33f2e10da few days old. On 29 July 2013 21:20, Vijayendra Bhamidipati wrote: > I see! What commit # are you on? I'd rather use the same one to get by for > now until this is fixed. > > Regards, > Vijay > > -Original Message- > From: Ian Duffy [mailto:i...@i

RE: deploydb jdbc/mysql connector error?

2013-07-29 Thread Vijayendra Bhamidipati
I see! What commit # are you on? I'd rather use the same one to get by for now until this is fixed. Regards, Vijay -Original Message- From: Ian Duffy [mailto:i...@ianduffy.ie] Sent: Monday, July 29, 2013 1:16 PM To: dev@cloudstack.apache.org Subject: Re: deploydb jdbc/mysql connector er

Re: deploydb jdbc/mysql connector error?

2013-07-29 Thread Ian Duffy
I'm having this issue too when trying to do deploydb on master. Its working fine on a slightly older branch. On 29 July 2013 20:53, Vijayendra Bhamidipati wrote: > Prachi and I had a look at this, I should have pasted the entire output of > the deploydb command - in Prachi's env we see that the

RE: CloudStack Compile Error

2013-07-29 Thread Donal Lafferty
FYI, this happens when you accidentally install different versions of Java for compile and runtime. E.g. Compile with javac... root@mgmtserver:~/github/cshv3/tools/apidoc# which javac /usr/bin/javac root@mgmtserver:~/github/cshv3/tools/apidoc# ls -al /usr/bin/javac lrwxrwxrwx 1 root root 23 J

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-07-29 Thread Toshiaki Hatano
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12849/#review24161 --- I tested this patch locally, VLAN isolated advanced zone with 1 KVM

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

2013-07-29 Thread Jonathan Creasy
Indeed. :) - Original Message - From: "Donal Lafferty" To: "dev@cloudstack.apache.org" , "Jonathan Creasy" , "cloudstack" Sent: Monday, July 29, 2013 2:34:59 PM Subject: RE: Review Request 13025: Creating is mis-spelled in CreateFirewallRuleCmd Ironically, there's no hyphen in 'missp

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

2013-07-29 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/#review24160 --- Review 13025 PASSED the build test The url of build cloudstack-maste

RE: deploydb jdbc/mysql connector error?

2013-07-29 Thread Vijayendra Bhamidipati
Prachi and I had a look at this, I should have pasted the entire output of the deploydb command - in Prachi's env we see that the mysql-connector.jar is getting copied over to the deps/ directory, but I see this warning in the beginning when I run deploydb: [WARNING] Some problems were encount

RE: Review Request 13025: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread Donal Lafferty
Ironically, there's no hyphen in 'misspelled' :) > -Original Message- > From: Jonathan Creasy [mailto:nore...@reviews.apache.org] On Behalf Of > Jonathan Creasy > Sent: 29 July 2013 8:19 PM > To: cloudstack; Jonathan Creasy > Subject: Review Request 13025: Creating is mis-spelled in > Cre

deploydb jdbc/mysql connector error?

2013-07-29 Thread Vijayendra Bhamidipati
After rebasing my code against the latest master/4.2, I'm running into this error when doing a " mvn -e -P developer -pl developer -Ddeploydb " across master and 4.2 : > WARNING: Provided file does not exist: /root/mywork/cloudstack/asf/latest/cloudstack/developer/developer-prefill.sql.

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

2013-07-29 Thread Jonathan Creasy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/ --- (Updated July 29, 2013, 7:20 p.m.) Review request for cloudstack. Summary (up

Review Request 13025: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread Jonathan Creasy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/ --- Review request for cloudstack. Bugs: CLOUDSTACK-3915 Repository: cloudstack-g

Re: 4.2 and 4.1 non-oss

2013-07-29 Thread David Nalley
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 there a reason we don't have non-oss >> packages(deb,rpm) building of 4.1 and 4.2 on jenkins?

Re: 4.2 and 4.1 non-oss

2013-07-29 Thread David Nalley
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 there a reason we don't have non-oss > packages(deb,rpm) building of 4.1 and 4.2 on jenkins? >

Re: Review Request 12912: Fix for packaging defect CLOUDSTACK-2133

2013-07-29 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12912/#review24157 --- Review 12912 PASSED the build test The url of build cloudstack-maste

Can you take a look at CLOUDSTACK-3229?

2013-07-29 Thread Min Chen
Hi John, Tom filed a blocker defect https://issues.apache.org/jira/browse/CLOUDSTACK-3229 related to s3xen plugin? It is related to https://issues.apache.org/jira/browse/CLOUDSTACK-2583 you recently fixed. Can you please take a look at the bug? I am not clear about any extra step required for

Re: Review Request 12912: Fix for packaging defect CLOUDSTACK-2133

2013-07-29 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12912/ --- (Updated July 29, 2013, 6:19 p.m.) Review request for cloudstack, Koushik Das,

RE: Create Affinity Group API

2013-07-29 Thread Prachi Damle
I cannot reproduce this using latest 4.2. Updated the bug with details of tests. Is this only seen when using automation tests? Prachi From: Girish Shilamkar [mailto:gir...@clogeny.com] Sent: Monday, July 29, 2013 12:04 AM To: dev@cloudstack.apache.org; Prachi Damle Cc: Prasanna Santhanam Subjec

4.2 and 4.1 non-oss

2013-07-29 Thread Ian Duffy
Hi, Just wondering, is there a reason we don't have non-oss packages(deb,rpm) building of 4.1 and 4.2 on jenkins?

Re: Review Request 13020: CLOUDSTACK-3252 An instance deployed using explicit or implicit dedication doesn't generate a usage even

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

Re: Review Request 13022: Cloudstack-3910 Upgrade from 3.0.6 to 4.2 is failing with DB exceptions.

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

RE: [DISCUSS] maven dependencies...

2013-07-29 Thread Alex Huang
Thanks for pointing that out. I think the latest fixed this. --Alex > -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: Monday, July 29, 2013 10:45 AM > To: dev@cloudstack.apache.org > Subject: RE: [DISCUSS] maven dependencies... > > Is it correct for

RE: [DISCUSS] maven dependencies...

2013-07-29 Thread Donal Lafferty
Is it correct for dependencies to appear twice in the base pom.xml? E.g. mysql-connector-java > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: 26 July 2013 11:10 PM > To: dev@cloudstack.apache.org > Subject: [DISCUSS] maven dependencies... > > Everyone, > >

Network Problem

2013-07-29 Thread Thomas Schneider
Hello, I have setup cloudstack and it work fine, I can launch VM etc... But the problem is I can only SSH to the VM from the host on which she is running. When I am in the vm I can ping exernal network and download package to setup lamp for example. I'm in basic network mode My config is: mgmt s

uuid lost on xenserver?

2013-07-29 Thread Daan Hoogland
LS, while trying an update path from 4.0.2 to 4.1.1 I get the appended exceptions. It does not seem to have to do with the upgrade itself as after starting a 4.0 management server with the old database the problem persists. Can anyone tell me what this means? thanks, Daan 2013-07-29 18:59:43,114

Review Request 13022: Cloudstack-3910 Upgrade from 3.0.6 to 4.2 is failing with DB exceptions.

2013-07-29 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13022/ --- Review request for cloudstack, Abhinandan Prateek and Kishan Kavala. Bugs: Clou

Becareful when cherry-picking from any branch to master...

2013-07-29 Thread Alex Huang
I've been working on master to place compilation boundaries between our components. Fixes include changes in pom.xml to include specific and accurate project dependencies. When cherry-picking to master, please do not assume your code will just work. Always compile before checking in. Also if

Re: Review Request 12941: [GSoC] refactor gre controller

2013-07-29 Thread Nguyen Anh Tu
Hi Hugo, Do you want to keep sdn.ovs.controller param in configuration? I have already replaced it by checking whether physical-network-isolation-type is GRE or not. But I just see two threads using this param and I don't know what they run for. It's placed at configure() method of OvsTunnelManage

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
> On July 29, 2013, 3:59 a.m., Jenkins Cloudstack.org wrote: > > Review 13008 failed the build test : FAILURE > > The url of build cloudstack-master-with-patch #65 is : > > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/65/ This patch is for 4.2 and has been tested on 4.2 as des

Re: Review Request 12995: [GSoC] Add midterm report to docs

2013-07-29 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12995/#review24142 --- Ship it! committed to master 79419d43736d49a86c2f7595bc8c62ddc1f993

Review Request 13020: CLOUDSTACK-3252 An instance deployed using explicit or implicit dedication doesn't generate a usage even

2013-07-29 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13020/ --- Review request for cloudstack, Devdeep Singh and Kishan Kavala. Bugs: 3252 Re

Re: Review Request 12991: GSOC2013 meng midterm report

2013-07-29 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12991/#review24138 --- Ship it! commited to master with 2238e2bbc5863ce926341b2d91fd4bc8bc

Re: Review Request 13016: Fix for CLOUDSTACK-3848: Usage events for dynamic scaling of cpu/ram operation on VM

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

Re: Review Request 13001: Automation: Adding test cases for CPU limits from Limit Resources.

2013-07-29 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13001/ --- (Updated July 29, 2013, 2:54 p.m.) Review request for cloudstack, Girish Shilam

Re: Review Request 13013: remove the duplicate entries of the data-server from the /etc/hosts

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

Re: Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

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

Re: Review Request 13012: createipAlias.sh/deleteipAliash.sh should not be allowed with Isolated Guest Networks

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

Re: Review Request 13015: CLOUDSTACK-1961: Common issues found in English OS with EN-US standard Keyboard.

2013-07-29 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13015/#review24130 --- Review 13015 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12942: CLOUDSTACK-3454: Fix test_portable_publicip

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12942/#review24129 --- Ship it! - Prasanna Santhanam On July 25, 2013, 12:50 p.m., Giris

Re: Review Request 12942: CLOUDSTACK-3454: Fix test_portable_publicip

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

Re: Review Request 12942: CLOUDSTACK-3454: Fix test_portable_publicip

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

Re: Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

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

Re: Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13014/#review24126 --- Ship it! Ship It! - Prasanna Santhanam On July 29, 2013, 12:39 p

Re: Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

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

Re: Review Request 13016: Fix for CLOUDSTACK-3848: Usage events for dynamic scaling of cpu/ram operation on VM

2013-07-29 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13016/#review24123 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Review Request 13016: Fix for CLOUDSTACK-3848: Usage events for dynamic scaling of cpu/ram operation on VM

2013-07-29 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13016/ --- Review request for cloudstack and Kishan Kavala. Bugs: CLOUDSTACK-3848 Reposi

Re: Review Request 13015: CLOUDSTACK-1961: Common issues found in English OS with EN-US standard Keyboard.

2013-07-29 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13015/#review24121 --- Cannot apply the patch because of whitespace errors. Can you fix it

Re: Review Request 12997: usage event is not generated for volume resize

2013-07-29 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12997/#review24120 --- UsageServer also has to support volume resize event. It is not suffi

Re: Review Request 12997: usage event is not generated for volume resize

2013-07-29 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12997/#review24119 --- Ship it! Committed to 4.2 and master. Kindly close the request. -

Re: Review Request 12994: CLOUDSTACK-3216 Logs in the Software router are not being rotated

2013-07-29 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12994/#review24116 --- Ship it! Committed to 4.2 and master. Kindly close the request. -

Re: Review Request 12942: CLOUDSTACK-3454: Fix test_portable_publicip

2013-07-29 Thread Murali Reddy
> On July 25, 2013, 1:28 p.m., Jenkins Cloudstack.org wrote: > > Review 12942 failed the build test : UNSTABLE > > The url of build cloudstack-master-with-patch #38 is : > > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/38/ test changes are fine. but looks like there is assumpt

Re: Review Request 12996: Unable to Resize Volume (kvm, vmware)

2013-07-29 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12996/#review24118 --- Ship it! Committed to 4.2 and master. Kindly close the request. -

Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

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

Review Request 13015: CLOUDSTACK-1961: Common issues found in English OS with EN-US standard Keyboard.

2013-07-29 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13015/ --- Review request for cloudstack, Devdeep Singh and Fang Wang. Bugs: CLOUDSTACK-19

Re: Review Request 13012: createipAlias.sh/deleteipAliash.sh should not be allowed with Isolated Guest Networks

2013-07-29 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13012/#review24114 --- Ship it! Ship It! - Abhinandan Prateek On July 29, 2013, 11:21 a

Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

2013-07-29 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13014/ --- Review request for cloudstack and Prasanna Santhanam. Repository: cloudstack-gi

Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

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

Re: Review Request 12934: Tests for egress firewall rules for advance zone

2013-07-29 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12934/#review24101 --- test/integration/component/test_egress_fw_rules.py

Re: Review Request 12999: [GSoC] Add mid-term report

2013-07-29 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12999/#review24112 --- Ship it! applied to master with fd32c33d6d278ec5913bb62b6828adb9450

Checking grep/sed commands in shell and python script files

2013-07-29 Thread Wei ZHOU
Guys, There are many shell/python script files in cloudstack which contain grep or sed commands (which may return greedy results) . Some issues are because of these files, for instance CLOUDSTACK-3540, CLOUDSTACK-3533, CLOUDSATCK-3843. We need to check all the script files in patches/ and scripts

Re: Review Request 13013: remove the duplicate entries of the data-server from the /etc/hosts

2013-07-29 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13013/ --- (Updated July 29, 2013, 12:17 p.m.) Review request for cloudstack and Abhinanda

Re: Review Request 13013: remove the duplicate entries of the data-server from the /etc/hosts

2013-07-29 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13013/#review24111 --- Ship it! Ship It! - Abhinandan Prateek On July 29, 2013, 11:39 a

Review Request 13013: remove the duplicate entries of the data-server from the /etc/hosts

2013-07-29 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13013/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: Cloudstack-3882 R

Review Request 13012: createipAlias.sh/deleteipAliash.sh should not be allowed with Isolated Guest Networks

2013-07-29 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13012/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: Cloudstack-2622 R

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 29, 2013, 11:07 a.m.) Review request for cloudstack, daan Hooglan

  1   2   >