Re: Review Request: Bugfix CLOUDSTACK-1594: Secondary storage host always remains Alert status

2013-06-17 Thread Nitin Mehta
Roxanne/Abhi - Thanks for following up. I guess this should be resolved by the Object Store work going on. Edison/Min - Would we still be having an entry for the secondary storage in the host table ? Thanks, -Nitin On 17/06/13 12:01 PM, "Abhinandan Prateek" wrote: > > >> On June 17, 2013, 5:06

Re: Object based Secondary storage.

2013-06-17 Thread Thomas O'Dowd
Thanks Min - I filed 3 small issues today. I've a couple more but I want to try and repeat them again before I file them and I've no time right now. Please let me know if you need any further detail on any of these. https://issues.apache.org/jira/browse/CLOUDSTACK-3027 https://issues.apache.org/ji

Re: Upgrade failure. 2.2.14 to 4.1.0

2013-06-17 Thread Wei ZHOU
The fix for Bug CLOUDSTACK-3005 may help you. -Wei 2013/6/16 Glen Baars > Hello Cloudstack dev, > > Just wanted to share my experiences on upgrading 2.2.14 to 4.1.0 with 6 > Xenserver 5.6SP2 hosts. > > The documentation doesn't work, so many errors. I ended up restoring after > 25 hours back t

Re: Cloud Usage and API

2013-06-17 Thread Wei ZHOU
1. No. APIs are for users, not for cloudstack. There is a thread running at the scheduled time. 2. No, generateUsageRecords API adds an usage job in database cloud_usage.usage_job. 3. There is a thread named Heartbeat in usage server which running every minute. It checks the database and generates

[GSOC] A short description about CloudStack Networking plugin

2013-06-17 Thread Nguyen Anh Tu
Hi all, I made an wiki entry about the CloudStack networking design. I think it's useful for all network plugins can follow. It's located in my gsoc project about improving the native SDN controller. Take a look on it https://cwiki.apache.org/confluence/display/CLOUDSTACK/Add+Xen+and+XCP+support+

Re: [GSoC] End of bonding period, start of 'Work Period'

2013-06-17 Thread Ian Duffy
Hi Sebastian, I have updated the JIRA case for my project to add more steps so it is like Nguyen's one. Just after looking through JIRA, results are as follows: (Resolved) CLOUDSTACK-2287 - Automation:LDAP: Appears to be solved. tsp added tests for it. (Resolved) CLOUDSTACK-1172 - Ldap enhancem

Re: jenkins jobs for new docs guide

