Re: Review Request: Fix for CLOUDSTACK-2085 VM weight on xen remain same as before vmscaleup

2013-05-20 Thread Harikrishna Patnala
> On May 19, 2013, 1:05 p.m., Prasanna Santhanam wrote: > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java, > > line 144 > > > > > > The if-else logic should be the other way

Review Request: Add package declaration to MidoNetElementTest

2013-05-20 Thread Dave Cahill
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11232/ --- Review request for cloudstack and Pranav Saxena. Description --- >From Mik

Re: MidoNetElementTest Issue

2013-05-20 Thread Dave Cahill
Hi, Mike - thanks for the heads up! I did an mvn clean install test and couldn't see the warning, but your change certainly makes sense. Pranav - I made the change and submitted to ReviewBoard with you marked as a reviewer:* *https://reviews.apache.org/r/11232/ Thanks, Dave. On Sat, May 18, 20

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11221/#review20753 --- scripts/vm/hypervisor/xenserver/xcposs/patch

RE: MidoNetElementTest Issue

2013-05-20 Thread Pranav Saxena
Thanks Dave . I'll apply the change after running mvn clean install test again . Regards, Pranav -Original Message- From: dcah...@midokura.jp [mailto:dcah...@midokura.jp] On Behalf Of Dave Cahill Sent: Monday, May 20, 2013 1:41 PM To: CloudStack Dev Subject: Re: MidoNetElementTest Issue

Re: Review Request: CLOUDSTACK -2297 : Delete Account/Domain is not updating the resources usage of the parent domain

2013-05-20 Thread Devdeep Singh
> On May 20, 2013, 8:53 a.m., Devdeep Singh wrote: > > Ship It! Committed in commid-id a2fea4d4499a963a7bb5aed73b4ee03016684529. Kindly close the request. - Devdeep --- This is an automatically generated e-mail. To reply, visit: https:

Re: Review Request: CLOUDSTACK -2297 : Delete Account/Domain is not updating the resources usage of the parent domain

2013-05-20 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11081/#review20754 --- Ship it! Ship It! - Devdeep Singh On May 14, 2013, 6:55 a.m., Sa

Marvin listPortForwardingRules API

2013-05-20 Thread Girish Shilamkar
Hello Prasanna, Tests using listportforwardingrules are failing with latest marvin. listportforwardingrules failed, due to: errorCode: 431, errorText:Unable to execute API command listportforwardingrules due to invalid value. Invalid parameter id value=c9373d55-3d01-480d-98dc-2365d3927378 due to

Re: Marvin listPortForwardingRules API

2013-05-20 Thread Prasanna Santhanam
On Mon, May 20, 2013 at 02:57:39PM +0530, Girish Shilamkar wrote: > Hello Prasanna, > > Tests using listportforwardingrules are failing with latest marvin. > > listportforwardingrules failed, due to: errorCode: 431, errorText:Unable to > execute API command listportforwardingrules due to invalid

Review Request: Automation of API based P1 cases belongs to LDAP Testplan

2013-05-20 Thread suresh sadhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11233/ --- Review request for cloudstack, Prasanna Santhanam and SrikanteswaraRao Talluri.

Re: Marvin listPortForwardingRules API

2013-05-20 Thread Prasanna Santhanam
On Mon, May 20, 2013 at 03:13:34PM +0530, Prasanna Santhanam wrote: > On Mon, May 20, 2013 at 02:57:39PM +0530, Girish Shilamkar wrote: > > Hello Prasanna, > > > > Tests using listportforwardingrules are failing with latest marvin. > > > > listportforwardingrules failed, due to: errorCode: 431, e

bug CLOUDSTACK-2371 after with nicira integration

2013-05-20 Thread Jayapal Reddy Uradi
Hi Hugo, Bug CLOUDSTACK-2371 NPE is introduced with commit 79c7da07abd4294f150851aa0c2d06a28564c5a9 while deleting private gateway. Can you please looking into the issue. commit 79c7da07abd4294f150851aa0c2d06a28564c5a9 Author: Hugo Trippaers Date: Wed Jul 11 16:14:28 2012 -0700 Phase 1

