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 ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12762/#review23566 --- Commit 5eaec0a5e0611f75a362e58cbb325eb2067ae94e in branch refs/heads

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 Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12762/#review23568 --- Ship it! - Prasanna Santhanam On July 19, 2013, 3:44 p.m., Sanjay

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 ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12762/#review23567 --- Commit ecfeaa1037b18920ce3c734fa67f0ab309d46956 in branch refs/heads

master unstable with CLOUDSTACK-3682/3683

2013-07-19 Thread Prasanna Santhanam
Both bugs are blocking the run of tests on master. -- Prasanna., Powered by BigRock.com

Re: Noisy ASFBot

2013-07-19 Thread Prasanna Santhanam
On Sat, Jul 20, 2013 at 12:48:28AM +0200, Hugo Trippaers wrote: > Hey guys, > > I've noticed that our friendly bots on #cloudstack-dev make a lot > more noise than we all do. I'm getting the impression that is is > actually hindering people trying to get a conversation going. It > difficult to kee

Re: Problem in adding Ceph RBD storage to CloudStack

2013-07-19 Thread Takuma Nakajima
I'm sorry but I forgot to tell you that the environment does not have the internet connection. It is not allowed to make a direct connection to the internet because of the security policy. Wido, > No, it works for me like a charm :) > > Could you set the Agent logging to DEBUG as well and show the

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

2013-07-19 Thread Chip Childers
Was this also put into master? On Fri, Jul 19, 2013 at 09:07:46AM +, ASF Subversion and Git Services wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12757/#review23490 > ---

Re: Regarding the BLOCKER bug Cloudstack-3589 VM created from VPC network is not getting IP

2013-07-19 Thread Sheng Yang
Add Daan's gmail. --Sheng On Fri, Jul 19, 2013 at 6:32 PM, Sheng Yang wrote: > Daan, I've checked the code, the issue is, original code Bharat > wrote(generated dnsmasq.conf) cannot deal with VPC. e.g. the VPC router > wouldn't listen on eth0 and eth1, but non-vpc router wouldn't listen on > e

Re: Object Store Region Assignment

2013-07-19 Thread Chip Childers
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. However, there is no way to assign an object store > definition to a region. To my way of thinking, we must be able to assign

Re: Regarding the BLOCKER bug Cloudstack-3589 VM created from VPC network is not getting IP

2013-07-19 Thread Sheng Yang
Daan, I've checked the code, the issue is, original code Bharat wrote(generated dnsmasq.conf) cannot deal with VPC. e.g. the VPC router wouldn't listen on eth0 and eth1, but non-vpc router wouldn't listen on eth1 and eth2. Also, some detail configurations of VPC are written into /etc/dnsmasq.d/clou

Re: Regarding the BLOCKER bug Cloudstack-3589 VM created from VPC network is not getting IP

2013-07-19 Thread Sheng Yang
Daan, Bharat thinks your fix caused a block bug https://issues.apache.org/jira/browse/CLOUDSTACK-3589? Could you check it? --Sheng On Thu, Jul 18, 2013 at 9:52 AM, Bharat Kumar wrote: > Hi Dann, > > The bug fix > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=b903262df5

RE: Noisy ASFBot

2013-07-19 Thread Alex Huang
I agree. What I do now is to use cloudstack-meeting to talk to people after establishing contact on cloudstack-dev. Cloudstack-dev is so noisy that it's basically useless to me. --Alex > -Original Message- > From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers > Sent:

Noisy ASFBot

2013-07-19 Thread Hugo Trippaers
Hey guys, I've noticed that our friendly bots on #cloudstack-dev make a lot more noise than we all do. I'm getting the impression that is is actually hindering people trying to get a conversation going. It difficult to keep track if ASFBot and Cato_Fong regularly make a lot of noise. While its

[ACS42] NFS Cache Naming

2013-07-19 Thread John Burwell
All, It was my understanding that we had agreed to rename the "NFS Cache" mechanism to reflect that it is not a cache and remove the assumption that it will always be backed by NFS. Is my understanding correct? Thanks, -John

Object Store Region Assignment

2013-07-19 Thread John Burwell
All, In reviewing the 4.2 object store enhancements, object stores are defined with region scope. However, there is no way to assign an object store definition to a region. To my way of thinking, we must be able to assign an object store to a region to avoid transfer of large assets across WA

Review Request 12782: Add encryption for Ucs password

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

RE: Noisy ASFBot

2013-07-19 Thread Animesh Chaturvedi
I agree human conversation should take precedence and toning down ASFBot would help > -Original Message- > From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers > Sent: Friday, July 19, 2013 3:48 PM > To: dev@cloudstack.apache.org > Subject: Noisy ASFBot > > Hey guys, > >

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

2013-07-19 Thread Animesh Chaturvedi
> -Original Message- > From: Musayev, Ilya [mailto:imusa...@webmd.net] > Sent: Friday, July 19, 2013 2:13 PM > To: dev@cloudstack.apache.org > Subject: RE: [DISCUSS] Upgrade path to ACS 4.2 from CCP > > Animesh, > > Would this have any effect on non-CCP installations? [Animesh>] No CCP

RE: code formatting for enums

2013-07-19 Thread Frank Zhang
Yes. we do need to fix such kind of typo > -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: Friday, July 19, 2013 12:28 AM > To: dev@cloudstack.apache.org > Subject: RE: code formatting for enums > > > -Original Message- > > From: Frank Zhang [m

Trigger System VM Upgrade - without destroy and deploy

2013-07-19 Thread Musayev, Ilya
Is there a way to trigger system VM upgrade without destroying and recreating? Reason behind this question, i don't want to mess up current system VMs as they have firewall rules bound to them and recreating them would mean new IP set and new firewall requests. Thanks ilya

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[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

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

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

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

  1   2   >