2013-06-17 Thread Prasanna Santhanam
On Sun, Jun 16, 2013 at 05:10:09PM -0400, Sebastien Goasguen wrote: > Hi, > > I have been working on some new doc guides: > > -the gsoc one is in master : docs/publican-gsoc-2013.cfg > > -in the ACS101 branch under docs/acs101/publican.cfg (tons of new goodies in > there libcloud, jclouds-cli,

Re: [GSoC] End of bonding period, start of 'Work Period'

2013-06-17 Thread Sebastien Goasguen
On Jun 17, 2013, at 4:54 AM, Ian Duffy wrote: > Hi Sebastian, > > I have updated the JIRA case for my project to add more steps so it is > like Nguyen's one. > > Just after looking through JIRA, results are as follows: > > (Resolved) CLOUDSTACK-2287 - Automation:LDAP: Appears to be solved. >

Re: [GSoC] End of bonding period, start of 'Work Period'

2013-06-17 Thread Abhinandan Prateek
On 17/06/13 2:34 PM, "Sebastien Goasguen" wrote: > >On Jun 17, 2013, at 4:54 AM, Ian Duffy wrote: > >> Hi Sebastian, >> >> I have updated the JIRA case for my project to add more steps so it is >> like Nguyen's one. >> >> Just after looking through JIRA, results are as follows: >> >> (Resol

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-17 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/ --- (Updated June 17, 2013, 10:01 a.m.) Review request for cloudstack, Abhinandan P

Re: [DISCUSS] Issue with cloudmonkey-4.1.0-0 on pypi

2013-06-17 Thread Prasanna Santhanam
On Sun, Jun 09, 2013 at 10:26:43AM -0400, David Nalley wrote: > On Sun, Jun 9, 2013 at 7:51 AM, Rohit Yadav wrote: > > Hi, > > > > I was about to test CloudStack but the cloudmonkey-4.1.0-0 release on pypi > > does not bundle failsafe api cache so when I install it I don't get any api > > commands

Re: [DISCUSS} review flow

2013-06-17 Thread Prasanna Santhanam
On Mon, Jun 17, 2013 at 09:07:54AM +, Daan Hoogland wrote: > H, > > Even though the rebase of both patches where without conflicts for a > change, this morning, I would really like to have them 'shipped'. Of > course I don't mind doing more work on them as I am taking > responsibility for the

FW: Karen Vuong

2013-06-17 Thread Karen Vuong
nee http://www.nccascience.com/aaorzqnt/RNDCHR,3,15%/ezfblvlyau/vqpivoesmuhbpy/oswigpk.htm wqj

Re: jenkins jobs for new docs guide

2013-06-17 Thread Sebastien Goasguen
On Jun 17, 2013, at 5:03 AM, Prasanna Santhanam wrote: > On Sun, Jun 16, 2013 at 05:10:09PM -0400, Sebastien Goasguen wrote: >> Hi, >> >> I have been working on some new doc guides: >> >> -the gsoc one is in master : docs/publican-gsoc-2013.cfg >> >> -in the ACS101 branch under docs/acs101/pu

Review Request: CLOUDSTACK-1047: tracking in logs using job id

2013-06-17 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11906/ --- Review request for cloudstack, Devdeep Singh, Nitin Mehta, and Sateesh Chodapune

Re: committer wanted for review

2013-06-17 Thread John Burwell
All, Please see my comments in-line below. Thanks, -John On Jun 15, 2013, at 6:11 AM, Hiroaki KAWAI wrote: > Probably we've agreed on that double slash should not > generated by cloudstack. > > If something went wrong and double slash was passed to > Winfows based NFS, the reason may A) there

Re: Review Request: (CLOUDSTACK-1301) VM Disk I/O Throttling

2013-06-17 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11782/ --- (Updated June 17, 2013, 12:03 p.m.) Review request for cloudstack, Wido den Hol

Re: Infra Issues from the IRC meeting (Wed, Jun 12)

2013-06-17 Thread Prasanna Santhanam
On Sat, Jun 15, 2013 at 01:48:57PM -0400, Chip Childers wrote: > > There's also a couple of issues here - > > 1. Does everyone know where the tests run? > Nope > > 2. Do people know how to spot the failures? > Nope > > 3. Do people know how to find the logs for the failures? > Nope > > > > If the

Re: Review Request: Selenium Headless configuration using PhantomJS

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10846/#review21978 --- Was this committed? I see a "Ship It" from Edison. - Chip Childers

Re: Review Request: Documentation changes for VMware dvSwitch and Nexus dvSwitch

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10366/#review21979 --- Radhika, Was this committed? If so, can you please close this revi

Review Request: Fix CLOUDSTACK-2168

2013-06-17 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11907/ --- Review request for cloudstack and Sebastien Goasguen. Description --- CLOU

Re: Review Request: remove dead allocations

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11324/ --- (Updated June 17, 2013, 1:40 p.m.) Review request for cloudstack and Devdeep Si

Re: committer wanted for review

2013-06-17 Thread Daan Hoogland
John, If I understand it correctly, you are stating that my take on the solution is 'not done/not the way to go'? For the record the case I solved was an instance of A, but I would not call it adding technical debt. A arose from existing code in combination of a requirement to work with a non-pos

Re: Review Request: set rpcProvider field correctly in constructor

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11325/#review21980 --- Ship it! 3a02942 - Prasanna Santhanam On May 22, 2013, 6:25 a.m.

Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread Nils Vogels
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/ --- Review request for cloudstack. Description --- This solves CLOUDSTACK-2902

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread Nils Vogels
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/ --- (Updated June 17, 2013, 1:58 p.m.) Review request for cloudstack. Description

Re: Review Request: Fix CLOUDSTACK-2168

2013-06-17 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11907/ --- (Updated June 17, 2013, 2 p.m.) Review request for cloudstack and Sebastien Goa

Review Request: Automation: (vpc network pf and lb rules) - Corrected code related to cleanup.

2013-06-17 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11909/ --- Review request for cloudstack and Prasanna Santhanam. Description --- Corr

Re: Review Request: Automation: (vpc network pf and lb rules) - Corrected code related to cleanup.

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11909/#review21981 --- Ship it! Applied but the test will fail because of a regression in

[GSOC] apache whirr startup docs

2013-06-17 Thread Sebastien Goasguen
Especially for Meng, Check out the pdf at: http://jenkins.cloudstack.org/job/docs-4.3-clients-wrappers-guide/ There is some basic apache whirr docs, that I tested on cloudstack. I also entered a bug on whirr at: https://issues.apache.org/jira/browse/WHIRR-725 A fix should be in jclouds 1.6.1 an

Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11910/ --- Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. Description

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11910/#review21982 --- Since XCP shares the same resource (XcpOssResource<-CitrixResourceBa

[ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Chip Childers
The Project Management Committee (PMC) for Apache CloudStack has asked Jayapal Reddy Uradi to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Rajesh Battala
Hearty Congratulations Jayapal Reddy :) > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, June 17, 2013 9:00 PM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi > > The Project Management Committee (PMC)

Re: enableStorageMaintenance

2013-06-17 Thread La Motta, David
Along the same lines… is there a REST command coming in 4.2 to quiesce one or multiple virtual machines? David La Motta Technical Marketing Engineer Citrix Solutions NetApp 919.476.5042 dlamo...@netapp.com On Jun 14, 2013, at 10:53 AM, "La Motta, David" mailto:da

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Prasanna Santhanam
On Mon, Jun 17, 2013 at 11:30:16AM -0400, Chip Childers wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Jayapal Reddy Uradi to become a committer and we are > pleased to announce that they have accepted. > > Being a committer allows many contributors to contribut

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Saksham Srivastava
Congrats Jayapal. -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, June 17, 2013 9:00 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi The Project Management Committee (PMC) for Apache CloudStack has asked Jaya

Re: systemvm.iso not updated in packages

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > Applied yet another fix for this from Rajesh: > > commit 6d140538c5efc394fda8a4ddc7cb72832470d0b3 > Author: Rajesh Battala > Date: Sat Jun 15 11:21:46 2013 +0530 > > CLOUDSTACK-3004: remove duplicate ssvm-check.sh >

Re: Object based Secondary storage.

2013-06-17 Thread Min Chen
Hi Tom, Thanks for your testing. Glad to hear that multipart is working fine by using Cloudian. Regarding your questions about .gz template, that behavior is as expected. We will upload it to S3 as its .gz format. Only when the template is used and downloaded to primary storage, we will us

Re: [GSOC] A short description about CloudStack Networking plugin

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 03:41:58PM +0700, Nguyen Anh Tu wrote: > Hi all, > > I made an wiki entry about the CloudStack networking design. I think it's > useful for all network plugins can follow. It's located in my gsoc project > about improving the native SDN controller. Take a look on it > > ht

Re: Object based Secondary storage.

2013-06-17 Thread John Burwell
Min, Why are objects being compressed before being sent to S3? Thanks, -John On Jun 17, 2013, at 12:24 PM, Min Chen wrote: > Hi Tom, > > Thanks for your testing. Glad to hear that multipart is working fine by > using Cloudian. Regarding your questions about .gz template, that behavior >

Re: Object based Secondary storage.

2013-06-17 Thread Min Chen
John, Let me clarify, we didn't do extra compression before sending to S3. Only when user provides a URL pointing to a compressed template during registering, we will just download that template to S3 without decompressing it afterwards as we did for NFS currently. If the register url pro

Re: committer wanted for review

2013-06-17 Thread John Burwell
Daan, Please see my comments in-line below. Thanks, -John On Jun 17, 2013, at 9:40 AM, Daan Hoogland wrote: > John, > > If I understand it correctly, you are stating that my take on the solution > is 'not done/not the way to go'? > > For the record the case I solved was an instance of A, bu

Re: systemvm.iso not updated in packages

2013-06-17 Thread Prasanna Santhanam
On Mon, Jun 17, 2013 at 12:08:54PM -0400, Chip Childers wrote: > On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > > Applied yet another fix for this from Rajesh: > > > > commit 6d140538c5efc394fda8a4ddc7cb72832470d0b3 > > Author: Rajesh Battala > > Date: Sat Jun 15 11:21:46

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Koushik Das
Congrats Jayapal > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, June 17, 2013 9:02 PM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi > > The Project Management Committee (PMC) for Apache CloudStack ha

Re: Object based Secondary storage.

2013-06-17 Thread John Burwell
Min, Cool. I just wanted to make sure we weren't compressing the template and template.properties … Thanks for the clarification, -John On Jun 17, 2013, at 12:49 PM, Min Chen wrote: > John, > > Let me clarify, we didn't do extra compression before sending to S3. > Only > when user pr

Re: systemvm.iso not updated in packages

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 10:22:44PM +0530, Prasanna Santhanam wrote: > On Mon, Jun 17, 2013 at 12:08:54PM -0400, Chip Childers wrote: > > On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > > > Applied yet another fix for this from Rajesh: > > > > > > commit 6d140538c5efc394fda8a4

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Abhinandan Prateek
Congrats Jayapal ! Well deserved. On 17/06/13 9:00 PM, "Chip Childers" wrote: >The Project Management Committee (PMC) for Apache CloudStack >has asked Jayapal Reddy Uradi to become a committer and we are >pleased to announce that they have accepted. > >Being a committer allows many contributors

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 04:59:00PM +, Kelven Yang wrote: > I'd like to kick off the official merge process. We will start the merge > process after the branch has passed necessary tests > > Kelven Can you share what testing is being run against the branch?

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21985 --- Commit 76d3c27bf4c0ab3690840e56ca162935cea91d48 in branch refs/heads

[MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Kelven Yang
I'd like to kick off the official merge process. We will start the merge process after the branch has passed necessary tests Kelven On 6/10/13 2:51 PM, "Kelven Yang" wrote: >Hi there, > >Alex Huang and I are targeting to finish the debugging process on VMsync >improvement by the end of this wee

Query String Request Authentication(QSRA) support by S3 providers

2013-06-17 Thread Min Chen
Tom filed a very good bug for ACL setting change on S3 object when users issue extractTemplate API (https://issues.apache.org/jira/browse/CLOUDSTACK-3030), and his recommendation of using Query String Request Authentication (QSRA) alternative sounds like a right approach to fix this bug. Before

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
I think it's due to this https://cwiki.apache.org/confluence/display/CLOUDSTACK/Zone-wide+primary+storage+target There are zone-wide storages, may only work with one particular hypervisor. For example, the data store created on VCenter can be shared by all the clusters in a DC, but only for vmwa

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Hi Edison, I haven't looked into this much, so maybe what I suggest here won't make sense, but here goes: What about a Hypervisor.MULTIPLE enum option ('Hypervisor' might not be the name of the enumeration...I forget). The ZoneWideStoragePoolAllocator could use this to be less choosy about if a s

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Kelven Yang
Low level classes were tested in unit tests(MessageBus, Job framework, Job dispatchers etc), interface layer changes are guarded through matching the old semantics, but changes are tested manually, we are planning to get this part of testing through BVT system after we have re-based the latest mast

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21988 --- Commit c1bb2a561b9a445241e02402232a29d75f612fde in branch refs/heads

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21989 --- Commit 9ef366e4b3d7e237503d7dd6a1f4b7af4b74b445 in branch refs/heads

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21990 --- Commit 25726277975fb9e39ade7c08d680a93d33dc16b2 in branch refs/heads

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21991 --- Thank you for submitting this patch! I've actually done the changes

RE: enableStorageMaintenance

2013-06-17 Thread Edison Su
> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Friday, June 14, 2013 7:54 AM > To: > Subject: enableStorageMaintenance > > ...works great for putting down the storage into maintenance mode (looking > forward seeing this for secondary storage as we

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 05:40:36PM +, Kelven Yang wrote: > Low level classes were tested in unit tests(MessageBus, Job framework, Job > dispatchers etc), interface layer changes are guarded through matching the > old semantics, but changes are tested manually, we are planning to get > this part

Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10858/#review21993 --- Commit a2c7d3a8a75b2cec266cef566b8828be7a1ebc72 in branch refs/heads

RE: enableStorageMaintenance

2013-06-17 Thread Edison Su
> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Monday, June 17, 2013 8:37 AM > To: > Subject: Re: enableStorageMaintenance > > Along the same lines... is there a REST command coming in 4.2 to quiesce one > or multiple virtual machines? Quiesce me

Re: Review Request: Removed String instantiation

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11751/#review21996 --- Ship it! Applied to master: ce8ada030d3150087357d7135c3877c25a4702c

Re: Review Request: String instantiation is not needed

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11750/#review21997 --- Ship it! Applied to master: 564013bec0d4356232d93ac52a3e44638578bff

Re: enableStorageMaintenance

2013-06-17 Thread La Motta, David
Yep. The purpose of quiescing is exactly as described in that document: taking a backup without powering off the VM. David La Motta Technical Marketing Engineer Citrix Solutions NetApp 919.476.5042 dlamo...@netapp.com On Jun 17, 2013, at 2:09 PM, Edison Su mail

Re: Review Request: use commons-lang StringUtils

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11767/#review21998 --- Ship it! Applied to master: c88d8fb3a2f6c418c6c7af8ff702a93bcdb2d75

Re: Review Request: removed 3 NumbersUtils methods

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11894/#review21999 --- I get the following conflicts when attempting to apply this: git am

Re: Review Request: NPE fix in StoragePoolJoinDaoImpl

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11899/#review22000 --- Ship it! Applied to master: 202cd1529054fe60acce0cce54686268797b65b

Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10858/#review22001 --- Commit 3e3e5830b45eeba16b3dacfb5475d53d8d2dee27 in branch refs/head

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
> On June 17, 2013, 3:12 p.m., Prasanna Santhanam wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 1818 > > > > > > can this be made case insensitive? so comparisons can compare as > > equalIgnoreCase?

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11910/ --- (Updated June 17, 2013, 6:37 p.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request: CLOUDSTACK-1623 MySQL Database connection can fail to "localhost" on a V6 enabled host

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9852/#review22005 --- Ship it! Applied to master: d4477ba8da60290d21afd58ce0901d12c85de3a9

Re: Review Request: CLOUDSTACK-1623 MySQL Database connection can fail to "localhost" on a V6 enabled host

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9852/#review22004 --- Commit d4477ba8da60290d21afd58ce0901d12c85de3a9 in branch refs/heads/

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Harikrishna Patnala
Congratulations Jayapal. On 17-Jun-2013, at 10:29 PM, Abhinandan Prateek wrote: > Congrats Jayapal ! Well deserved. > > On 17/06/13 9:00 PM, "Chip Childers" wrote: > >> The Project Management Committee (PMC) for Apache CloudStack >> has asked Jayapal Reddy Uradi to become a committer and we

Re: doc hacking at Hack Day at CCC13

2013-06-17 Thread Joe Brockmeier
On Sat, Jun 15, 2013, at 09:27 AM, Daan Hoogland wrote: > To all of you planning to hack away at documentation: > > There is quite a lot of commeted out code. And there is quite a lot of > code > uncommented; classes and public/protected and package scope methods > without > description. I have

Re: Review Request: remove duplicated VPC router in return value of DomainRouterDaoImpl.listByStateAndNetworkType

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10062/#review22007 --- Wei, Should this still be applied? If so, can you go ahead and do

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Ahmad Emneina
Awe yeah. Good work Jaypal. Thanks for the contributions so far and look forward to more! Ahmad On Jun 17, 2013, at 8:30 AM, Chip Childers wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Jayapal Reddy Uradi to become a committer and we are > pleased to announ

Re: Git Push Summary

2013-06-17 Thread David Nalley
What is this branch for? On Jun 17, 2013 1:27 PM, wrote: > Updated Branches: > refs/heads/master-6-17-stable [created] fc16e29f9 >

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread John Burwell
Congrats and welcome, Jayapal. On Jun 17, 2013, at 12:55 PM, Koushik Das wrote: > Congrats Jayapal > >> -Original Message- >> From: Chip Childers [mailto:chip.child...@sungard.com] >> Sent: Monday, June 17, 2013 9:02 PM >> To: dev@cloudstack.apache.org >> Subject: [ANNOUNCE] New committ

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread John Burwell
Kelven, Did this patch get pushed to Review Board? If so, what is the URL? Thanks. -John On Jun 17, 2013, at 1:40 PM, Kelven Yang wrote: > Low level classes were tested in unit tests(MessageBus, Job framework, Job > dispatchers etc), interface layer changes are guarded through matching the >

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Joe Brockmeier
On Mon, Jun 17, 2013, at 10:30 AM, Chip Childers wrote: > Please join me in congratulating Jayapal! Woot! Congrats! Best, jzb -- Joe Brockmeier j...@zonker.net Twitter: @jzb http://www.dissociatedpress.net/

Re: Git Push Summary

2013-06-17 Thread Chiradeep Vittal
David, this is a temporary scratch branch to perform some integration testing since atm the master looks stable. On 6/17/13 11:57 AM, "David Nalley" wrote: >What is this branch for? >On Jun 17, 2013 1:27 PM, wrote: > >> Updated Branches: >> refs/heads/master-6-17-stable [created] fc16e29f9 >>

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Hi Edison, How's about if I add this logic into ZoneWideStoragePoolAllocator (below)? After filtering storage pools by tags, it saves off the ones that are for any hypervisor. Next, we filter the list down more by hypervisor. Then, we add the storage pools back into the list that were for any h

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Mike, I know my thoughts will come as a galloping shock, but the idea of a hypervisor type being attached to a volume is the type of dependency I think we need to remove from the Storage layer. What attributes of a DataStore/StoragePool require association to a hypervisor type? My thought is

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I figured you might have something to say about this, John. :) Yeah, I have no idea behind the motivation for this change other than what Edison just said in a recent e-mail. It sounds like this change went in so that the allocators could look at the VM characteristics and see the hypervisor type

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Marcus Sorensen
I can understand the intention, for example templates are tied to a hypervisor because the OS installed works with that hypervisor (drivers, etc), and templates end up on primary storage. To some extent what's on the volume is hypervisor dependent, AND the storage technology is possibly hypervisor

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
There are storages which can only work with one hypervisor, e.g. Currently, Ceph can only work on KVM. And the data store created in VCenter, can only work with Vmware. > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Monday, June 17, 2013 1:12

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
What do we do, though, if the storage can only work on a subset of the ones listed in the enum? For example, XenServer and VMware. On Mon, Jun 17, 2013 at 2:27 PM, Edison Su wrote: > There are storages which can only work with one hypervisor, > e.g. Currently, Ceph can only work on KVM. And t

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Edison, For me, this issue comes back to the whole notion of the overloaded StoragePoolType. A hypervisor plugin should declare a method akin to getSupportedStorageProtocols() : ImmutableSet which the Hypervisor layer can use to filter the available DataStores from the Storage subsystem. For

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread Mike Tutkowski
FYI: I added the IOPS-capacity parameter and related code over the weekend. The final bit of work for me comes when Wei's code is merged into master and I pull it down. At that point, I need to add the GUI and API logic to support mutual exclusion of our features. On Sat, Jun 15, 2013 at 11:12

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Marcus, I am coming of the viewpoint that ImageService (ISOs and Templates), hypervisor snapshotting, and DataMotionService should moved from the Storage layer into the Hypervisor layer for the following reasons: The storage layer should treat the data it stores as opaque. These services deal

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread John Burwell
Mike, Great news. Let me know when Review Board is updated, and I will perform a second round review. Thanks, -John On Jun 17, 2013, at 4:48 PM, Mike Tutkowski wrote: > FYI: I added the IOPS-capacity parameter and related code over the weekend. > > The final bit of work for me comes when W

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread Mike Tutkowski
Sounds good I have a little bit of clean-up work to do and then I must figure out how to generate a patch (since I was using merge rather than rebase during development), so it will probably be tomorrow. On Mon, Jun 17, 2013 at 2:50 PM, John Burwell wrote: > Mike, > > Great news. Let me know

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
But currently there is no such hypervisor layer yet, and to me it's related to storage, not related to hypervisor. It's a property of a storage to support one hypervisor, two hypervisors, or all the hypervisors, not a property of hypervisor. I agree, that add a hypervisor type on the storagepool

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I think Zadara Storage will be looking to implement a plug-in in an upcoming release. They have a similar use case to SolidFire where, I believe, their primary storage represents a SAN at the zone level. On Mon, Jun 17, 2013 at 2:54 PM, Edison Su wrote: > But currently there is no such hypervi

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Edison, As part of the hack day discussion, I think we need to determine how to establish that layer and invert these dependencies. Hypervisors must know about storage and network devices. A VM is the nexus of a particular set of storage devices/volumes and network devices/interfaces. From a

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I think a hack-day session on this would be great. To me, since we're so late in the game for 4.2, I think we need to take two approaches here: 1) Short-term solution for 4.2 (that hopefully will not make future refactoring work too much more difficult than it might already be) and 2) Long-term so

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Mike, My goal is not incur further technical debt in 4.2 by adding more Storage->Hypervisor dependencies that need to be inverted. Recognizing that we are close to 4.2, the question becomes is there a simple approach that will permit this dependency to be inverted? I will dig into the code t

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Sounds good...We don't want to add technical debt if it's going to make our work a lot harder in the future. On Mon, Jun 17, 2013 at 3:14 PM, John Burwell wrote: > Mike, > > My goal is not incur further technical debt in 4.2 by adding more > Storage->Hypervisor dependencies that need to be inve

  1   2   >