Re: Review Request: CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10881/#review20758 --- Commit 8da7ec05f1c7dae0aab827ae0568e120111c6579 in branch refs/heads

Re: Review Request: Fix for CLOUDSTACK-2085 VM weight on xen remain same as before vmscaleup

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10903/#review20757 --- Commit 8da7ec05f1c7dae0aab827ae0568e120111c6579 in branch refs/heads

Re: Review Request: Fix for CLOUDSTACK-2085 VM weight on xen remain same as before vmscaleup

2013-05-20 Thread Prasanna Santhanam
> On May 19, 2013, 1:05 p.m., Prasanna Santhanam wrote: > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java, > > line 144 > > > > > > The if-else logic should be the other way

Re: Review Request: Automation of API based P1 cases belongs to LDAP Testplan

2013-05-20 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11233/#review20760 --- Ship it! 2778486c34427dbc4e66a7f5db0e3b2364cf9f11 - Prasanna Santh

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11221/ --- (Updated May 20, 2013, 1:14 p.m.) Review request for cloudstack, Prasanna Santh

Re: stackmate ToDo

2013-05-20 Thread Chip Childers
Dharmesh, Adding Chiradeep to this thread, since Stackmate isn't actually part of Apache CloudStack (but Chiradeep is the author). -chip On Fri, May 17, 2013 at 11:27:07PM +0530, Dharmesh Kakadia wrote: > Hi, > > I would like to contribute to stackmate. I am trying to understand the high > leve

Re: 3.0.x->4.1 DB upgrade: missing table for Netscaler

2013-05-20 Thread Chip Childers
On Fri, May 17, 2013 at 08:30:26PM +, Alena Prokharchyk wrote: > Hi Murali, > > I found that the table "netscaler_pod_ref" (introduced with your checkin > 3b1aca19) is missing on the upgraded setups. The table was introduced in > 40, but it never was a part of 302->40 db upgrade. So for custom

Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/ --- Review request for cloudstack and Chip Childers. Description --- CLOUDSTAC

Re: 3.0.x->4.1 DB upgrade: missing table for Netscaler

2013-05-20 Thread Murali Reddy
On 20/05/13 7:08 PM, "Chip Childers" wrote: >On Fri, May 17, 2013 at 08:30:26PM +, Alena Prokharchyk wrote: >> Hi Murali, >> >> I found that the table "netscaler_pod_ref" (introduced with your checkin >> 3b1aca19) is missing on the upgraded setups. The table was introduced in >> 40, but it n

Re: [DISCUSS] EIP Enhancements FS & Design Document

2013-05-20 Thread Murali Reddy
Swamy, As mentioned in my merge request [1], I have generalised this feature and tried to not enforce AWS EIP semantics. Please see the updated FS [2] [1] http://s.apache.org/xjy [2] https://cwiki.apache.org/confluence/display/CLOUDSTACK/portable+public+IP Let me know if you need further clari

Re: [DISCUSS] Should we be releasing -beta releases?

2013-05-20 Thread Joe Brockmeier
On Tue, May 14, 2013, at 10:59 AM, David Nalley wrote: > Are you going to support upgrades from your Betas to release (and > betaN to betaN+1)? > If the answer is no, then there is no interest on my part. It's not > better than us producing nightly builds, or highlighting jenkins > builds. The pro

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11221/#review20768 --- Ship it! Ship It! - Koushik Das On May 20, 2013, 1:14 p.m., sanj

Re: Review Request: debian: fix build of cloudstack-awsapi package

2013-05-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11180/ --- (Updated May 20, 2013, 3:42 p.m.) Review request for cloudstack, Chip Childers,

Re: Review Request: (CLOUDSTACK-1475) update keystore in SSVM and change download iso/template url after Update SSL Certificate

