Re: Review Request 12747: Fix configuration of Jetty-based execution so that CloudStack can execute scripts properly

2013-07-19 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12747/#review23488 --- Ship it! Ship It! - Hugo Trippaers On July 18, 2013, 9:34 p.m.,

Re: Review Request 12233: CLOUDSTACK-3324: Fix/Improve upgrade system vm templates 41 to 42 (Stable master Patch)

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12233/#review23489 --- Commit 85e32552a33bc93d00bdfc9297fad5daac7f0a27 in branch refs/heads

RE: code formatting for enums

2013-07-19 Thread Donal Lafferty
> -Original Message- > From: Frank Zhang [mailto:frank.zh...@citrix.com] > Sent: 19 July 2013 12:19 AM > To: dev@cloudstack.apache.org > Subject: RE: code formatting for enums > > Then the most important factor of name > convention is user experience where Running is more user friendly tha

reviewer wanted

2013-07-19 Thread Daan Hoogland
H, Can someone have a quick look at review request 12685 , please? thanks, Daan

New user - Introduction

2013-07-19 Thread René Diepstraten
Hi everyone. Since I have joined the mailing list, I'd like to say hello and introduce myself. My name is Rene Diepstraten, am a system administrator and the company I work for ( PCextreme ) uses Apache CloudStack. I'd like to contribute here and there with my skills in python and bash, and perha

Re: Query String Request Authentication(QSRA) support by S3 providers

2013-07-19 Thread Thomas O'Dowd
Hi Min, I will have time next week to check through the bugs and verify everything. Thanks for checking on me. Tom. On Thu, 2013-07-18 at 23:37 +, Min Chen wrote: > Thanks Jessica. Tom, did you still see the issue? > > -min > > On 7/8/13 1:20 PM, "Jessica Wang" wrote: > > >Min, > > > >>

Re: documentation links to epub/pdf broken

2013-07-19 Thread Sebastien Goasguen
On Jul 19, 2013, at 2:54 AM, Jörgen Maas wrote: > Hi, > > Some links for the 4.0.2 pdf's are also invalid: > - api developers guide > - installation guide > > Can someone take a look at this please? fixed > > Thanks! > > > > On Wed, Jul 17, 2013 at 7:01 PM, Jörgen Maas wrote: > >> No s

Re: code formatting for enums

2013-07-19 Thread Daan Hoogland
all, I am for adding a convention on enums to the coding conventions, thought I don't like Johns best, I'm fine with it and with Alex'. Let's add this one and my proposal on method length and number of methods in classes to the conventions and keep in mind that a convention is not a law. We won't

Re: Demo Lab - To rent or To buy

2013-07-19 Thread Sebastien Goasguen
On Jul 18, 2013, at 7:56 PM, Ian Duffy wrote: > Hi, > > I'm looking for peoples personal opinions. > > I'm thinking about undertaking a project for a 2 month college assignment > sometime in the future to create an API wrapper and web front end with the > end goal of producing something like t

Re: New user - Introduction

2013-07-19 Thread Wei ZHOU
Welcome! 2013/7/19 René Diepstraten > Hi everyone. > > Since I have joined the mailing list, I'd like to say hello and introduce > myself. > My name is Rene Diepstraten, am a system administrator and the company I > work for ( PCextreme ) uses Apache CloudStack. > > I'd like to contribute here

Review Request 12757: CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12757/ --- Review request for cloudstack, Jayapal Reddy and Kishan Kavala. Bugs: CLOUDSTAC

Re: Review Request 12757: [4.2]CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12757/ --- (Updated July 19, 2013, 8:52 a.m.) Review request for cloudstack, Jayapal Reddy

Re: New user - Introduction

2013-07-19 Thread Prasanna Santhanam
Welcome Ren??. You can comb through JIRA for anything that interests you. On Fri, Jul 19, 2013 at 10:19:14AM +0200, Ren? Diepstraten wrote: > Hi everyone. > > Since I have joined the mailing list, I'd like to say hello and introduce > myself. > My name is Rene Diepstraten, am a system administrat

Re: Is it possible for reviewer to add other reviewers in the reviewboard?

2013-07-19 Thread Daan Hoogland
I'll have a quick look on rbtool. I don't agree at all that more ASFBot noise should be produced. I'd like to take the time to learn lua, but the dev channel is not inviting to me because of it. I would propose to remove the present day message from it instead. They get send to the mail list which

Re: New user - Introduction

