Re: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-10 Thread Hugo Trippaers
Hey, I’m all for automated solutions. I’m a happy gerrit user on some other projects and quite fond of working with Github pull requests as well. However there is one important point that makes working with those tools work and that is a willingness by the committers to review requests. Both

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22364/#review45200 ---

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22354/#review45201 --- Commit a5902f1db4b14012ecd1d5660257655e9dfe4354 in cloudstack's

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22354/#review45203 --- Commit a1f278e9d47e8029d58df6d3aae13495965ca434 in cloudstack's

[ACS44] CS6850

2014-06-10 Thread Sebastien Goasguen
Daan, Can you cherry-pick: a1f278e9d47e8029d58df6d3aae13495965ca434 from 4.4 forward, it fixes CS-6850 thank you -sebastien

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22354/#review45204 --- Ship it! Ok fine with me, I trust that you will stay on top of

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22354/#review45205 --- Please mark you reviews as 'submitted' and update the JIRA tickets

Re: [GSoC] [CLOUDSTACK-6114] Progress update

2014-06-10 Thread Sebastien Goasguen
On Jun 9, 2014, at 4:20 PM, Ian Duffy i...@ianduffy.ie wrote: Hi All, Started making some fast progress on this. I uploaded the xenserver box[1] to vagrant cloud. This means people can easily get a xenserver VM by executing vagrant init duffy/xenserver vagrant up. I adjusted the

Re: [GSOC2014] Progress Update - week two

2014-06-10 Thread Sebastien Goasguen
On Jun 3, 2014, at 5:30 PM, Darren Brogan brogan...@darrenbrogan.ie wrote: Dif you upload a new version to pypi ? Not yet, this will be done very shortly though. I dont recall if we had the same version number or not. Probably good to increment it so we can upggrade via pip Yup, the

Re: [GSoC] [CLOUDSTACK-6114] Progress update

2014-06-10 Thread Rohit Yadav
Hi Ian, Thanks for sharing, I was trying to do something miserably to make an appliance for testing xenserver/xenproject with ACS. Will try it in the evening. Cheers. On Tue, Jun 10, 2014 at 1:50 AM, Ian Duffy i...@ianduffy.ie wrote: Hi All, Started making some fast progress on this. I

RE: redundant virtual routers for VPCs

2014-06-10 Thread Daan Hoogland
Quite right: - Feasibility experiment plans by Schuberg Philis - Reusable work by Karl - Problems Citrix encountered with the regular redundant router (and how to avoid them) - Work division - (next meeting needed?) Any additions? \DaanH From:

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22364/ --- (Updated June 10, 2014, 8:24 a.m.) Review request for cloudstack, daan

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Rajani Karuturi
On June 10, 2014, 7:07 a.m., Koushik Das wrote: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java, line 5132 https://reviews.apache.org/r/22364/diff/2/?file=605542#file605542line5132 These methods are getting used by the job framework. Check

Re: redundant virtual routers for VPCs

2014-06-10 Thread Hugo Trippaers
I can’t make 15:00, but 22:00 could work. Cheers, Hugo On 10 jun. 2014, at 10:14, Daan Hoogland dhoogl...@schubergphilis.com wrote: Quite right: - Feasibility experiment plans by Schuberg Philis - Reusable work by Karl - Problems Citrix encountered with

[EVENT] CloudStack conference Budapest Nov. 19-21

2014-06-10 Thread sebgoa
Hi folks, There is only 15 days left to submit your talks to CloudStack Conference Budapest. http://events.linuxfoundation.org/events/cloudstack-collaboration-conference-europe/program/cfp This year is going to be super exciting, thanks to your amazing talks :) -Sebastien

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22364/#review45210 --- Ship it! master - b666a1f3a5bcd17663af1675e82759c3ff8cbeb9

unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Rajani Karuturi
Hi Kelven, while fixing some of the issues reported by coverity I encountered some unused private methods in VirtualMachineManagerImpl.java (https://reviews.apache.org/r/22364 ) and removed them. Koushik pointed that they are getting used by the job framework. Looks like these are used by the

Re: Review Request 22232: Fix for test_01_create_volume to use the correct volume name for KVM

2014-06-10 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22232/ --- (Updated June 10, 2014, 10:09 a.m.) Review request for cloudstack and

Re: Review Request 22232: Fix for test_01_create_volume to use the correct volume name for KVM

2014-06-10 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22232/#review45215 --- Ship it! for future review requests, please create a bug in

Re: redundant virtual routers for VPCs

2014-06-10 Thread Daan Hoogland
I would like to have Hugo in the meeting So let's make it 20:00 UTC I would suggest #cloudstack-meeting On Tue, Jun 10, 2014 at 10:29 AM, Hugo Trippaers h...@trippaers.nl wrote: I can’t make 15:00, but 22:00 could work. Cheers, Hugo On 10 jun. 2014, at 10:14, Daan Hoogland

Re: Review Request 21973: CLOUDSTACK-6780: Resolved cleanup issue in portable ip test cases

2014-06-10 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21973/ --- (Updated June 10, 2014, 11:16 a.m.) Review request for cloudstack, Girish

Re: [ACS44] Cherry pick request

2014-06-10 Thread Daan Hoogland
2 picked On Tue, Jun 10, 2014 at 2:09 AM, Amogh Vasekar amogh.vase...@citrix.com wrote: Hi Daan, Request you to please cherry-pick the following two commits to 4.4 branch : 1. ac92b3690304ff224e7e2530ea7d8e39f28a05c3 for CLOUDSTACK-6710 2. a4b401f29f83f2f0b467a9d05b509f951b5a3bca for

Review Request 22414: CLOUDSTACK-6865, CLOUDSTACK-6868: [hyper-v] attach volume of vhd format volume is failing

2014-06-10 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22414/ --- Review request for cloudstack, Devdeep Singh and Rajesh Battala. Bugs:

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22354/#review45217 --- Commit c934e7b052c09df2fae52a005e69951ed0235574 in cloudstack's

Re: [ACS44] CS6850

2014-06-10 Thread Daan Hoogland
On Tue, Jun 10, 2014 at 9:13 AM, Sebastien Goasguen run...@gmail.com wrote: a1f278e9d47e8029d58df6d3aae13495965ca434 is in -- Daan

[ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread sebgoa
The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased to announce that he has accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes and

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Rohit Yadav
Congrats Pierre-Luc! Regards. On Tue, Jun 10, 2014 at 5:15 PM, sebgoa run...@gmail.com wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased to announce that he has accepted. Being a committer allows many

Re: Review Request 22354: CLOUDSTACK-6850: Return cpu cores, cpu speed and memory in listUsageRecords

2014-06-10 Thread Olivier Lemasle
On June 10, 2014, 9:14 a.m., Sebastien Goasguen wrote: Ok fine with me, I trust that you will stay on top of this, test and submit further fixes. I applied your patch to master and 4.4-forward. It should be cherry-picked to 4.4 Nothing done to 4.3. Thanks. I commit myself

Re: Review Request 22356: Fixed few coverity issues reported

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22356/#review45218 --- most of it looks good. thanks for great work. Have a closer look

Re: hackathon CCCEU14

2014-06-10 Thread Daan Hoogland
Hello everybody, Do we still want to have a Hackathon next CCC? we need to plan for the resources (noticably the room) pretty soon. On Thu, Jun 5, 2014 at 12:01 PM, Daan Hoogland dhoogl...@schubergphilis.com wrote: People, I added two more hackathon subjects at

Re: redundant virtual routers for VPCs

2014-06-10 Thread Karl Harris
Recovering from a laptop hard disk crash should be back online by 22:00 UTC! Karl On Tuesday, June 10, 2014, Daan Hoogland daan.hoogl...@gmail.com wrote: I would like to have Hugo in the meeting So let's make it 20:00 UTC I would suggest #cloudstack-meeting On Tue, Jun 10, 2014 at 10:29

RE: redundant virtual routers for VPCs

2014-06-10 Thread Daan Hoogland
Karl, That’s midnight over here! :( \DaanH From: Karl Harris [mailto:karl.har...@sungardas.com] Sent: dinsdag 10 juni 2014 2:43 To: Daan Hoogland Cc: Hugo Trippaers; dev; Sheng Yang; Sander Botman; Alena Prokharchyk; kis...@cloud.com; Christopher Litsinger; Hugo Trippaers; int-toolkit Subject:

Re: redundant virtual routers for VPCs

2014-06-10 Thread Karl Harris
20:00 utc ;-) missed the last change in time! Karl On Tuesday, June 10, 2014, Daan Hoogland dhoogl...@schubergphilis.com wrote: Karl, That’s midnight over here! :( *\*DaanH *From:* Karl Harris [mailto:karl.har...@sungardas.com

Re: Review Request 21375: CLOUDSTACK-6654: Configkey parameters are not validated

2014-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21375/#review45223 --- Commit 5bcd017de6f421a6125406120b39fb8602276dc7 in cloudstack's

Re: hackathon CCCEU14

2014-06-10 Thread Rohit Yadav
Hi Daan, I'm up for the new (restful/hateoas) API redesign and unit tests! Regards. On Tue, Jun 10, 2014 at 6:05 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: Hello everybody, Do we still want to have a Hackathon next CCC? we need to plan for the resources (noticably the room) pretty

Re: feature : changing volume properties dynamically in 4.5

2014-06-10 Thread Punith S
hi mike, thanks for the reply, i like your approach which is a very generic way and also we only need to do a few changes to the current cloudstack, but on the other hand we are tying every property of the vendor to a disk offering through key/value pairs, since we offer lot of properties like i

[CS 4.3 | XenServer] Unable to start Virtual Router (com.cloud.exception.AgentUnavailableException)

2014-06-10 Thread Florin Dumitrascu
Hi, Could anyone please help with the following issue I am seeing? I am running CloudStack 4.3 and XenServer 6.2 (SP1) I have removed a host which was working normally and re-added it (it is Host 15 in the log appended). The communication with the host seems to be established as shown in the

Re: redundant virtual routers for VPCs

2014-06-10 Thread Daan Hoogland
ok, write you then. On Tue, Jun 10, 2014 at 2:52 PM, Karl Harris karl.har...@sungardas.com wrote: 20:00 utc ;-) missed the last change in time! Karl On Tuesday, June 10, 2014, Daan Hoogland dhoogl...@schubergphilis.com wrote: Karl, That’s midnight over here! :( \DaanH From:

Re: Review Request 21767: Preload modules so that some sysctl settings will actually get set during boot and some ftp modules to get pasive ftp to work through iptables

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21767/#review45227 --- Ship it! bfccf439cfe120f5d1380a642e8b798335e1cf2e - daan Hoogland

Re: Review Request 21769: blacklisting pcspkr as cosmetic improvement. fixing aesni_intel blacklisting because modprobe.d files have to end with .conf

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21769/#review45228 --- Ship it! 11f532bbecca08b41dd15d1abdf04957c84eebda - daan Hoogland

Re: Review Request 21771: Cosmetic fixes in cloud-early-config script

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21771/#review45229 --- systemvm/patches/debian/config/etc/init.d/cloud-early-config

Re: Review Request 21772: dnsmasq stops logging if logrotate creates the file

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21772/#review45230 --- Ship it! c54ce3cafbc51febe71fb2a997dbfc8ac9167fb0 - daan Hoogland

Re: Review Request 21808: xenstore-utils on debian wheezy does not have /usr/sbin/xenstore so these commands file. It does have xenstore-write and xenstore-rm so by adding a - this is fixed easily.

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21808/#review45232 --- Ship it! 2e83baaca2c998cfdda281301dffb09a1a20aa5c - daan Hoogland

Re: hackathon CCCEU14

2014-06-10 Thread Daan Hoogland
Great Rohit, Feel free to edit the schedule on the wiki On Tue, Jun 10, 2014 at 3:02 PM, Rohit Yadav bhais...@apache.org wrote: Hi Daan, I'm up for the new (restful/hateoas) API redesign and unit tests! Regards. On Tue, Jun 10, 2014 at 6:05 PM, Daan Hoogland daan.hoogl...@gmail.com

Re: hackathon CCCEU14

2014-06-10 Thread Pierre-Luc Dion
I don't know for others but I can't edit to page to add my name to the docs subject. Pierre-Luc Dion Architecte de Solution Cloud | Cloud Solutions Architect 855-OK-CLOUD (855-652-5683) x1101 - - - *CloudOps*420 rue Guy Montréal QC H3J 1S6 www.cloudops.com @CloudOps_ On Tue, Jun 10, 2014 at

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Chip Childers
On Tue, Jun 10, 2014 at 01:45:36PM +0200, sebgoa wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased to announce that he has accepted. Being a committer allows many contributors to contribute more

Re: hackathon CCCEU14

2014-06-10 Thread Chip Childers
On Tue, Jun 10, 2014 at 10:39:39AM -0400, Pierre-Luc Dion wrote: I don't know for others but I can't edit to page to add my name to the docs subject. What's your Confluence user ID?

Re: hackathon CCCEU14

2014-06-10 Thread Daan Hoogland
you do have rights on the wiki, no? On Tue, Jun 10, 2014 at 4:39 PM, Pierre-Luc Dion pd...@cloudops.com wrote: I don't know for others but I can't edit to page to add my name to the docs subject. Pierre-Luc Dion Architecte de Solution Cloud | Cloud Solutions Architect 855-OK-CLOUD

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Hugo Trippaers
Congrats Pierre-Luc! Cheers, Hugo On 10 jun. 2014, at 16:40, Chip Childers chipchild...@apache.org wrote: On Tue, Jun 10, 2014 at 01:45:36PM +0200, sebgoa wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased

Re: Review Request 19616: Added check for null return.

2014-06-10 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19616/#review45234 --- is this still a valid patch? should it be deleted? - daan Hoogland

Re: Review Request 19616: Added check for null return.

2014-06-10 Thread Alex Hitchins
I believe it to be valid still. I'll review later and confirm. I know you had comments on the patch which I had addressed. On 10 Jun 2014, at 15:50, daan Hoogland daan.hoogl...@gmail.com wrote: --- This is an automatically

Re: Review Request 19616: Added check for null return.

2014-06-10 Thread Daan Hoogland
it wasn't addressed in the uploaded patch. On Tue, Jun 10, 2014 at 4:59 PM, Alex Hitchins a...@alexhitchins.com wrote: I believe it to be valid still. I'll review later and confirm. I know you had comments on the patch which I had addressed. On 10 Jun 2014, at 15:50, daan Hoogland

Re: hackathon CCCEU14

2014-06-10 Thread Chip Childers
On Tue, Jun 10, 2014 at 10:39:39AM -0400, Pierre-Luc Dion wrote: I don't know for others but I can't edit to page to add my name to the docs subject. I found your account and have given you edit perms. -chip

RE: Review Request 19616: Added check for null return.

2014-06-10 Thread Alex Hitchins
I will take a look later tonight. Alex Hitchins | 07788 423 969 | 01892 523 587 -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: 10 June 2014 16:01 To: dev Subject: Re: Review Request 19616: Added check for null return. it wasn't addressed in the uploaded

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Mike Tutkowski
Congratulations! On Tuesday, June 10, 2014, Hugo Trippaers h...@trippaers.nl wrote: Congrats Pierre-Luc! Cheers, Hugo On 10 jun. 2014, at 16:40, Chip Childers chipchild...@apache.org javascript:; wrote: On Tue, Jun 10, 2014 at 01:45:36PM +0200, sebgoa wrote: The Project Management

Re: feature : changing volume properties dynamically in 4.5

2014-06-10 Thread Mike Tutkowski
Hi Punith, Yeah, I hear you about the number of permutations involved. Traditionally Compute and Disk Offerings have been immutable. It makes it easier from an accounting point of view for chargeback and billing. You should definitely feel free to extend the CloudStack API. I think NetApp did

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Nguyen Anh Tu
Congratulation, Pierre-Luc! --Tuna On Tue, Jun 10, 2014 at 10:34 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Congratulations! On Tuesday, June 10, 2014, Hugo Trippaers h...@trippaers.nl wrote: Congrats Pierre-Luc! Cheers, Hugo On 10 jun. 2014, at 16:40, Chip

Re: feature : changing volume properties dynamically in 4.5

2014-06-10 Thread Mike Tutkowski
I plan to draw up a design document surrounding generic key/value pairs today. Perhaps you can take a look at it when you have time, Punith? On Tue, Jun 10, 2014 at 10:06 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hi Punith, Yeah, I hear you about the number of permutations

installing cloudstack development environment with eclipse

2014-06-10 Thread Elapavuluri, Jaya
Hello, I am trying to setup cloudstack development environment using eclipse. These are the errors I am encountering, Could someone help me out to resolve these? Plugin execution not covered by lifecycle configuration: org.apache.maven.plugins:maven-checkstyle-plugin:2.11:check (execution:

Re: installing cloudstack development environment with eclipse

2014-06-10 Thread Mike Tutkowski
As for Plugin execution not covered by lifecycle configuration, you can ignore that one (there are probably many instances of this error). I think m2eclipse doesn't currently support something in our POM files and that's why this error is displayed. On Tue, Jun 10, 2014 at 10:21 AM, Elapavuluri,

Re: [GSoC] [CLOUDSTACK-6114] Progress update

2014-06-10 Thread Ian Duffy
Cool! if you hit *any* issues do let me know. I've a feeling there may be some around vagrant and berkshelf. This almost completes design 1 - Running Cloudstack on Host machine with a MySQL + NFS box supplied and a Hypervisor.

Re: hackathon CCCEU14

2014-06-10 Thread Pierre-Luc Dion
Thanks! Pierre-Luc Dion Architecte de Solution Cloud | Cloud Solutions Architect 855-OK-CLOUD (855-652-5683) x1101 - - - *CloudOps*420 rue Guy Montréal QC H3J 1S6 www.cloudops.com @CloudOps_ On Tue, Jun 10, 2014 at 11:23 AM, Chip Childers chipchild...@apache.org wrote: On Tue, Jun 10, 2014

Re: Review Request 21973: CLOUDSTACK-6780: Resolved cleanup issue in portable ip test cases

2014-06-10 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21973/#review45256 --- Ship it! Ship It! - Santhosh Edukulla On June 10, 2014, 11:16

Re: VPC's VR missing public NIC eth1

2014-06-10 Thread Marcus
If we get the 4.3.0 to 4.3.1 in then there is no need for my previous comment. I was concerned it would not happen. On Wed, Jun 4, 2014 at 9:20 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Wed, Jun 4, 2014 at 4:27 PM, Marcus shadow...@gmail.com wrote: Regarding

Re: VPC's VR missing public NIC eth1

2014-06-10 Thread Marcus
That would be a good idea On Wed, Jun 4, 2014 at 9:16 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: Marcus, I didn't do the db thing for 4.3 but it is idem-potent and can go in a Upgrade430to431.java as well. This one doesn't exist yet. On Wed, Jun 4, 2014 at 4:27 PM, Marcus

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Ian Duffy
Congrats Pierre-Luc! On 10 June 2014 17:07, Nguyen Anh Tu t...@apache.org wrote: Congratulation, Pierre-Luc! --Tuna On Tue, Jun 10, 2014 at 10:34 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Congratulations! On Tuesday, June 10, 2014, Hugo Trippaers h...@trippaers.nl

Re: ssvm logs

2014-06-10 Thread Nitin Mehta
Refer to NfsSecondaryStorageResource.java. This file is the agent code that takes all the commands from CS Management server and executes them. You can see what all logs it writes. It would be great if you could write a wiki as well pointing what all logs can be expected. I can help you make it

[PROPOSAL] Brocade network plugin to orchestrate Brocade VDX Switches for CloudStack 4.5

2014-06-10 Thread Ritu Sabharwal
Hi CS Developers, I have added the Design document for the Plugin in the wiki. Here is the link : https://cwiki.apache.org/confluence/display/CLOUDSTACK/Brocade+Network+Plugin+to+Orchestrate+Brocade+VDX+Switches Please review it and provide your comments. Thanks Regards, Ritu S.

Re: redundant virtual routers for VPCs

2014-06-10 Thread Daan Hoogland
We are on #cloudstack-meeting to start discussing the redundant vpc vr. Please all feel welcome to join in. On Tue, Jun 10, 2014 at 3:32 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: ok, write you then. On Tue, Jun 10, 2014 at 2:52 PM, Karl Harris karl.har...@sungardas.com wrote: 20:00

Re: feature : changing volume properties dynamically in 4.5

2014-06-10 Thread Mike Tutkowski
Here is that design document I was referring to, Punith: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=42566111 I've been working with a student in Tunisia who is participating in Google Summer of Code (GSoC) (I'm his mentor). He'll be working on part of this as will I. (He

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Tim Mackey
Félicitations mon ami! On Tue, Jun 10, 2014 at 1:46 PM, Ian Duffy i...@ianduffy.ie wrote: Congrats Pierre-Luc! On 10 June 2014 17:07, Nguyen Anh Tu t...@apache.org wrote: Congratulation, Pierre-Luc! --Tuna On Tue, Jun 10, 2014 at 10:34 PM, Mike Tutkowski

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Nux!
On 10.06.2014 12:45, sebgoa wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased to announce that he has accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it

Re: [ANNOUNCE] Pierre-Luc Dion as a committer

2014-06-10 Thread Daan Hoogland
Welcome to the family Pierre-Luc (though you were long in, of course) On Tue, Jun 10, 2014 at 11:49 PM, Nux! n...@li.nux.ro wrote: On 10.06.2014 12:45, sebgoa wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Luc Dion to become a committer and we are pleased

Re: feature : changing volume properties dynamically in 4.5

2014-06-10 Thread Mike Tutkowski
I'll send out a [PROPOSAL] e-mail so others who may not be following this e-mail thread have a better chance to comment on the feature. On Tue, Jun 10, 2014 at 2:58 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Here is that design document I was referring to, Punith:

[PROPOSAL] Collect and provide generic key/value pairs to storage plug-ins

2014-06-10 Thread Mike Tutkowski
Hi, Please feel free to review the following proposal for 4.5: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=42566111 Here is the summary: Today the way you associate a Compute Offering (CO) or a Disk Offering (DO) with a Primary Storage (PS) is via storage tagging. This

Re: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-10 Thread Sheng Yang
Hi Hugo, The main point I want to bring up an automation tool here is, enforce mandatory review process and test(regression test probably) to happen before commit. That would slow down the process of course, but it should greatly help on code quality. Even committers would make mistakes from

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Kelven Yang
The usage of these private methods are through reflection, making it private is to avoid exposing unnecessary internal structures to outside. Kelven On 6/10/14, 2:46 AM, Rajani Karuturi rajani.karut...@citrix.com wrote: Hi Kelven, while fixing some of the issues reported by coverity I

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Mike Tutkowski
I haven't looked, but perhaps we should provide sufficient comments in each of these cases so no one removes the fields thinking that they are old and no longer in use. On Tue, Jun 10, 2014 at 5:13 PM, Kelven Yang kelven.y...@citrix.com wrote: The usage of these private methods are through

Re: [PROPOSAL] Brocade network plugin to orchestrate Brocade VDX Switches for CloudStack 4.5

2014-06-10 Thread Chiradeep Vittal
Well done. Question: How do you figure out the VLAN –to- Brocade switch port mapping? By using the mac address of the VM nic? Comment: It looks like you are creating a table (BrocadeNetworkHostMapping). Could you add details of this table (schema). Make sure this table appears in any upgrade

Re: [CS 4.3 | XenServer] Unable to start Virtual Router (com.cloud.exception.AgentUnavailableException)

2014-06-10 Thread Chiradeep Vittal
Found this after some googling. http://comments.gmane.org/gmane.comp.apache.cloudstack.devel/22230 From: Florin Dumitrascu florin.dumitra...@intunenetworks.commailto:florin.dumitra...@intunenetworks.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org

Re: KVM + LXC on the same host

2014-06-10 Thread Chiradeep Vittal
I thought the LXC 2.0 project uses KVM system vms on the same host as LXC containers? https://cwiki.apache.org/confluence/x/oJNMAg From: Kishan Kavala kishan.kav...@citrix.commailto:kishan.kav...@citrix.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org

[DISCUSS] continuous integration for plugins requiring proprietary / hardware infra

2014-06-10 Thread Chiradeep Vittal
Hi, Since the Nuage VSP and Brocade plugins have been proposed recently, I was wondering what is the mechanism to ensure that CI will test their plugins? Thanks ‹ Chiradeep

Re: [Proposal] NuageVsp Network Plugin

2014-06-10 Thread Chiradeep Vittal
Can you describe any tables being added / modified? Do you intend to write to any existing tables? How do you plan to test this? Will you be contributing any continuous integration resources for this? Are you using any 3rd party libraries / source and what is the license? From: Suresh

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Kelven Yang
Maybe utilizing special java annotation would get developer heads-up? Content in comments text is easily skipped as it is not usually considered as mandatory language constructs Kelven On 6/10/14, 4:25 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: I haven't looked, but perhaps we

Re: NetworkOrchestrator selects 2 NetworkGurus at one time....

2014-06-10 Thread Chiradeep Vittal
That is strange. Looks like a bug to me. That is because the ExternalGuestNetworkGuru returns ‘true’ for canHandle. From: Ritu Sabharwal rsabh...@brocade.commailto:rsabh...@brocade.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Mike Tutkowski
Good idea On Tuesday, June 10, 2014, Kelven Yang kelven.y...@citrix.com wrote: Maybe utilizing special java annotation would get developer heads-up? Content in comments text is easily skipped as it is not usually considered as mandatory language constructs Kelven On 6/10/14, 4:25 PM, Mike

[ACS4.4]Cherry-pick 09a357fb90b48ed6e2725ea60e632a2ad5529f79

2014-06-10 Thread Min Chen
Hi Daan, Would you please cherry-pick the following commit from 4.4-forward to 4.4? Commit: 09a357fb90b48ed6e2725ea60e632a2ad5529f79 CLOUDSTACK-6890:createVPC invoked by admin does not observe start flag. Thanks -min

Re: Review Request 22356: Fixed few coverity issues reported

2014-06-10 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22356/#review45330 --- framework/db/src/com/cloud/utils/db/TransactionLegacy.java

Review Request 22415: CLOUDSTACK-6867: added the option for vhdx image format in upload volume

2014-06-10 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22415/ --- Review request for cloudstack, Devdeep Singh and Rajesh Battala. Bugs:

Re: Review Request 21973: CLOUDSTACK-6780: Resolved cleanup issue in portable ip test cases

2014-06-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21973/#review45337 --- Commit 0190c936689fc252e51b7c412283c128b5386b24 in cloudstack's

Re: Review Request 22260: [Windows] Due to progress bar changes mysql bin path was not getting read from the Path environment variable, fixing the same

2014-06-10 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22260/#review45338 --- Ship it! Ship It! - Abhinandan Prateek On June 5, 2014, 10:21

Re: NetworkOrchestrator selects 2 NetworkGurus at one time....

2014-06-10 Thread Murali Reddy
This is know design issue. Unlike service orchestration (which has prescriptive way to tell which network elements to be called for with network offerings ) there is no such logic for network design. Orchestrator just loops through all the network guru's asking to design the network which can

RE: Unable to upload SSL certificate

2014-06-10 Thread Saksham Srivastava
Fixed the issue in CLOUDSTACK-6864 Now we should not require double encoding in the API. Thanks, Saksham -Original Message- From: Saksham Srivastava Sent: Thursday, June 5, 2014 7:54 PM To: 'Syed Ahmed' Cc: Vijay Venkatachalam; dev@cloudstack.apache.org Subject: RE: Unable to upload SSL

Re: Review Request 22377: tagging the test case test_01_sys_vm_start with the bugId.

2014-06-10 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22377/#review45339 --- Ship it! Ship It! - Abhinandan Prateek On June 9, 2014, 1:30