2013-05-20 Thread Chip Childers
On Fri, May 17, 2013 at 06:24:21AM +, Wei Zhou wrote: > > > > On May 17, 2013, 5:18 a.m., Nitin Mehta wrote: > > > Ship It! > > > > Animesh Chaturvedi wrote: > > Nitin can you commit the patch > > > > Nitin Mehta wrote: > > Some files seem to be removed or atleast the location has c

Re: How to seperate the user portal and admin portal?

2013-05-20 Thread Joe Brockmeier
On Sun, May 19, 2013, at 08:50 PM, sx chen wrote: > hello,I want to seperate the user portal and admin portal,So admin and > user > will login to different server and seperate from each other. Can you give > me some suggestion. > another mgr server for user only and disable some functions,Is it > p

Re: 3.0.x->4.1 DB upgrade: missing table for Netscaler

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 01:53:00PM +, Murali Reddy wrote: > On 20/05/13 7:08 PM, "Chip Childers" wrote: > > >On Fri, May 17, 2013 at 08:30:26PM +, Alena Prokharchyk wrote: > >> Hi Murali, > >> > >> I found that the table "netscaler_pod_ref" (introduced with your checkin > >> 3b1aca19) is

Re: Review Request: debian: fix build of cloudstack-awsapi package

2013-05-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11180/ --- (Updated May 20, 2013, 3:42 p.m.) Review request for cloudstack, Prachi Damle a

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11221/#review20767 --- Commit aa60105a84e1260ec9687759a069a0c9cf0c46bf in branch refs/heads

Re: Review Request: debian: fix build of cloudstack-awsapi package

2013-05-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11180/#review20771 --- Applied to 4.1. However, master seems to be out of sync with 4.1 ri

RE: [PROPOSAL] [CLOUDSTACK-2328] Linux native VXLAN support on KVM hypervisor

2013-05-20 Thread Toshiaki Hatano
Thanks David! I've assigned the task to myself. Sincerely, -- Toshiaki > -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Friday, May 17, 2013 6:58 PM > To: dev@cloudstack.apache.org > Subject: Re: [PROPOSAL] [CLOUDSTACK-2328] Linux native VXLAN support on > KVM hyp

Re: stackmate ToDo