2013-07-19 Thread Jayapal Reddy Uradi
Welcome Rene! On 19-Jul-2013, at 1:49 PM, René Diepstraten wrote: > Hi everyone. > > Since I have joined the mailing list, I'd like to say hello and introduce > myself. > My name is Rene Diepstraten, am a system administrator and the company I > work for ( PCextreme ) uses Apache CloudStack. >

Re: Review Request 12757: [4.2]CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12757/#review23490 --- Commit 7c664728f7d9d82c32fae09f62f98d77b74634b5 in branch refs/heads

Re: Review Request 12757: [4.2]CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12757/#review23491 --- Ship it! Ship It! - Jayapal Reddy On July 19, 2013, 8:52 a.m., H

Re: Review Request 12752: Async commands can inject the job id

2013-07-19 Thread Murali Reddy
Ryan, You should open a bug and add details on what the use case you are trying to solve. Since CloudStack is not in control of UUID generation, you should at least check that uniqueness of passed in UUID. Also please introduce a new smoke test for 'injected job id' instead of piggy backing the t

Re: New user - Introduction

2013-07-19 Thread Wido den Hollander
Hi Rene! For everybody here: Rene works for the company I work for to as well. I encouraged him to start working on CloudStack. Fix what's broken as you encounter things :) Wido On 07/19/2013 10:19 AM, René Diepstraten wrote: Hi everyone. Since I have joined the mailing list, I'd like to s

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

2013-07-19 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23492 --- The changes related to CLOUDSTACK-2536 is not required. updateCluste

Re: Review Request 12685: CLOUDSTACK-1532 added utility functions to scan URIs

2013-07-19 Thread Wido den Hollander
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12685/#review23493 --- It seems good to me? Code-wise I can't find anything that isn't righ

Review Request 12758: Cloustack - 3452: Resolved ssh issue for basic zone. Added security group for vms created in basic zone to allow ssh

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

Re: Review Request 12685: CLOUDSTACK-1532 added utility functions to scan URIs

2013-07-19 Thread Hugo Trippaers
> On July 19, 2013, 9:41 a.m., Wido den Hollander wrote: > > It seems good to me? Code-wise I can't find anything that isn't right. > > > > I see that Spark404 is also mentioned here, did he get a chance to look at > > this yet? Yeah, i'm ok with this. I'm actually abusing this patch to test

Re: Review Request 12427: Fix for CLOUDSTACK-3365: cluster level parameters cluster.(cpu/memory).allocated.capacity.notificationthreshold is not considering overcommit value

2013-07-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/ --- (Updated July 19, 2013, 10:14 a.m.) Review request for cloudstack, Abhinandan P

Re: Review Request 12222: CLOUDSTACK-2510: Getting NPE while executing listStoragePools API command.

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1/#review23497 --- Commit 6e63f10a4f4f77c1b57f1e804b5a443c6b75 in branch refs/heads

Re: Review Request 12544: CLOUDSTACK-3496: [Regression]Primary Storage resource usage is not getting updated if detached volume is deleted.

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12544/#review23495 --- Commit 3771e8a0e61385aa6bda0fd73afa961b6a6cb091 in branch refs/heads

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

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12629/#review23496 --- Commit 539eb38908ede53a36d4bb680b848caf9fe469c4 in branch refs/heads

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

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12629/#review23499 --- Commit 5de96db81d666a680f99e0a719ef79794c6358ff in branch refs/heads

Re: Review Request 12222: CLOUDSTACK-2510: Getting NPE while executing listStoragePools API command.

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1/#review23500 --- Commit 6770c5d92dae37fc48f5b63900936616218815ea in branch refs/heads

Re: Review Request 12544: CLOUDSTACK-3496: [Regression]Primary Storage resource usage is not getting updated if detached volume is deleted.

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

Re: Review Request 12222: CLOUDSTACK-2510: Getting NPE while executing listStoragePools API command.

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1/#review23501 --- Ship it! Ship It! - Devdeep Singh On July 2, 2013, 5:26 a.m., Sa

Re: Review Request 12544: CLOUDSTACK-3496: [Regression]Primary Storage resource usage is not getting updated if detached volume is deleted.

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12544/#review23503 --- Ship it! Ship It! - Devdeep Singh On July 15, 2013, 9:16 a.m., S

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

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12629/#review23502 --- Ship it! Ship It! - Devdeep Singh On July 17, 2013, 5:11 a.m., S

