Difference between SSHing to vm in basic zone vs advanced zone

2013-07-16 Thread Gaurav Aradhye
Hi all, What is the difference between SSHing to vm in basic zone vs advanced zone? Typically the test cases which try to get ssh client of Virtual Machine internally, fail in basic zone while they work fine in advanced zone. (Observed with KVM) It keeps on retrying the ssh connection and eventual

Re: Exception when inserting Event

2013-07-16 Thread Chiradeep Vittal
On 7/16/13 11:18 PM, "Chip Childers" wrote: >I'd suggest shortening your event type literals. > +1 > >On Tue, Jul 16, 2013 at 1:45 PM, Soheil Eizadi >wrote: > >> I have code to delete the Plugin device from the system, when I try to >> execute this code I get an exception before my Command g

Review Request 12633: Fixed argument list to the format string

2013-07-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12633/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: 17975 Repository:

Review Request 12632: CLOUDSTACK-3551: Fix value over flow due to type conversion

2013-07-16 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12632/ --- Review request for cloudstack and Nitin Mehta. Bugs: CLOUDSTACK-3551 Reposito

Re: Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-16 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12324/#review23243 --- api/src/org/apache/cloudstack/api/command/user/template/CreateTempl

Review Request 12630: change all instances of self.account_1/account_2.account. to self.account_1/account_2.

2013-07-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12630/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: 17984 Repository:

Review Request 12631: fix for LB tests with netscaler

2013-07-16 Thread Sowmya Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12631/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: CLOUDSTACK-3582 R

Re: Review Request 12251: Incorporating the review comments given by Alena and Sheng

2013-07-16 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12251/ --- (Updated July 17, 2013, 5:52 a.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-16 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12324/#review23242 --- api/src/org/apache/cloudstack/api/command/user/template/CreateTempl

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-16 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23239 --- server/src/com/cloud/alert/AlertManagerImpl.java

Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm

2013-07-16 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12358/#review23238 --- plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenSe

Review Request 12629: CLOUDSTACK-3492: [Regression]Update Resource Count for an account is updating the primary storage incorretly

2013-07-16 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12629/ --- Review request for cloudstack and Devdeep Singh. Bugs: CLOUDSTACK-3492 Reposi

Review Request 12625: Fix typo in TestAddMultipleNSDiffZone

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

Re: Thank you Apache community !

2013-07-16 Thread Pranav Saxena
Thank you everyone for your wishes. They definitely mean a lot to me and I hope I am able to contribute even more till the end of time :) . Please feel free to add/contact me on Linked In - http://in.linkedin.com/pub/pranav-saxena/13/904/98 or my personal email ID's (psb...@gmail.com /prana...@and

Re: Review Request 12109: HAProxy maxconn is not configurable