2013-05-20 Thread Chiradeep Vittal
Hi Dharmesh, You got the code structure right. Are you available on IRC (#cloudstack-dev) so we could discuss more? I am in the process of putting a web interface on stackmate. I am in PDT time zone. On 5/20/13 6:36 AM, "Chip Childers" wrote: >Dharmesh, > >Adding Chiradeep to this thread, since

Re: 3.0.x->4.1 DB upgrade: missing table for Netscaler

2013-05-20 Thread Alena Prokharchyk
On 5/20/13 6:53 AM, "Murali Reddy" wrote: >On 20/05/13 7:08 PM, "Chip Childers" wrote: > >>On Fri, May 17, 2013 at 08:30:26PM +, Alena Prokharchyk wrote: >>> Hi Murali, >>> >>> I found that the table "netscaler_pod_ref" (introduced with your >>>checkin >>> 3b1aca19) is missing on the upgrad

Re: How to seperate the user portal and admin portal?

2013-05-20 Thread Ahmad Emneina
this sounds like you either want to customize the UI (maybe as simple as check to see if an admin is trying to log in from the 'user portal' and reject it at login) or create your own custom UI... On Mon, May 20, 2013 at 9:05 AM, Joe Brockmeier wrote: > On Sun, May 19, 2013, at 08:50 PM, sx che

Re: [MERGE]object_store branch into master

2013-05-20 Thread Chip Childers
On Fri, May 17, 2013 at 08:19:57AM -0400, David Nalley wrote: > On Fri, May 17, 2013 at 4:11 AM, Edison Su wrote: > > Hi all, > > Min and I worked on object_store branch during the last one and half > > month. We made a lot of refactor on the storage code, mostly related to > > secondary st

Re: Properties file question

2013-05-20 Thread Chip Childers
On Fri, May 17, 2013 at 02:24:52PM -0600, Mike Tutkowski wrote: > Looks like the correct properties file to modify is here: > > client/WEB-INF/classes/resources/messages.properties Thanks for circling back to the list with the answer to this Mike (and other times you have done it). > > > On Th

Re: [ACS41][REQUEST] 5646f5e9772380cc91a8ccb9894c8ed05ffc891f

2013-05-20 Thread Chip Childers
On Sat, May 18, 2013 at 10:53:59AM +0200, Wido den Hollander wrote: > On 05/17/2013 06:00 PM, Chip Childers wrote: > >Bug ID? > > Sorry, I actually forgot to create one! > > Here you go: https://issues.apache.org/jira/browse/CLOUDSTACK-2566 > > I'd like to request 239bb13dde0d32ca3e05a8ea2b35a88

Re: Review Request: Add package declaration to MidoNetElementTest

2013-05-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11232/#review20773 --- Ship it! committed to master - Chip Childers On May 20, 2013, 8:

Re: MidoNetElementTest Issue

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 08:29:49AM +, Pranav Saxena wrote: > Thanks Dave . I'll apply the change after running mvn clean install test > again . Applied. > > Regards, > Pranav > > -Original Message- > From: dcah...@midokura.jp [mailto:dcah...@midokura.jp] On Behalf Of Dave > Cahil

Re: [DISCUSS] EIP Enhancements FS & Design Document

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 02:19:24PM +, Murali Reddy wrote: > > Swamy, > > As mentioned in my merge request [1], I have generalised this feature and > tried to not enforce AWS EIP semantics. Please see the updated FS [2] > > [1] http://s.apache.org/xjy > [2] > https://cwiki.apache.org/conflue

RE: MidoNetElementTest Issue

2013-05-20 Thread Pranav Saxena
Thanks Chip ! -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, May 20, 2013 11:29 PM To: dev@cloudstack.apache.org Subject: Re: MidoNetElementTest Issue On Mon, May 20, 2013 at 08:29:49AM +, Pranav Saxena wrote: > Thanks Dave . I'll apply the ch

RE: 4-2-advanced-zone-security-group

2013-05-20 Thread Anthony Xu
Thanks for reporting this, It didn't work in normal advanced zone or advanced zone with security group. In advanced zone with security group, only one shared network security group network is supported, which is created when you create the zone, you may not be able to create another guest netwo

RE: [MERGE]object_store branch into master

2013-05-20 Thread Sudha Ponnaganti
Also open defects need to be addressed as testing has been done on feature branch and the numbers seem to be high on this branch - atleast blocker and critical should be addressed. Key Summary AssigneeReporterPriorityStatus CLOUDSTACK-2586 [Object_Store_Refactor] Ext

Re: [MERGE]object_store branch into master

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 06:22:03PM +, Sudha Ponnaganti wrote: > Also open defects need to be addressed as testing has been done on feature > branch and the numbers seem to be high on this branch - atleast blocker and > critical should be addressed. > > Key Summary AssigneeReporter

Re: Review Request: CLOUDSTACK-681: Dedicated Resources - Explicit Dedication, Private zone, pod, cluster or host

2013-05-20 Thread Prachi Damle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10797/#review20775 --- One more change we should make to all APIs: - I see that most of the

Re: Review Request: CLOUDSTACK-681: Dedicated Resources - Explicit Dedication, Private zone, pod, cluster or host

2013-05-20 Thread Prachi Damle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10797/#review20776 --- Also, the patch is failing on latest master, please can you create a

Re: [MERGE]object_store branch into master

2013-05-20 Thread David Nalley
On Mon, May 20, 2013 at 2:22 PM, Sudha Ponnaganti wrote: > Also open defects need to be addressed as testing has been done on feature > branch and the numbers seem to be high on this branch - atleast blocker and > critical should be addressed. > > Key Summary AssigneeReporter

Re: Review Request: CLOUDSTACK-681: Dedicated Resources - Explicit Dedication, Private zone, pod, cluster or host

2013-05-20 Thread Prachi Damle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10797/#review20777 --- It will be good to add an integration test using marvin on these lin

Re: Review Request: Add package declaration to MidoNetElementTest

2013-05-20 Thread Mike Tutkowski
You know, I probably should have mentioned where I was seeing the warning. :) It was in Eclipse. On Mon, May 20, 2013 at 11:58 AM, Chip Childers wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