Re: New user - Introduction

2013-07-19 Thread Nitin Mehta
Welcome aboard Rene. Feel free to browse the wiki, JIRA bugs, code and ask us questions :) On 19/07/13 3:04 PM, "Wido den Hollander" wrote: >Hi Rene! > >For everybody here: Rene works for the company I work for to as well. I >encouraged him to start working on CloudStack. Fix what's broken as yo

Re: 41-42 db upgrade

2013-07-19 Thread Harikrishna Patnala
Hi During system vm template upgrade from 41 to 42 takes care of insertion of router.template.xen/* parameters. But checked in my part so that it does nt break any upgrade by developer. Thanks Harikrishna On 18-Jul-2013, at 2:39 AM, Min Chen mailto:min.c...@citrix.com>> wrote: Checked in my pa

Re: Review Request 12720: CLOUDSTACK: 3382 Unable to Migrate VM's If the hosts are implicitly or explicitly dedicated.

2013-07-19 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12720/ --- (Updated July 19, 2013, 11:52 a.m.) Review request for cloudstack and Devdeep S

Re: Review Request 12720: CLOUDSTACK: 3382 Unable to Migrate VM's If the hosts are implicitly or explicitly dedicated.

2013-07-19 Thread Saksham Srivastava
> On July 18, 2013, 11:09 a.m., Prasanna Santhanam wrote: > > Do you think you can add a integration test to test_explicit_dedication.py > > for this failure? That would be really useful to catch this part of tricky > > code Prasanna the changes I have made are specifically related to alert ge

Re: Demo Lab - To rent or To buy

2013-07-19 Thread Ian Duffy
@IIya - Thanks for the response. You dont need a server, a nicely loaded desktop or even large laptop would > do the job. I've currently been trying to do this on a late 2011 macbook pro with i7 processor, 16gb of ram, OS on SSD. I was running VMware fusion with an xenserver VM and a Cloudstack

Re: Demo Lab - To rent or To buy

2013-07-19 Thread David Nalley
So it's not ideal, particularly from a RAM perspective. But my personal, local demo lab is using Mac Mini Servers. Mac Mini's are small, quiet, don't draw a ton of power and can be repopulated with up to 16GB of RAM. The UEFI is somewhat painful for installing things that aren't OSX, but not imposs

[discuss] add code style checker rules to master?

2013-07-19 Thread Donal Lafferty
For automated code style checking, I created a rules file for CheckStyle based on the ACS coding conventions, and blogged on how to use it with Eclipse ( http://dlafferty.blogspot.co.uk/2013/07/apache-cloudstack-java-coding.html ). However, the rules are not 100% correct, and it is difficult for

Review Request 12759: CLOUDSTACK-3669: Bringing down the dynamic scalable check (for VM whether it has xstools or vmware tools in it) to the management layer.

2013-07-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12759/ --- Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. Bugs: CLOUDS

RE: Does the default Xen SystemVM Template understand S3?

2013-07-19 Thread Donal Lafferty
FYI: the default XenServer System VM does not use the latest JSON serialisation design. Must be some old code in there. See https://issues.apache.org/jira/browse/CLOUDSTACK-3377?focusedCommentId=13713542&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13713542

Re: Does the default Xen SystemVM Template understand S3?

2013-07-19 Thread Prasanna Santhanam
See CLOUDSTACK-3480. if using object store the templates are going to be wrong since the storage code will get the templates as in the template table. There is no "seeding" the template when using the object store feature. Is this related to what you are seeing? On Fri, Jul 19, 2013 at 12:17:25PM

Re: [rant] stupid test cases

2013-07-19 Thread Prasanna Santhanam
My problem is that we have tests that already check cleanup of accounts. A test should do the most crystallized set of steps to achieve the scenario and not try to put everything and the kitchen sink in to it. If we see ourselves doing that - we need to break down our tests into smaller blocks. The

Re: Review Request 12758: Cloustack - 3452: Added check if security groups feature is enabled in basic zone before creating any sec group

2013-07-19 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12758/ --- (Updated July 19, 2013, 1:07 p.m.) Review request for cloudstack and Prasanna S

[DOCS] Known issues in release notes

2013-07-19 Thread Chip Childers
Hi all, I've been working on the release notes for 4.1.1, and when I got to the known issues section I took a look at past releases. For the 4.0.x series, we used a query that checked for unresolved bugs that had a fix version of the release that the notes were for. This doesn't seem quite right

Re: [DOCS] Known issues in release notes

2013-07-19 Thread Chip Childers
On Fri, Jul 19, 2013 at 09:09:26AM -0400, Chip Childers wrote: > Hi all, > > I've been working on the release notes for 4.1.1, and when I got to the > known issues section I took a look at past releases. > > For the 4.0.x series, we used a query that checked for unresolved bugs > that had a fix v

Re: code formatting for enums

2013-07-19 Thread John Burwell
Frank, I expect that renaming an enum value to be a minor change (i.e. once the codebase compiles, the change is good). However, i can say I am surprised to learn that such an operation could break the API and/or corrupt that database. To my mind, this issue has nothing to do with developer c

Re: Demo Lab - To rent or To buy

2013-07-19 Thread Kelly Hair
@Ian - which version of Fusion were you using? According to http://www.virtualizationadmin.com/blogs/lowe/news/vmware-fusion-5-enable-vt-xept-inside-a-virtual-machine-288.html, it is supported on version 5. I had a similar problem in Workstation when running nested ESXi VMs until I upgraded. Min

Re: Review Request 12752: Async commands can inject the job id

2013-07-19 Thread Marcus Sorensen
This is maybe a tangent, but does cloudstack check for uniqueness when generating a UUID? I haven't seen it anywhere, I've always seen it do UUID.randomUUID() and no checking. However, I guess the point is that CS is in control of making sure it's randomly generated, with this patch CS could be fed

Re: Review Request 11582: CLOUDSTACK-2053: fixed if/else logic to address redundant entires, fixed boto/cloud-init support by adding metadata rewrite rule

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

Re: Review Request 12685: CLOUDSTACK-1532 added utility functions to scan URIs

2013-07-19 Thread Alex Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12685/#review23515 --- Ship it! master: 2d4464d One review comment is if you are checking

RE: Does the default Xen SystemVM Template understand S3?

2013-07-19 Thread Donal Lafferty
Followed "Maven procedure developed for Cloudstack 4.1.0 and later" on http://cloudstack.apache.org/develop/environment.html I'll try seeding as per "CloudStack Installation Guide" (https://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.2/html/Installation_Guide/management-server-insta

[DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Chip Childers
As it stands now, we currently use a "Lazy Consensus" model (yes Noah, I know we didn't define that term correctly as of now, but I think that's a different discussion). We currently have that term defined as: > Lazy Consensus - Lazy consensus requires 3 binding +1 votes and no > binding -1 votes

Re: Review Request 11582: CLOUDSTACK-2053: fixed if/else logic to address redundant entires, fixed boto/cloud-init support by adding metadata rewrite rule

2013-07-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11582/#review23511 --- Commit 88b36edf87140692861f00da05f3fa8955340c80 in branch refs/heads

RE: code formatting for enums

2013-07-19 Thread Alex Huang
John, Just to be clear. Not doing enum change correctly can cause problems with existing databases. That's a known problem with enum persistence in Java because enums by default using the name of the constant to represent the string of the constant. I was just cautioning anyone who wants to

Re: Demo Lab - To rent or To buy

2013-07-19 Thread Joe Brockmeier
On Thu, Jul 18, 2013, at 08:27 PM, Musayev, Ilya wrote: > Specs would really depend on budget, in US I can build this setup for > about $800, more or less depends on the componets. One of the things I've usually done is go to Geeks.com and buy a second-hand workstation. Typically you can pick up a

RE: New user - Introduction

2013-07-19 Thread Alex Huang
Welcome Rene! --Alex > -Original Message- > From: René Diepstraten [mailto:r...@renediepstraten.nl] > Sent: Friday, July 19, 2013 1:19 AM > To: dev@cloudstack.apache.org > Subject: New user - Introduction > > Hi everyone. > > Since I have joined the mailing list, I'd like to say hello a

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Chiradeep Vittal
There's several places in the by laws that call for Lazy Consensus. Are we discussing modifying all of them or just new committer votes? On 7/19/13 9:02 PM, "Chip Childers" wrote: >As it stands now, we currently use a "Lazy Consensus" model (yes Noah, I >know we didn't define that term correctly

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Chip Childers
On Fri, Jul 19, 2013 at 04:29:07PM +, Chiradeep Vittal wrote: > There's several places in the by laws that call for Lazy Consensus. Are we > discussing modifying all of them or just new committer votes? New committer and PMC membership. sorry, I think the email could be more clear. This is p

Re: New user - Introduction

2013-07-19 Thread Joe Brockmeier
On Fri, Jul 19, 2013, at 03:19 AM, René Diepstraten wrote: > A few small patches that I wrote have been accepted into Master, and I'd > like to contribute more to this great project in the future. Welcome René! Best, jzb -- Joe Brockmeier j...@zonker.net Twitter: @jzb http://www.dissociatedpre

Re: Review Request 11582: CLOUDSTACK-2053: fixed if/else logic to address redundant entires, fixed boto/cloud-init support by adding metadata rewrite rule

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

Re: [DISCUSS] Upgrade path to ACS 4.2 from CCP

2013-07-19 Thread Chip Childers
On Thu, Jul 18, 2013 at 10:01:02PM +, Animesh Chaturvedi wrote: > [Animesh>] Yes this seems late for 4.2, but it makes it easier to add support > now. If there is objection we can track it for 4.3. Are you sure it's easier to add support now? That basically puts us in a position where it's

Re: Reviewboard and jenkins

2013-07-19 Thread Chip Childers
Freaking awesome. I saw that hit the ML, and was just about to ask about it. On Fri, Jul 19, 2013 at 12:51 PM, Hugo Trippaers wrote: > Hey all, > > I've been working on some scripting to get patches submitted on review > board automagically tested with Jenkins. > > The way it works is that a s

Re: Reviewboard and jenkins

2013-07-19 Thread Daan Hoogland
good stuff, does it run the risk of running while another instance is busy? On Fri, Jul 19, 2013 at 6:57 PM, Prasanna Santhanam wrote: > On Fri, Jul 19, 2013 at 06:51:29PM +0200, Hugo Trippaers wrote: > > Hey all, > > > > I've been working on some scripting to get patches submitted on review >

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread David Nalley
On Fri, Jul 19, 2013 at 11:32 AM, Chip Childers wrote: > As it stands now, we currently use a "Lazy Consensus" model (yes Noah, I > know we didn't define that term correctly as of now, but I think that's > a different discussion). We currently have that term defined as: > >> Lazy Consensus - Lazy

RE: reviewer wanted

2013-07-19 Thread Alex Huang
Done. I like that you just request this on the dev list too. --Alex > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Friday, July 19, 2013 1:15 AM > To: dev > Subject: reviewer wanted > > H, > > Can someone have a quick look at review request 12685 ,

Review Request 12762: CLOUDSTACK-3620: [Automation] API removevpnuser failed, in test case test_project_usage.TestVpnUsage.test_01_vpn_usage.

2013-07-19 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12762/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: CLOUDSTACK-3620 R

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Noah Slater
Specifically, Chip is calling for us to change committer / PMC votes from "lazy consensus" to "2/3 majority". (That is, the vote type for that specific decision making process changes, but the vote type definitions are left alone.) On 19 July 2013 17:32, Chip Childers wrote: > On Fri, Jul 19, 2

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Joe Brockmeier
On Fri, Jul 19, 2013, at 10:32 AM, Chip Childers wrote: > As it stands now, we currently use a "Lazy Consensus" model (yes Noah, I > know we didn't define that term correctly as of now, but I think that's > a different discussion). We currently have that term defined as: I would be in favor of de

Re: Demo Lab - To rent or To buy

2013-07-19 Thread Ian Duffy
@Kelly - Assuming you mis-read there were no HVM/VT-passthrough issues with my version of vmware fusion. @Joe and @David - Thanks for the feedback, will definitely look into that.

Re: Trouble uploading ISO file

2013-07-19 Thread Mike Tutkowski
Thanks a lot! I wasn't sure if I was doing something wrong there or if a bug should be logged. On Fri, Jul 19, 2013 at 11:36 AM, Soheil Eizadi wrote: > I logged jira bug to track this problem: > https://issues.apache.org/jira/browse/CLOUDSTACK-3671 > -Soheil > ___

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Joe Brockmeier
On Fri, Jul 19, 2013, at 10:32 AM, Chip Childers wrote: > Are there any objections to me starting a VOTE on this change? Even though I occasionally disagree with the outcome of the current bylaws, I think there's value in having a "veto" over adding committers or PMC members. I'd be -1 to this. B

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

2013-07-19 Thread Soheil Eizadi
I am trying to figure out if the behavior I am seeing for Basic Networking: For my use case I have a Network Service Offering that has replaced Virtual Router service, e.g. DHCP/DNS with my plugin service. Then I created a zone with Basic Networking configuration with that Network Service Offeri

RE: Query String Request Authentication(QSRA) support by S3 providers

2013-07-19 Thread Jessica Wang
Sanjeev, Thanks for the confirmation. Jessica -Original Message- From: Sanjeev Neelarapu Sent: Thursday, July 18, 2013 11:05 PM To: dev@cloudstack.apache.org; Jessica Wang Cc: Edison Su; Thomas O'Dowd Subject: RE: Query String Request Authentication(QSRA) support by S3 providers Hi, I

Re: Review Request 12752: Async commands can inject the job id

2013-07-19 Thread Ryan Dietrich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12752/ --- (Updated July 19, 2013, 5:54 p.m.) Review request for cloudstack and Marcus Sor

Re: Review Request 12762: CLOUDSTACK-3620: [Automation] API removevpnuser failed, in test case test_project_usage.TestVpnUsage.test_01_vpn_usage.

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

Re: [DISCUSS] Bylaw changes for new committer / new PMC member votes

2013-07-19 Thread Noah Slater
Yes, ignoring our egregious mis-use of that term, I am +1 on your idea. Though, perhaps 3/4 is safer. Consensus is important, yes. But the bigger the PMC, the harder it is to achieve. And more often than not, I see no reason to block an action when a supermajority are clearly in favour of it. Let'

Reviewboard and jenkins

2013-07-19 Thread Hugo Trippaers
Hey all, I've been working on some scripting to get patches submitted on review board automagically tested with Jenkins. The way it works is that a script will check if the jenkins user on review board (catofong) has commented on the review. If it hasn't it will submit a job to jenkins.cloudst

RE: Trouble uploading ISO file

2013-07-19 Thread Soheil Eizadi
I logged jira bug to track this problem: https://issues.apache.org/jira/browse/CLOUDSTACK-3671 -Soheil From: Soheil Eizadi [seiz...@infoblox.com] Sent: Wednesday, July 17, 2013 11:57 AM To: dev@cloudstack.apache.org Subject: RE: Trouble uploading ISO file T

Re: [discuss] add code style checker rules to master?

2013-07-19 Thread Mike Tutkowski
I think this is a really good idea, Donal. Nice post, too, by the way. On Fri, Jul 19, 2013 at 6:08 AM, Donal Lafferty wrote: > For automated code style checking, I created a rules file for CheckStyle > based on the ACS coding conventions, and blogged on how to use it with > Eclipse ( > http://

RE: code formatting for enums

2013-07-19 Thread Frank Zhang
Alex, John, I do think we should think carefully about changing the enum names. First to me it's not a surprise that changing enum names may break API and database, as in java enum name natively represent value. If we want to overcome this restriction I would suggest directly using String so vari

RE: code formatting for enums

2013-07-19 Thread Alex Huang
John, That's actually not necessary. Sometimes code speak clearer than words. Take Donal's example: Enum HypervisorType { Hyperv; } Let's say we have a VO with the following declared. @Column(name="type", updatable = true, nullable=false) @Enumerated(value=EnumType.STRING) priv

Re: code formatting for enums

2013-07-19 Thread John Burwell
Alex/Frank/Daan, In terms of naming convention, it is not the highest priority to me. However, I would encourage us to follow the conventions of the larger Java eco system unless we have a good reason to diverge. Closely tracking the conventions and norms of the larger Java eco-system reduces

Re: Query String Request Authentication(QSRA) support by S3 providers

2013-07-19 Thread Min Chen
Thanks, Tom and Sanjeev for verification. On 7/19/13 1:26 AM, "Thomas O'Dowd" wrote: >Hi Min, > >I will have time next week to check through the bugs and verify >everything. Thanks for checking on me. > >Tom. > >On Thu, 2013-07-18 at 23:37 +, Min Chen wrote: >> Thanks Jessica. Tom, did you s

Re: Review Request 12712: Add a new snapshot flag in cloudstack for KVM 6.3 and verify it during the upgrade

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

Re: New user - Introduction

2013-07-19 Thread Ahmad Emneina
Awesome, the more fixes the better. Welcome!! On Fri, Jul 19, 2013 at 1:19 AM, René Diepstraten wrote: > Hi everyone. > > Since I have joined the mailing list, I'd like to say hello and introduce > myself. > My name is Rene Diepstraten, am a system administrator and the company I > work for ( PC

Re: Object Store Region Assignment

2013-07-19 Thread John Burwell
Chip, So two regions won't share the same database? Thanks, -John On Jul 19, 2013, at 4:06 PM, Chip Childers wrote: > On Fri, Jul 19, 2013 at 03:46:36PM -0400, John Burwell wrote: >> All, >> >> In reviewing the 4.2 object store enhancements, object stores are defined >> with region scope. H

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

2013-07-19 Thread John Burwell
All, I agree with Chip's sentiments. Personally, I strive for the "one screen full" rule of thumb. However, I think a hard and fast rule is a bit restrictive. Thanks, -John On Jul 18, 2013, at 5:21 PM, Mike Tutkowski wrote: > Like eight or so years ago, I sent a method I had to modify to t

Re: Object Store Region Assignment

2013-07-19 Thread Chip Childers
On Fri, Jul 19, 2013 at 4:31 PM, John Burwell wrote: > Chip, > > So two regions won't share the same database? > Nope, that's why you have to use something (not provided yet) to sync the acct / domain metadata between them. > > Thanks, > -John > > On Jul 19, 2013, at 4:06 PM, Chip Childers >

Re: Object Store Region Assignment

2013-07-19 Thread John Burwell
Chip, Why do we have a region entity in the database if everything is assumed to be one and only region? Thanks, -John On Jul 19, 2013, at 4:34 PM, Chip Childers wrote: > On Fri, Jul 19, 2013 at 4:31 PM, John Burwell wrote: > >> Chip, >> >> So two regions won't share the same database? >>

RE: [DISCUSS] Upgrade path to ACS 4.2 from CCP

2013-07-19 Thread Musayev, Ilya
Animesh, Would this have any effect on non-CCP installations? I see this move as a positive thing and it would be ideal to have in 4.2, but we don't want to risk stability, hence I'm curious on the safety and possible issues for non-CCP users. Thanks ilya > -Original Message- > From

Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

2013-07-19 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12775/ --- Review request for cloudstack, ilya musayev and Wido den Hollander. Bugs: cloud

Re: Trigger System VM Upgrade - without destroy and deploy

2013-07-19 Thread Ian Service
Yes! Shut them down and then *very quickly* (before they're automatically restarted) you can issue the changeServiceForSystemVm API call to change their Service Offering. - Ian On Fri, Jul 19, 2013 at 4:48 PM, Musayev, Ilya wrote: > Is there a way to trigger system VM upgrade without destroyin

Re: Reviewboard and jenkins

2013-07-19 Thread Hugo Trippaers
On Jul 19, 2013, at 7:15 PM, Daan Hoogland wrote: > good stuff, does it run the risk of running while another instance is busy? Probably not, it's a single threaded script. I intend to use jenkins to schedule it and i can tell jenkins not to start another instance of a job once one is running

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

2013-07-19 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12623/#review23523 --- scripts/vm/network/vnet/modifyvxlan.sh

storage.max.volume.size

2013-07-19 Thread Damien Mannix
Hi, I have to present a 3TB Volume, I add it as a primary storage add the appropriate tags, add an offering. When customer tries to attach it it gets stuck in creating state. I have set the value in global settings to 5000GB. Is there a limit to the size of a volume? Input appreciated. Thank

RE: [DISCUSS] Upgrade path to ACS 4.2 from CCP

2013-07-19 Thread Animesh Chaturvedi
> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, July 19, 2013 9:39 AM > To: dev@cloudstack.apache.org > Subject: Re: [DISCUSS] Upgrade path to ACS 4.2 from CCP > > On Thu, Jul 18, 2013 at 10:01:02PM +, Animesh Chaturvedi wrote: > > [Anime

Re: Reviewboard and jenkins

2013-07-19 Thread Prasanna Santhanam
On Fri, Jul 19, 2013 at 06:51:29PM +0200, Hugo Trippaers wrote: > Hey all, > > I've been working on some scripting to get patches submitted on review board > automagically tested with Jenkins. > > The way it works is that a script will check if the jenkins user on > review board (catofong) has c

Re: Object Store Region Assignment

2013-07-19 Thread Chip Childers
Well, you can add / list the other regions. I'm still trying to get my head around why, but honestly I think it's more of a future looking thing. On Fri, Jul 19, 2013 at 04:39:14PM -0400, John Burwell wrote: > Chip, > > Why do we have a region entity in the database if everything is assumed to b

  1   2   >