2013-07-16 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12109/ --- (Updated July 17, 2013, 3:41 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request 12273: Cloudstack-2150 DB table entries of phisical network is not proper.Shows Duplicate entries Cloudstack-2980 Adding a VLAN range that overlaps with two existing ranges results

2013-07-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12273/#review23235 --- Ship it! Ship It! - Abhinandan Prateek On July 10, 2013, 5:43 p.

Re: Review Request 12622: Cloudstack-3313 Unable to deploy a vm in ipv6 network.

2013-07-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12622/#review23234 --- Ship it! Ship It! - Abhinandan Prateek On July 17, 2013, 2:06 a.

Re: Review Request 12109: HAProxy maxconn is not configurable

2013-07-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12109/#review23233 --- Ship it! Ship It! - Abhinandan Prateek On July 17, 2013, 2:07 a.

Review Request 12623: CLOUDSTACK-2328: Linux native VXLAN support on KVM hypervisor

2013-07-16 Thread Toshiaki Hatano
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12623/ --- Review request for cloudstack, Alena Prokharchyk, Chiradeep Vittal, Murali Reddy

Re: Review Request 12109: HAProxy maxconn is not configurable

2013-07-16 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12109/ --- (Updated July 17, 2013, 2:07 a.m.) Review request for cloudstack, Abhinandan Pr

Review Request 12622: Cloudstack-3313 Unable to deploy a vm in ipv6 network.

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

Re: Thank you Apache community !

2013-07-16 Thread Ian Duffy
Good Luck Pranav. Thank you for all your advice with UI modifications.

Re: Review Request 12591: CLOUDSTACK-3566 Not able to create snapshot for KVM host -- backup snapshot fails

2013-07-16 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12591/#review23222 --- Ship it! Ship It! - edison su On July 16, 2013, 10:37 p.m., Fang

Re: Review Request 12612: Fix NPE when attaching vmware-tools iso to guest VM on ESX

2013-07-16 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12612/ --- (Updated July 17, 2013, 12:39 a.m.) Review request for cloudstack, Chip Childer

Re: Review Request 12612: Fix NPE when attaching vmware-tools iso to guest VM on ESX

2013-07-16 Thread Venkata Siva Vijayendra Bhamidipati
> On July 17, 2013, 12:24 a.m., Sheng Yang wrote: > > plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java, > > line 875 > > > > > > need proper indent Hmm, thanks for catching that.

Re: Review Request 12612: Fix NPE when attaching vmware-tools iso to guest VM on ESX

2013-07-16 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12612/#review23217 --- plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareSto

Review Request 12612: Fix NPE when attaching vmware-tools iso to guest VM on ESX

2013-07-16 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12612/ --- Review request for cloudstack, Chip Childers, edison su, Kelven Yang, Sateesh Ch

Review Request 12611: Patch to fix stale cookie and refresh issue in UCS plugin

2013-07-16 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12611/ --- Review request for cloudstack, Animesh Chaturvedi and Frank Zhang. Bugs: CLOUDS

[ACS42] Release Status Update: Two weeks to code freeze

2013-07-16 Thread Animesh Chaturvedi
Folks We are now just 2 weeks from ACS 4.2 code freeze on 7/29. For the past week there has been vibrant activity in filing and fixing issues. 120 new issues were created and 140 issues were resolved. We managed to contain open defects to less than 400 but blockers and critical continue to r

Re: Review Request 12566: Fix NPE when getting resource count

2013-07-16 Thread Sheng Yang
> On July 16, 2013, 11:13 p.m., Sheng Yang wrote: > > Ship It! Committed to 4.2 and MASTER. - Sheng --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12566/#review23208

Re: Review Request 12566: Fix NPE when getting resource count

2013-07-16 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12566/#review23208 --- Ship it! Ship It! - Sheng Yang On July 16, 2013, 1:43 a.m., Venk

Re: chmod u+x createtmplt.sh?

2013-07-16 Thread Chip Childers
Packages perhaps? On Jul 16, 2013, at 6:38 PM, Donal Lafferty wrote: > I can't see were createtmplt.sh gets turned into an executable script. > > Where does this occur? > > E.g. > # mvn clean install > # mvn -P developer -pl developer -Ddeploydb > # export MAVEN_OPTS="-XX:MaxPermSize=256m -Xmx1g

chmod u+x createtmplt.sh?

2013-07-16 Thread Donal Lafferty
I can't see were createtmplt.sh gets turned into an executable script. Where does this occur? E.g. # mvn clean install # mvn -P developer -pl developer -Ddeploydb # export MAVEN_OPTS="-XX:MaxPermSize=256m -Xmx1g" # mvn -pl :cloud-client-ui jetty:run Wait for system to settle, and... root@mgmtse

Review Request 12591: CLOUDSTACK-3566 Not able to create snapshot for KVM host -- backup snapshot fails

2013-07-16 Thread Fang Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12591/ --- Review request for cloudstack and edison su. Bugs: CLOUDSTACK-3566 Repository

RE: Thank you Apache community !

2013-07-16 Thread Musayev, Ilya
Pranav, Its sad to see you leave us, I wish you all best in your endeavors, thank you for the work you've done. Regards ilya > -Original Message- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: Tuesday, July 16, 2013 3:36 PM > To: dev@cloudstack.apache.org > Subject: Re: T

RE: Review board inaccessible

2013-07-16 Thread Fang Wang
The review board is working now Thanks a lot! -Original Message- From: Fang Wang [mailto:fang.w...@citrix.com] Sent: Tuesday, July 16, 2013 3:28 PM To: dev@cloudstack.apache.org Subject: RE: Review board inaccessible Yes, the review board is still down. I just tried and wanted to post a

Re: [ACS411] Fix for CLOUDSTACK-3540

2013-07-16 Thread Sheng Yang
Done. Thanks! --Sheng On Mon, Jul 15, 2013 at 6:30 PM, Chip Childers wrote: > The branch is free for you to push stuff into. Feel free. > > On Jul 15, 2013, at 9:06 PM, Sheng Yang wrote: > > > Hi, > > > > Please cherry-pick the following commit from MASTER(it's already in > MASTER > > and 4.2

RE: Review board inaccessible

2013-07-16 Thread Fang Wang
Yes, the review board is still down. I just tried and wanted to post a review When will it be fixed? Thanks, -Fang -Original Message- From: Nguyen Anh Tu [mailto:ng.t...@gmail.com] Sent: Tuesday, July 16, 2013 11:01 AM To: dev@cloudstack.apache.org Subject: Review board inaccessible Hi

Re: Modify Site-to-Site VPN interface to include ID/FQQN capabilites

2013-07-16 Thread Sheng Yang
Hi Ian, I've implemented the S2S vpn, but at this memory I am focus on 4.2 release, so I am afraid I don't have much time for new feature. But you can file a Jira ticket anyway, and probably somebody interested in it in the community would take it. Java is not hard. :) Please make sure you would

Re: Yet anther KVM blocker: https://issues.apache.org/jira/browse/CLOUDSTACK-3538

2013-07-16 Thread Wei ZHOU
Edison, Thanks a lot . I will have a look at it tomorrow. -Wei 2013/7/16 Edison Su > The doc says, the agent command can block forever: > http://wiki.libvirt.org/page/Qemu_guest_agent > > Warning: Like any monitor command, GA commands can block indefinitely > > > -Original Message- >

Re: [jira] [Commented] (CLOUDSTACK-2725) [Automation] Re-attaching volume on VM failing in KVM

2013-07-16 Thread Wei ZHOU
Marcus, Nice. I think it is because of the inconsistent between hypervisor and virtual machines , for instance, hypervisor thinks volume is attached but in fact not. It happens in some cases: (1) attach volume when VM starting. To avoid it, it is better to allow attach/detach only when vm is runn

Modify Site-to-Site VPN interface to include ID/FQQN capabilites

2013-07-16 Thread Ian Service
After working with a few different hardware VPN gateways in a few different configurations I've found there's a relatively simple component missing to allow us to easily support those other configurations. I've been able to get the networks to connect with some modifications in the VPC router VM,

RE: Yet anther KVM blocker: https://issues.apache.org/jira/browse/CLOUDSTACK-3538

2013-07-16 Thread Edison Su
The doc says, the agent command can block forever: http://wiki.libvirt.org/page/Qemu_guest_agent Warning: Like any monitor command, GA commands can block indefinitely > -Original Message- > From: Wei ZHOU [mailto:ustcweiz...@gmail.com] > Sent: Tuesday, July 16, 2013 1:39 PM > To: dev@clo

Re: In-Development Release Naming

2013-07-16 Thread Daan Hoogland
if this is how jira works I would like to put the ugly +1 to that. If we don't go for code names we can still work like that by using a new label 'master' all the time and renaming that one before creating a new one. As I said, I like the code naming principle. regards, On Mon, Jul 15, 2013 at 1

Re: Yet anther KVM blocker: https://issues.apache.org/jira/browse/CLOUDSTACK-3538

2013-07-16 Thread Wei ZHOU
I am wondering what causes passCmdLine hanging forever? -Wei 2013/7/16 Edison Su > Due to bug fix for CLOUDSTACK-2823, we added a fix in the KVM code, to > retry passCmdLine several times during system vm start. > Somehow, during the retry, passCmdLine(the patchviasocket.pl itself) is > hangin

Yet anther KVM blocker: https://issues.apache.org/jira/browse/CLOUDSTACK-3538

2013-07-16 Thread Edison Su
Due to bug fix for CLOUDSTACK-2823, we added a fix in the KVM code, to retry passCmdLine several times during system vm start. Somehow, during the retry, passCmdLine(the patchviasocket.pl itself) is hanging forever, thus blocking startcommand, even the vm is already started. The possible fix:

Re: Thank you Apache community !

2013-07-16 Thread Sebastien Goasguen
All the best Pranav, I am sure you will do well in your new endeavors ! On Jul 16, 2013, at 3:27 PM, Wei ZHOU wrote: > Thanks for your great job! > Best wishes for your further study and life. > -Wei > > > 2013/7/16 Pranav Saxena > >> Hello community , >> >> Let me take the opportunity t

Re: Thank you Apache community !

2013-07-16 Thread Wei ZHOU
Thanks for your great job! Best wishes for your further study and life. -Wei 2013/7/16 Pranav Saxena > Hello community , > > Let me take the opportunity to thank each one you in the Apache CloudStack > community for your valuable contributions. It has been a pleasure working > with you all and

Re: Thank you Apache community !

2013-07-16 Thread Min Chen
Nice to work with Pranav. You are a great contributor, best wishes to your advanced study. -min On 7/16/13 11:27 AM, "Prachi Damle" wrote: >Good work Pranav! All the best for your future. > >-Prachi > >-Original Message- >From: Pranav Saxena [mailto:psb...@gmail.com] >Sent: Tuesday, Ju

RE: Thank you Apache community !

2013-07-16 Thread Prachi Damle
Good work Pranav! All the best for your future. -Prachi -Original Message- From: Pranav Saxena [mailto:psb...@gmail.com] Sent: Tuesday, July 16, 2013 8:09 AM To: dev@cloudstack.apache.org; us...@cloudstack.apache.org; market...@cloudstack.apache.org Subject: Thank you Apache community

Re: Thank you Apache community !

2013-07-16 Thread Mike Tutkowski
Thanks for all your help, Pranav! On Tue, Jul 16, 2013 at 9:08 AM, Pranav Saxena wrote: > Hello community , > > Let me take the opportunity to thank each one you in the Apache CloudStack > community for your valuable contributions. It has been a pleasure working > with you all and unfortunately

Re: Review board inaccessible

2013-07-16 Thread David Nalley
Yes - Infra is aware of the issue and working on addressing it. --David On Tue, Jul 16, 2013 at 2:00 PM, Nguyen Anh Tu wrote: > Hi, > > Review board is seemly inaccessible, isn't it? > > -- > > N.g.U.y.e.N.A.n.H.t.U

Review board inaccessible

2013-07-16 Thread Nguyen Anh Tu
Hi, Review board is seemly inaccessible, isn't it? -- N.g.U.y.e.N.A.n.H.t.U

Re: Exception when inserting Event

2013-07-16 Thread Chip Childers
I'd suggest shortening your event type literals. On Tue, Jul 16, 2013 at 1:45 PM, Soheil Eizadi wrote: > I have code to delete the Plugin device from the system, when I try to > execute this code I get an exception before my Command gets invoked in > system code that tries to create an event. I

Exception when inserting Event

2013-07-16 Thread Soheil Eizadi
I have code to delete the Plugin device from the system, when I try to execute this code I get an exception before my Command gets invoked in system code that tries to create an event. I debugged this to the event table underlying SQL definition. I wanted to know what the best practice was for C

RE: Vmware - change virtual hardware version

2013-07-16 Thread Sateesh Chodapuneedi
Hi Fabrice, Changing the default config version at cluster level should do. Did you try with a new template registered after modifying the cluster level setting? Subsequently deployed VMs should follow. CloudStack does clone the VMs from already deployed template. All the VMs inherit the hardwa

RE: Another KVM blocker issue: https://issues.apache.org/jira/browse/CLOUDSTACK-2725

2013-07-16 Thread Edison Su
I updated the steps on JIRA to reproduce this issue without using cloudstack, pretty sure it's kvm's bug, maybe only on certain specific version. > -Original Message- > From: Wei ZHOU [mailto:ustcweiz...@gmail.com] > Sent: Tuesday, July 16, 2013 4:11 AM > To: dev@cloudstack.apache.org >

Re: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Joe Brockmeier
On Tue, Jul 16, 2013, at 10:37 AM, Alex Huang wrote: > I updated the wiki because several people asked me to post my eclipse > preference file which includes 180 column widths. There's also a follow > up mail thread after I checked it in. The column width was not > questioned. I can send out a s

any idea about upgrade xenserver 6.0.2 to 6.1 with CloudStack 4.1 management server

2013-07-16 Thread William Jiang
Hi, We just migrated our cloudstack management server (use basic networking) from 3.0.2 to 4.1, now we want upgrade cloudstack host xenserver from 6.0.2 to 6.1. We have 4 host servers in the same pool and 30 VMs on it, and we also need migrate them without downtime. So we choose xencenter "rolli

RE: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Donal Lafferty
> -Original Message- > From: Joe Brockmeier [mailto:j...@zonker.net] > Sent: 16 July 2013 3:02 PM > To: dev@cloudstack.apache.org > Subject: Re: [discuss] 80 char max length leads to illegiable source > > On Tue, Jul 16, 2013, at 08:02 AM, Donal Lafferty wrote: > > Can we loosen/ignore thi

Vmware - change virtual hardware version

2013-07-16 Thread Fabrice Brazier
Hi Folks, With VMware ESXi 5.1, CloudStack is provisioning all VMs with virtual hardware version “vmx-09”. Is there a way to use version 7 or 8? Even forcing the vSphere cluster to use the version 8, all VMs are deployed from CloudStack with the version vmx-09. Thanks for your help, Fabrice

RE: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Alex Huang
I updated the wiki because several people asked me to post my eclipse preference file which includes 180 column widths. There's also a follow up mail thread after I checked it in. The column width was not questioned. I can send out a separate proposal if needed but supposedly developers are a

Re: Specifying Storage Provider for Primary Storage Creation/Registration

2013-07-16 Thread La Motta, David
John, UI requirements? A reusable wizard for plugin implementers to leverage, and this has already been brought up by Chris Suich in another thread. That would allow for the same look and feel as other wizards in CS, but what is gathered in the wizard screens is entirely up to the vendor. I

Re: UI Plugin Internationalization

2013-07-16 Thread SuichII, Christopher
Brian (and all), I've come back to looking into this feature and want to discuss it a bit more. I like the idea of putting each plugins dictionary into it's own namespace, but that may make utilizing it a bit more difficult. Currently, strings are automagically localized through the _l() functi

Thank you Apache community !

2013-07-16 Thread Pranav Saxena
Hello community , Let me take the opportunity to thank each one you in the Apache CloudStack community for your valuable contributions. It has been a pleasure working with you all and unfortunately I have quit my $dayjob for pursuing my higher studies in the US which means that I might not get suf

Re: Weekly IRC Meeting Tomorrow at 17:00 UTC

2013-07-16 Thread Daan Hoogland
I refrained from lurking for a couple of weeks, but do definitely have the intention to be a better citizen. On Tue, Jul 16, 2013 at 4:16 PM, Joe Brockmeier wrote: > Hi all, > > A reminder that we have the weekly CloudStack meeting in > #cloudstack-meeting at 17:00 UTC on Wednesdays. We scuttle

Re: NULL entry in folder colum in volumes table

2013-07-16 Thread Mike Tutkowski
When I was developing the SolidFire plug-in for 4.2, I noticed the folder column wasn't being used, as well. I repurposed it for SolidFire plug-in volumes to store the ID of the iSCSI volume (LUN) on our SAN. On Tue, Jul 16, 2013 at 8:39 AM, Sanjay Tripathi wrote: > Thanks for the reply Edison

RE: NULL entry in folder colum in volumes table

2013-07-16 Thread Sanjay Tripathi
Thanks for the reply Edison, I still have a query though... if CS has more than one primary storage in a cluster and admin wants to know about location of a volume i.e. in which primary storage this volume is in, than as per current behavior, admin has to search in all the primary storage by usi

Re: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Joe Brockmeier
On Tue, Jul 16, 2013, at 08:53 AM, Alex Huang wrote: > Donal, > > Our coding convention says 180. It was discussed about a week or two > back. So, I am surprised that it's already been changed. I noticed the comment when I was doing the weekly news - but it was buried in an email with the subjec

RE: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Donal Lafferty
My mistake. I'll have to update my style rules. DL > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: 16 July 2013 2:54 PM > To: dev@cloudstack.apache.org > Subject: RE: [discuss] 80 char max length leads to illegiable source > > Donal, > > Our coding conven

Weekly IRC Meeting Tomorrow at 17:00 UTC

2013-07-16 Thread Joe Brockmeier
Hi all, A reminder that we have the weekly CloudStack meeting in #cloudstack-meeting at 17:00 UTC on Wednesdays. We scuttled the meeting last week because we had low turnout, so I hope to see more folks turn up tomorrow! Best, jzb -- Joe Brockmeier j...@zonker.net Twitter: @jzb http://www.diss

[EVENTS] Xen Project user summit

2013-07-16 Thread Sebastien Goasguen
The deadline for proposals to the Xen Project user summit is near: http://blog.xen.org/index.php/2013/06/03/call-for-proposals-opens-for-xen-project-user-summit-and-xen-project-developer-summit/

RE: Invoking an API from an API plugin

2013-07-16 Thread Alex Huang
Chris, If it is then it was written wrong. You can change it to public and send in a patch. --Alex > -Original Message- > From: SuichII, Christopher [mailto:chris.su...@netapp.com] > Sent: Monday, July 15, 2013 9:23 PM > To: > Cc: Alex Huang; Chiradeep Vittal; Min Chen > Subject: Re:

Re: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Joe Brockmeier
On Tue, Jul 16, 2013, at 08:02 AM, Donal Lafferty wrote: > Can we loosen/ignore this rule when readability becomes an issue? Alex raised this issue last week: http://markmail.org/message/z2hzz7efujgmhfnh I didn't see many responses to that particular part of his email, though. Personally, I th

RE: [discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Alex Huang
Donal, Our coding convention says 180. It was discussed about a week or two back. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Coding+conventions --Alex > -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: Tuesday, July 16, 2013 6:02 AM > To:

Re: Review board is not accessible.

2013-07-16 Thread Harikrishna Patnala
Me too not able to access Review board. Fix for a blocker bug is in pending so attaching the patch to the bug ticket itself. https://issues.apache.org/jira/browse/CLOUDSTACK-3551 Can someone review my patch and commit. Thanks, Harikrishna On 16-Jul-2013, at 3:46 PM, Bharat Kumar wrote: > Can

Re: Specifying Storage Provider for Primary Storage Creation/Registration

2013-07-16 Thread John Burwell
David, Are you able to share more specific UI requirements? For example, would it be possible to describe a basic flow through the UI you require? My thoughts around the property bag are that it would be available at the lowest levels of the storage layer-- passed into each driver operation

Re: Specifying Storage Provider for Primary Storage Creation/Registration

2013-07-16 Thread La Motta, David
John, The generic properties would fit the bill for a vendor that just wants to provide the metadata and let the traditional primary-storage workflow take its course. But, a form with additional fields may not be enough for a particular implementation, so having the flexibility to display a cu

Re: codestyle checker?

2013-07-16 Thread Chip Childers
On Tue, Jul 16, 2013 at 01:22:22PM +, Donal Lafferty wrote: > Turns out faster to use CheckStyle, Eclipse and ACS code style file. > > See http://dlafferty.blogspot.co.uk/2013/07/apache-cloudstack-java-coding.html Awesome post Donal! > > > -Original Message- > > From: Donal Lafferty

RE: codestyle checker?

2013-07-16 Thread Donal Lafferty
Turns out faster to use CheckStyle, Eclipse and ACS code style file. See http://dlafferty.blogspot.co.uk/2013/07/apache-cloudstack-java-coding.html > -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: 08 July 2013 10:15 AM > To: dev@cloudstack.apache.org

[discuss] 80 char max length leads to illegiable source

2013-07-16 Thread Donal Lafferty
This issue appeared when I used StyleCheck and Eclipse to find and fix my coding convention violations automatically. Details at http://dlafferty.blogspot.co.uk/2013/07/apache-cloudstack-java-coding.html We inherit the 80 char line limit inherited from the Java Programming Style, and it makes

Re: [DISCUSS] coding convention for method - and class length

2013-07-16 Thread Daan Hoogland
I am all for re-oping discussions on the ten commandments, but let's stick to method and class level for now. On Tue, Jul 16, 2013 at 12:19 PM, Donal Lafferty wrote: > Hmm. The original comment was "maximum length of 80 lines per method". > > Did you want to start a new thread covering max ch

Re: [ACS42] Three weeks to code freeze

2013-07-16 Thread Jayapal Reddy Uradi
You are right Abhi. Enable RPS/RFS (CLOUDSTACK-2433) changes doesn't involve template changes. Thanks, jayapal On 16-Jul-2013, at 5:18 PM, Abhinandan Prateek wrote: >> Given the debate on system template changes in last few days of 4.1 > requiring big testing effort and potential >regression,

Re: [ACS42] Three weeks to code freeze

2013-07-16 Thread Abhinandan Prateek
>Given the debate on system template changes in last few days of 4.1 requiring big testing effort and potential >regression, I would like to see that as community we lock down system templates for 4.2 pretty soon. If any changes >are needed we should call it out now and get them resolved. As per m

Re: Another KVM blocker issue: https://issues.apache.org/jira/browse/CLOUDSTACK-2725

2013-07-16 Thread Wei ZHOU
Can someone log into the VM , and check if acpiphp is already loaded? root@NEW:~# lsmod |grep acpiphp acpiphp24231 0 if not, please run "modprobe acpiphp", and try again -Wei 2013/7/16 Marcus Sorensen > I'll try to track it down tonight. As far as anyone knows this is 4.2 > spe

Re: How to fix libvirt storage pool refresh issue?

2013-07-16 Thread Wido den Hollander
On 07/16/2013 12:34 PM, Wei ZHOU wrote: I agree with Wido. So how do you propose how we refresh the pool? Right now we call storagePoolRefresh() every time we do a getStoragePool() to the LibvirtStorageAdapter, which is called on a very regular basis by all KVM agents for various tasks. W

Re: How to fix libvirt storage pool refresh issue?

2013-07-16 Thread Wei ZHOU
I agree with Wido. Moreover, the file lock will cause performane degrade of VM deployment. -Wei 2013/7/16 Wido den Hollander > On 07/16/2013 12:27 AM, Marcus Sorensen wrote: > >> I'm ok with a symptom fix on our end, if the root cause is in >> Libvirt we can't do much about that. This is

RE: [DISCUSS] coding convention for method - and class length

2013-07-16 Thread Donal Lafferty
Hmm. The original comment was "maximum length of 80 lines per method". Did you want to start a new thread covering max characters per line? > -Original Message- > From: Isaac Chiang [mailto:isaacchi...@gmail.com] > Sent: 16 July 2013 11:12 AM > To: dev@cloudstack.apache.org > Subject: Re

Review board is not accessible.

2013-07-16 Thread Bharat Kumar
Can some one restart it .

Re: How to fix libvirt storage pool refresh issue?

2013-07-16 Thread Wido den Hollander
On 07/16/2013 12:27 AM, Marcus Sorensen wrote: I'm ok with a symptom fix on our end, if the root cause is in Libvirt we can't do much about that. This is the sort of patch that tends to get pulled into the regular update cycle of the distributions, so unless there's more to it and it's not a

Re: [DISCUSS] coding convention for method - and class length

2013-07-16 Thread Isaac Chiang
+1 for the maximum length per line I also agree with Wido that around 120 characters will be more fit current needs. Regards Isaac

Re: GlusterFS QEMU libgfapi

2013-07-16 Thread Wido den Hollander
On 07/15/2013 06:38 PM, John Skinner wrote: I dug a little bit deeper and found that it IS a supported storage type of NETFS, as a valid poor format type (see below from libvirt.org). Now, not being familiar with how CloudStack handles storage; I was think that under cloud-plugin-hypervisoer-kv

Re: [DISCUSS] coding convention for method - and class length

2013-07-16 Thread Wido den Hollander
Hi, On 07/16/2013 10:59 AM, Daan Hoogland wrote: LS, I am looking at a method 200 lines long. I've seen more of those. Yes, that is pretty long. I would propose a maximum length of 80 lines per method and There was a recent discussion about this, but I wasn't able to find it that quickl

RE: [DISCUSS] coding convention for method - and class length

2013-07-16 Thread Donal Lafferty
What is the method? DL > -Original Message- > From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] > Sent: 16 July 2013 10:00 AM > To: dev@cloudstack.apache.org > Subject: [DISCUSS] coding convention for method - and class length > > LS, > > I am looking at a method 200 lines long.

[DISCUSS] coding convention for method - and class length

2013-07-16 Thread Daan Hoogland
LS, I am looking at a method 200 lines long. I've seen more of those. I would propose a maximum length of 80 lines per method and A maximum of 10 methods per class, not including getters and setters of course. I think this would help with the readability of the source base and thus the growth o

Re: Git Push Summary

2013-07-16 Thread Pranav Saxena
Thanks Isaac for creating this branch. I'll keep an eye on the commits you make your here . Once the code for handling nested query API calls are ready , you can merge it into master. @Jessica - I have had conversations with Isaac and he is in the process of re-factoring the code for dedicated res