RE: [MERGE]object_store branch into master

2013-05-20 Thread Animesh Chaturvedi
> -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Monday, May 20, 2013 11:36 AM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE]object_store branch into master > > On Mon, May 20, 2013 at 2:22 PM, Sudha Ponnaganti > wrote: > > Also open defects need to be ad

Re: Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/#review20778 --- Commit 85d54cd1c088997dd08f0328984bee1a55703636 in branch refs/heads

Re: Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/#review20779 --- Commit 7ea2c950f5c1c2446ae94087d7f78812cc94a658 in branch refs/heads

Re: Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/#review20780 --- Commit 806aeb990d28179ff532e97dbf64f87cd3d5ca34 in branch refs/heads

Re: Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/#review20782 --- Commit 8ea1601bd3a9ad04f6f0a6c03be540303d05fef8 in branch refs/heads

Re: Review Request: [ACS41][PATCH] CLOUDSTACK-2554: XCP 1.6 fails to load

2013-05-20 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11238/#review20783 --- Ship it! Ship It! - Chip Childers On May 20, 2013, 1:46 p.m., Pr

Modifying the DB schema

2013-05-20 Thread Mike Tutkowski
Hi, I've been writing code to enable users to select min, max, and burst IOPS for Disk Offerings. I've gotten to the point where I need to create these new fields in the DB, but I'm not sure where we control this logic. My new code would look something like this: alter table disk_offering add co

Re: Modifying the DB schema

2013-05-20 Thread Mike Tutkowski
Just throwing this out there, but maybe in the schema-410to420.sql file? I'm not sure what our upgrade paths are, though, so maybe this file needs to get changed and others? On Mon, May 20, 2013 at 12:55 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Hi, > > I've been writing code

Re: Review Request: Fixed SRX icmp firewall rule configuration issue

2013-05-20 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11224/#review20784 --- plugins/network-elements/juniper-srx/src/com/cloud/network/resource

Re: Modifying the DB schema

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 12:55:27PM -0600, Mike Tutkowski wrote: > Hi, > > I've been writing code to enable users to select min, max, and burst IOPS > for Disk Offerings. > > I've gotten to the point where I need to create these new fields in the DB, > but I'm not sure where we control this logic.

Re: Modifying the DB schema

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 12:58:46PM -0600, Mike Tutkowski wrote: > Just throwing this out there, but maybe in the schema-410to420.sql file? If you are planning on being feature complete by the feature freeze window, then that's the one! > > I'm not sure what our upgrade paths are, though, so mayb

Re: Modifying the DB schema

2013-05-20 Thread Wido den Hollander
On 05/20/2013 08:58 PM, Mike Tutkowski wrote: Just throwing this out there, but maybe in the schema-410to420.sql file? Yes, that's where it should go. If this hits 4.2 I mean. I'm not sure what our upgrade paths are, though, so maybe this file needs to get changed and others? The code wi

Re: Modifying the DB schema

2013-05-20 Thread Mike Tutkowski
Hi everyone, Thanks for that info! When I started with my development work, it was mainly focused on implementing a storage plug-in to be based on Edison's new storage framework. Definitely let me know, but I was thinking such a plug-in wasn't something the list would be as interested in partici

Re: Modifying the DB schema

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 01:09:21PM -0600, Mike Tutkowski wrote: > Hi everyone, > > Thanks for that info! > > When I started with my development work, it was mainly focused on > implementing a storage plug-in to be based on Edison's new storage > framework. > > Definitely let me know, but I was t

Re: Modifying the DB schema

2013-05-20 Thread Mike Tutkowski
Hi Chip, Yes, the intent is to provide the code for release in 4.2. Not just the plug-in, but also the improvements to the storage framework (which should benefit any company wanting to implement dynamic, zone-wide storage). On Mon, May 20, 2013 at 1:11 PM, Chip Childers wrote: > On Mon, May 20

Re: Modifying the DB schema

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 01:14:08PM -0600, Mike Tutkowski wrote: > Hi Chip, > > Yes, the intent is to provide the code for release in 4.2. Not just the > plug-in, but also the improvements to the storage framework (which should > benefit any company wanting to implement dynamic, zone-wide storage).

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Chip Childers
On Fri, May 17, 2013 at 03:32:50PM -0400, Sebastien Goasguen wrote: > > On May 17, 2013, at 3:01 PM, Animesh Chaturvedi > wrote: > > > > > > >> -Original Message- > >> From: Sebastien Goasguen [mailto:run...@gmail.com] > >> Sent: Friday, May 17, 2013 11:47 AM > >> To: dev@cloudstack.a

Re: CLOUDSTACK-2554: Yet another blocker?

2013-05-20 Thread Chip Childers
On Sun, May 19, 2013 at 06:38:01PM +0530, Prasanna Santhanam wrote: > On Fri, May 17, 2013 at 06:28:38PM +, Koushik Das wrote: > > The problem is that there is no default ctor in > > XcpServerResource.java. Now the default ctor was changed to a > > non-default one by this commit. Adding a defau

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread John Burwell
Chip, In addition to this issue, we still do not have a resolution for the system VM clock drift on Xen (CLOUDSTACK-2492 [1]). Thanks, -John [1]: https://issues.apache.org/jira/browse/CLOUDSTACK-2492 On May 20, 2013, at 3:36 PM, Chip Childers wrote: > On Fri, May 17, 2013 at 03:32:50PM -0400

Re: Modifying the DB schema

2013-05-20 Thread Mike Tutkowski
Thanks for the links, Chip. Yeah...I need to become more familiar with our workflow. I plan to polish off what I've been working on in the next day or two. On Mon, May 20, 2013 at 1:18 PM, Chip Childers wrote: > On Mon, May 20, 2013 at 01:14:08PM -0600, Mike Tutkowski wrote: > > Hi Chip, > > >

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 03:43:01PM -0400, John Burwell wrote: > Chip, > > In addition to this issue, we still do not have a resolution for the system > VM clock drift on Xen (CLOUDSTACK-2492 [1]). > > Thanks, > -John > > [1]: https://issues.apache.org/jira/browse/CLOUDSTACK-2492 Yes, but diffe

Review Request: Nop assignment due to missing 'this.'

2013-05-20 Thread Dave Brosius
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11245/ --- Review request for cloudstack. Description --- simple missed 'this.' cause

Re: Review Request: Nop assignment due to missing 'this.'

2013-05-20 Thread Dave Brosius
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11245/ --- (Updated May 20, 2013, 7:47 p.m.) Review request for cloudstack. Description

Review Request: when comparing scopes, compare scope ids by value, not reference.

2013-05-20 Thread Dave Brosius
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11246/ --- Review request for cloudstack. Description --- code compares Long values w

Review Request: remove bogus self assign to parent

2013-05-20 Thread Dave Brosius
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11247/ --- Review request for cloudstack. Description --- code assigns this.parent

Re: [MERGE]object_store branch into master

2013-05-20 Thread John Burwell
All, Since this change is so large, it makes reviewing and commenting in detail extremely difficult. Would it be possible to push this patch through Review Board to ease comprehension and promote a conversation about this patch? Reading through the FS, I have the following questions regarding

Re: [ACS41] System VMs not syncing time - does this block the release?

2013-05-20 Thread Chip Childers
On Wed, May 15, 2013 at 01:11:53PM -0700, Chiradeep Vittal wrote: > For VMWare, the command > vmware-toolbox-cmd timesync status returns 'Disabled'. I can submit a > patch for /etc/init.d/cloud-early-config to enable it Please do! -chip

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Ahmad Emneina
My preference leans towards option 2. Dont release until we have a valid path for the 2.x cast aways to return back to a more modern version. Do we know the LOE required to bring the advanced security groups back for KVM/Xen? On Mon, May 20, 2013 at 12:45 PM, Chip Childers wrote: > On Mon, May 2

Re: [ACS41] System VMs not syncing time - does this block the release?

2013-05-20 Thread Chip Childers
On Mon, May 20, 2013 at 3:59 PM, Chip Childers wrote: > On Wed, May 15, 2013 at 01:11:53PM -0700, Chiradeep Vittal wrote: >> For VMWare, the command >> vmware-toolbox-cmd timesync status returns 'Disabled'. I can submit a >> patch for /etc/init.d/cloud-early-config to enable it > > Please do! > >

[ACS41] Outstanding Blockers - Please Read!

2013-05-20 Thread Chip Childers
Here's where we are. Help resolving or actually reaching consensus on the path forward would be appreciated: CLOUDSTACK-2215 ACS41 SSVM does not use allocated storage ip range Currently unassigned. CLOUDSTACK-2463 CS Upgrade 2.2.14 to 4.1.0 failed due to no public network found (configuration :

Re: [ACS41] System VMs not syncing time - does this block the release?

2013-05-20 Thread Chip Childers
We appear to be at an impasse here as well. I'm going to start a VOTE on this issue, given that we have no easy answer. On Wed, May 15, 2013 at 02:49:41PM -0700, Chiradeep Vittal wrote: > Well, I disagree, from the perspective of hundreds of production clouds. > No harm has been perceived in tho

[VOTE] Move forward with 4.1 without a Xen-specific fix for CLOUDSTACK-2492?

2013-05-20 Thread Chip Childers
All, As discussed on another thread [1], we identified a bug (CLOUDSTACK-2492) in the current 3.x system VMs, where the System VMs are not configured to sync their time with either the host HV or an NTP service. That bug affects the system VMs for all three primary HVs (KVM, Xen and vSphere). P

[DISCUSS] Should we pause merges into master until 4.1 is out the door?

2013-05-20 Thread Chip Childers
All, I can't help but notice that we continue to have features being developed and proposed / merged into master while I struggle to get opinions / help on 4.1. I doubt that we (as a community) want to abandon 4.1, but I can't be certain. Should we abandon an attempt at releasing 4.1, and instea

Re: [VOTE] Move forward with 4.1 without a Xen-specific fix for CLOUDSTACK-2492?

2013-05-20 Thread Chiradeep Vittal
+1 On 5/20/13 1:15 PM, "Chip Childers" wrote: >All, > >As discussed on another thread [1], we identified a bug >(CLOUDSTACK-2492) in the current 3.x system VMs, where the System VMs >are not configured to sync their time with either the host HV or an NTP >service. That bug affects the system VM

Re: [VOTE] Move forward with 4.1 without a Xen-specific fix for CLOUDSTACK-2492?

2013-05-20 Thread Ahmad Emneina
I'm +0 on this, dont want to hold up a release with a neg 1 vote. My opinion is that time sync is critical piece for system vm's. Having the wrong time can lead to system vm's booting and waiting for manual intervention via consistency checks (potential blocker bug IMO). On Mon, May 20, 2013 at 2

RE: [MERGE]object_store branch into master

2013-05-20 Thread Edison Su
> -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Monday, May 20, 2013 12:56 PM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE]object_store branch into master > > All, > > Since this change is so large, it makes reviewing and commenting in detail > ext

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Chiradeep Vittal
May I humbly suggest that the configuration (SG + Advanced + VMWare) was never supported and the end user got themselves into an unfortunate situation by using an unsupported configuration (even if the software let them do it). I have perused both 2.2.13 and 2.2.14 install guides and it is quite cl

RE: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Animesh Chaturvedi
> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, May 20, 2013 12:36 PM > To: dev@cloudstack.apache.org > Subject: Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2 > > On Fri, May 17, 2013 at 03:32:50PM -0400, Sebastien Goasguen

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-20 Thread Ahmad Emneina
in light of this and not understanding the benefit of having sec groups + advanced zone features.. are the security group ACL's even honored in this setup? I'm more inclined to see Decision 3 implemented. Is there a VOTE for this coming up? On Mon, May 20, 2013 at 2:37 PM, Chiradeep Vittal < chir

Review Request: object storage refactor

2013-05-20 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11277/ --- Review request for cloudstack. Description --- The diff generated by: git

RE: [MERGE]object_store branch into master

2013-05-20 Thread Edison Su
> -Original Message- > From: Edison Su [mailto:edison...@citrix.com] > Sent: Monday, May 20, 2013 2:30 PM > To: dev@cloudstack.apache.org > Subject: RE: [MERGE]object_store branch into master > > > > > -Original Message- > > From: John Burwell [mailto:jburw...@basho.com] > > Se

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-20 Thread Sheng Yang
OK, I would just go ahead and merge it. --Sheng On Thu, May 16, 2013 at 8:03 PM, Hiroaki KAWAI wrote: > +1 > > I'm sorry that I could not make time to respond. > I'll catch up after the merge. > > > (2013/05/17 3:57), Chip Childers wrote: > >> On Thu, May 16, 2013 at 06:55:30PM +, Chiradeep

RE: [ACS41] Outstanding Blockers - Please Read!

2013-05-20 Thread Prachi Damle
I am looking into CLOUDSTACK-2568 Thanks, Prachi -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, May 20, 2013 1:04 PM To: dev@cloudstack.apache.org Subject: [ACS41] Outstanding Blockers - Please Read! Here's where we are. Help resolving or actual

Re: [VOTE] Move forward with 4.1 without a Xen-specific fix for CLOUDSTACK-2492?

2013-05-20 Thread John Burwell
All, While it is tough to do, I must cast a -1 for the following reasons: Given that system VMs write files, this defect makes every file created/modified timestamp unreliable. Operational log correlation/debugging is nearly impossible since the clock is out of sync. It renders S3-backed Second

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-20 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9871/#review20797 --- Commit 7260e8d83f07d90b48c34adaeb227de265019487 in branch refs/heads/

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-20 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9871/#review20799 --- Ship it! Shipped. PVLAN migration support shipped with it. - Sheng

RE: [DISCUSS] Should we pause merges into master until 4.1 is out the door?

2013-05-20 Thread Animesh Chaturvedi
> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, May 20, 2013 1:35 PM > To: dev@cloudstack.apache.org > Subject: [DISCUSS] Should we pause merges into master until 4.1 is out the > door? > > All, > > I can't help but notice that we continue t

Re: [VOTE] Move forward with 4.1 without a Xen-specific fix for CLOUDSTACK-2492?

2013-05-20 Thread Chip Childers
On May 20, 2013, at 7:14 PM, John Burwell wrote: > All, > > While it is tough to do, I must cast a -1 for the following reasons: > > Given that system VMs write files, this defect makes every file > created/modified timestamp unreliable. > Operational log correlation/debugging is nearly impossib

Re: CLOUDSTACK-2554: Yet another blocker?

2013-05-20 Thread Chiradeep Vittal
Also, why would the 4.2 system vm template block 4.1 ? On 5/20/13 12:40 PM, "Chip Childers" wrote: >On Sun, May 19, 2013 at 06:38:01PM +0530, Prasanna Santhanam wrote: >> On Fri, May 17, 2013 at 06:28:38PM +, Koushik Das wrote: >> > The problem is that there is no default ctor in >> > XcpSer

Re: [DISCUSS] Should we pause merges into master until 4.1 is out the door?

2013-05-20 Thread Chip Childers
On May 20, 2013, at 7:50 PM, Animesh Chaturvedi wrote: > > >> -Original Message- >> From: Chip Childers [mailto:chip.child...@sungard.com] >> Sent: Monday, May 20, 2013 1:35 PM >> To: dev@cloudstack.apache.org >> Subject: [DISCUSS] Should we pause merges into master until 4.1 is out the >

  1   2   >