Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Wei ZHOU
If nobody object, I will merge into master today. -Wei 2013/6/11 John Burwell > Mike, > > From a CloudStack perspective, it will keep implementation specific > concepts from the base data model, and provide a great test case to develop > a mechanism to capture this information in 4.3. Ideally

Re: ACS 4.1.1 release - bugfixes to backport

2013-06-13 Thread Ryan Lei
How about CLOUDSTACK-2337: [UI] Resize Volume option is enabled only for ROOT Admin? I experienced the same UI bug in the 4.1.0 release. According to Pranav's comments, "I have made a fix from the UI side the better fix would be to fix it in the API as well." -

Review Request: Fixed issue in applying private gw network acl

2013-06-13 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11856/ --- Review request for cloudstack, Abhinandan Prateek and Murali Reddy. Description

Re: Review Request: Fixed issue in applying private gw network acl

2013-06-13 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11856/#review21845 --- Ship it! Ship It! - Abhinandan Prateek On June 13, 2013, 8:54 a.

Re: Review Request: fixed not showing uuid of ip address id and network in list firewall and list egress firewall rules response

2013-06-13 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11803/#review21846 --- Ship it! Ship It! - Abhinandan Prateek On June 12, 2013, 9:46 a.

Review Request: CLOUDSTACK-2340 [AWS Style Health Checks] Response of the API listLoadBalancerRuleInstances should show the service state of a VM if health check is configured for it

2013-06-13 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11858/ --- Review request for cloudstack, Murali Reddy and Ram Ganesh. Description ---

Re: Summary of IRC meeting in #cloudstack-meeting, Wed Jun 12 17:08:56 2013

2013-06-13 Thread Daan Hoogland
Reading the meeting summary, I learned about the [off] directive the hard way. Is there a irc-etiquette for dummies somewhere that handles ASFBot and other things newbees should know? On Wed, Jun 12, 2013 at 7:55 PM, ASF IRC Services < asf...@wilderness.apache.org> wrote: > Members present: Anim

Re: Review Request: Automation: Network life cycle tests

2013-06-13 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11835/ --- (Updated June 13, 2013, 10:30 a.m.) Review request for cloudstack, Prasanna San

Re: Review Request: Automation: Network life cycle tests

2013-06-13 Thread Gaurav Aradhye
> On June 12, 2013, 3:44 p.m., Prasanna Santhanam wrote: > > test/integration/smoke/test_network.py, line 149 > > > > > > account is not derefenced as account.account anymore. Was this change > > unintended? All insta

Review Request: Fix for CLOUDSTACK-2502 and CLOUDSTACK-2939

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

Re: Review Request: Fix for CLOUDSTACK-2502 and CLOUDSTACK-2939

2013-06-13 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11860/#review21848 --- Ship it! Ship It! - Abhinandan Prateek On June 13, 2013, 10:42 a

Re: Review Request: Fix for CLOUDSTACK-2502 and CLOUDSTACK-2939

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

Re: Review Request: Fix for CLOUDSTACK-2502 and CLOUDSTACK-2939

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

Review Request: double slash fix for windows based nfs servers [CLOUDSTACK-2968]

2013-06-13 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11861/ --- Review request for cloudstack. Description --- double slash breaks windows

committer wanted for review

2013-06-13 Thread Daan Hoogland
H, Can someone look at Review Request #11861 for me please? Thanks, Daan Hoogland

Review Request: CLOUDSTACK-2398 , ssvm-check failed with permission error

2013-06-13 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11862/ --- Review request for cloudstack. Description --- In the new system vm templa

Re: Review Request: generalisation of network code (needed for CLOUDSTACK-1532)

2013-06-13 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10970/ --- (Updated June 13, 2013, 1:09 p.m.) Review request for cloudstack, Murali Reddy,

Re: committer wanted for review

2013-06-13 Thread Musayev, Ilya
Daan, Has this been tested on linux NFS? On 6/13/13 8:15 AM, "Daan Hoogland" wrote: >H, > >Can someone look at Review Request >#11861 for me please? > >Thanks, >Daan Hoogland

RE: committer wanted for review

2013-06-13 Thread Daan Hoogland
It has been tested on nexenta (sunos). I have no linux on my hands for this. -Original Message- From: Musayev, Ilya [mailto:imusa...@webmd.net] Sent: donderdag 13 juni 2013 15:27 To: dev@cloudstack.apache.org Subject: Re: committer wanted for review Daan, Has this been tested on linux

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread La Motta, David
Looking at this in more detail, that's not going to do it. If we have to create a template and call deployVirtualMachine, that will do--intentionally--what it was intended to do: spin up a new VM in the hypervisor and allocate a chunk of disk out of primary storage. What we are looking to do,

RE: committer wanted for review

2013-06-13 Thread Daan Hoogland
Ilya, Why do you ask? It only touches the database, not the nfs server itself! Regards, Daan Hoogland -Original Message- From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] Sent: donderdag 13 juni 2013 15:32 To: 'dev@cloudstack.apache.org' Subject: RE: committer wanted for review

Review Request: Change getListAll() within the LDAPConfigCmd.java file to return a boolean instead of a string.

2013-06-13 Thread Ian Duffy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11863/ --- Review request for cloudstack. Summary (updated) - Change getL

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread John Burwell
Wei, There are open questions on the thread regarding mutual exclusion of hypervisor throttled I/O and storage provisioned IOPS. We need to understand how and where it will be implemented in both the UI and service layers. Also, can you resend the Review Board review? My email search skills hav

UI Plugin Questions

2013-06-13 Thread SuichII, Christopher
Hi everyone, first time poster here… I've started working on a UI Plugin and have come across a few things I'd like to inquire about and run by some of you: -Internationalization: I see that the UI has a system for internationalization, but I don't see a way for UI plugins to contribute strings

Re: Git question about applying patch files

2013-06-13 Thread John Burwell
Prasanna, +1 to using rebase on feature branches. At least as I understand things and have experienced rebase, it preserves all commits on the feature branch. For Review Board and master merges, those commits need to be collapsed, or in git parlance, squashed. The script I referenced below squ

Re: Review Request: Automation: Network life cycle tests

2013-06-13 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11835/#review21859 --- Ship it! ed1980c7417216efa541c978d4233391d1659b79 - Prasanna Santh

Re: Summary of IRC meeting in #cloudstack-meeting, Wed Jun 12 17:08:56 2013

2013-06-13 Thread Joe Brockmeier
On Thu, Jun 13, 2013, at 04:48 AM, Daan Hoogland wrote: > Reading the meeting summary, I learned about the [off] directive the hard > way. Is there a irc-etiquette for dummies somewhere that handles ASFBot > and other things newbees should know? There's a manual for ASFBot here: http://wilderness

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Wei ZHOU
John, Please review the code on https://reviews.apache.org/r/11782 The storage provisioned IOPS does not affect hypervisor throttled I/O, I think. Mike may change UI and java code for storage provisioned IOPS after the merge. -Wei 2013/6/13 John Burwell > Wei, > > There are open questions on

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread Marcus Sorensen
CreateCommand in the agent code is responsible for making a template into a new volume. With Xen and VMware, you're limited to what the hypervisor is capable of as far as storage cloning to create a new copy from an existing template on your primary storage. With KVM, you can create your own storag

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Mike Tutkowski
Yeah, I think it's fine for Wei to merge in his changes. I can then fetch and merge into my branch and add the additional GUI and API code for mutual exclusion. On Thu, Jun 13, 2013 at 9:34 AM, Wei ZHOU wrote: > John, > > Please review the code on https://reviews.apache.org/r/11782 > The storag

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread La Motta, David
Cool, we'll take a look at CreateCommand. I like your suggestion to tweak the execute method to accomplish this. We are literally getting our feet wet at this point, and have some ideas that we are looking to implement in a NetApp plugin to CS. Hopefully we'll get to meet some of you at the CC

tests involving deploying a vm fail

2013-06-13 Thread Shane Witbeck
Hi all, I'm attempting to run the following: mvn -Pdeveloper,marvin.test -Dmarvin.config=setup/dev/advanced.cfg -pl :cloud-marvin integration-test from instructions [1]. It seems all tests involving deploying a VM fail for me [2]. I've also tried running the /smoke/test_deploy_vm.py test in i

Object_Store storage refactor Meeting Notes

2013-06-13 Thread John Burwell
All, Edison Su, Min Chen, Animesh Chaturvedi, and myself met via teleconference on 11 June 2013 @ 1:30 PM EDT. The goal of the meeting was determine the path forward for merging the object_store branch by the 4.2 freeze date, 30 June 2013. The conversation focused on the following topics:

Re: Git question about applying patch files

2013-06-13 Thread Mike Tutkowski
Thanks, everyone! Once I finish up implementing review suggestions, I can try again with building a squashed patch file. On Thu, Jun 13, 2013 at 8:41 AM, John Burwell wrote: > Prasanna, > > +1 to using rebase on feature branches. > > At least as I understand things and have experienced rebase,

RE: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Sudha Ponnaganti
Thanks John for summary. From QA stand point it would make sense to merge once - assigned test cases are executed and pass rate is on par with release criteria ( test plans published and execution results are being posted) - automation runs are successful and shows same pass rate as Master - blo

RE: UI Plugin Questions

2013-06-13 Thread Pranav Saxena
Please see comments inline - Thanks, Pranav -Original Message- From: SuichII, Christopher [mailto:chris.su...@netapp.com] Sent: Thursday, June 13, 2013 8:11 PM To: dev@cloudstack.apache.org Subject: UI Plugin Questions Hi everyone, first time poster here... I've started working on a UI

Regarding ssvm-check script

2013-06-13 Thread Rajesh Battala
Hi All, While fixing an issue ( https://reviews.apache.org/r/11862/ )in ssvm-check script I figured out some issues. 1.There are two ssvm_check scripts(duplicates). ./services/console-proxy/server/scripts/ssvm-check.sh ./services/secondary-storage/scripts/ssvm-check.sh When building the code, t

Re: Object_Store storage refactor Meeting Notes

2013-06-13 Thread John Burwell
Sudha, The current plan is to merge once. We explored the feasibility of decomposing it into independent, testable chunks, and determined it was not possible. Thanks, -John On Jun 13, 2013, at 12:54 PM, Sudha Ponnaganti wrote: > Thanks John for summary. From QA stand point it would make sen

RE: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Sudha Ponnaganti
Sure John! I am referring to the point that even for one time merge, let us go through the planned QA cycle. Thanks /sudha -Original Message- From: John Burwell [mailto:jburw...@basho.com] Sent: Thursday, June 13, 2013 10:14 AM To: dev@cloudstack.apache.org Subject: Re: Object_Store s

Re: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Chip Childers
First, thanks for bringing this back to the list. I'm +1 on the technical approach. A couple of thoughts though, just so that we make sure that we keep operating in the right manner as an Apache project: Let's be careful about declaring something a "decision" or that something was "determined" w

Re: Regarding ssvm-check script

2013-06-13 Thread Chip Childers
On Thu, Jun 13, 2013 at 05:10:06PM +, Rajesh Battala wrote: > Hi All, > While fixing an issue ( https://reviews.apache.org/r/11862/ )in ssvm-check > script I figured out some issues. > > 1.There are two ssvm_check scripts(duplicates). > > ./services/console-proxy/server/scripts/ssvm-check.sh

RE: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Animesh Chaturvedi
> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Thursday, June 13, 2013 10:43 AM > To: dev@cloudstack.apache.org > Subject: Re: Object_Store storage refactor Meeting Notes > > First, thanks for bringing this back to the list. I'm +1 on the > technica

RE: UI Plugin Questions

2013-06-13 Thread Brian Federle
>> I know the infrastructure code is too huge to handle and that is why all new >> features which would be developed would be developed in the form of UI >> modules (Eg - CISCO asa 100 support / vnmcNetworkProvider / VPC) . That is correct -- the infrastructure section handles a lot of central p

Re: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Chip Childers
On Thu, Jun 13, 2013 at 05:52:01PM +, Animesh Chaturvedi wrote: > > > > -Original Message- > > From: Chip Childers [mailto:chip.child...@sungard.com] > > Sent: Thursday, June 13, 2013 10:43 AM > > To: dev@cloudstack.apache.org > > Subject: Re: Object_Store storage refactor Meeting Not

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread Chip Childers
On Thu, Jun 13, 2013 at 04:17:10PM +, La Motta, David wrote: > Cool, we'll take a look at CreateCommand. I like your suggestion to tweak > the execute method to accomplish this. > > We are literally getting our feet wet at this point, and have some ideas that > we are looking to implement i

RE: tests involving deploying a vm fail

2013-06-13 Thread Prachi Damle
Shane, Can you share the management server log file? Prachi -Original Message- From: Shane Witbeck [mailto:sh...@digitalsanctum.com] Sent: Thursday, June 13, 2013 9:18 AM To: dev@cloudstack.apache.org Subject: tests involving deploying a vm fail Hi all, I'm attempting to run the foll

Re: Object_Store storage refactor Meeting Notes

2013-06-13 Thread David Nalley
On Thu, Jun 13, 2013 at 1:55 PM, Chip Childers wrote: > On Thu, Jun 13, 2013 at 05:52:01PM +, Animesh Chaturvedi wrote: >> >> >> > -Original Message- >> > From: Chip Childers [mailto:chip.child...@sungard.com] >> > Sent: Thursday, June 13, 2013 10:43 AM >> > To: dev@cloudstack.apache.o

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

2013-06-13 Thread John Burwell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11782/#review21865 --- api/src/com/cloud/agent/api/to/VolumeTO.java

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread Kelcey Jamison Damage
Hey David, not sure if I mentioned this, but there are some guys at NetApp in Calgary that are interested in CloudStack development as well - Original Message - From: "Chip Childers" To: dev@cloudstack.apache.org Sent: Thursday, June 13, 2013 10:56:51 AM Subject: Re: Cloning VMs in Cloud

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread La Motta, David
Hey Kelcey, you'd mentioned there was somebody, but never got any specifics. You can share names outside of this thread if you'd like, and I'll reach out to them internally. If not I'll have to go on a witch hunt :-) David La Motta Technical Marketing Engineer Citrix Solutions NetApp 919.47

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread John Burwell
Wei, I published my review. I didn't see any code to validating the rate values (e.g. values greater than 0, values less than an maximum value). Did I miss it? I also noticed that 0 is being used when no value has been specified. I recommend using the Long type rather primitive long in order

Re: Object_Store storage refactor Meeting Notes

2013-06-13 Thread John Burwell
David and Chip, There was no intention to side step to the list. I will be more careful in my phrasing in the future. I apologize for the lack of clarity, -John On Jun 13, 2013, at 1:58 PM, David Nalley wrote: > On Thu, Jun 13, 2013 at 1:55 PM, Chip Childers > wrote: >> On Thu, Jun 13, 2013

Re: UI Plugin Questions

2013-06-13 Thread SuichII, Christopher
I'm using require.js to get some custom strings into the dictionary now, so I'll look into extending the plugin api to add internationalized string js files for plugins while I'm in there. Some helpers with the Infrastructure would be great, but maybe what I meant wasn't clear. We don't want to co

Hack Day at CloudStack Collaboration Conference

2013-06-13 Thread Joe Brockmeier
Hey all, As you know, the conference is coming up in less than two weeks. The first day is going to be a "hack day" using an un-conference/BarCamp type structure where we ask attendees to set the agenda and have spaces set aside to work on things or have more interactive sessions to hammer out ide

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread Kelcey Jamison Damage
I was tipped off by one of the Vancouver guys by the name of Dave Dye. He never told me who specifically or which team, so I'm afraid it's a witch hunt, or maybe get in touch with Dave, he might know who to connect you to. - Original Message - From: "David La Motta" To: "" Sent: Thursd

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Mike Tutkowski
Hi Edison, I notice after I updated from master that Hypervisor Type is now a required parameter for creating a storage pool. Should I just use HypervisorType.Any? Thanks! On Thu, Jun 13, 2013 at 12:21 PM, John Burwell wrote: > Wei, > > I published my review. I didn't see any code to valida

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

2013-06-13 Thread Wei Zhou
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > John, The validation of input fields is in CreateDiskOfferingCmd.java and CreateServiceOfferingCmd.java, like: public long getBytesReadRate() { return (bytesReadRate == null) || (bytesReadRate < 0) ? 0 : bytesReadRate; } I

Mac OS X Terminal Question

2013-06-13 Thread Mike Tutkowski
Hi, Has anyone else noticed in the Mac OS X Terminal that CTRL-Cing out of the CS MS sometimes puts you back to a prompt that works, but does not show you the text you're typing? If so, do you know how to fix this issue? Thanks! -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.*

Re: Cloning VMs in CloudStack, or not...

2013-06-13 Thread La Motta, David
Cool. I'll circle around internally. Thanks. David La Motta Technical Marketing Engineer Citrix Solutions NetApp 919.476.5042 dlamo...@netapp.com On Jun 13, 2013, at 2:38 PM, Kelcey Jamison Damage mailto:kel...@bbits.ca>> wrote: I was tipped off by one of the

Re: Mac OS X Terminal Question

2013-06-13 Thread Joe Brockmeier
On Thu, Jun 13, 2013, at 02:21 PM, Mike Tutkowski wrote: > If so, do you know how to fix this issue? I have not noticed it there, but other programs do this. If you get this, try typing "reset" and hitting enter, that will sometimes help reset the terminal. Best, jzb -- Joe Brockmeier j...@zonk

RE: UI Plugin Questions

2013-06-13 Thread Pranav Saxena
Widgetizing the various wizards won't be targeted for 4.2 for sure . It would mostly be for some future release ..may be 4.3 or later since it would require pretty good amount of refactoring apart from a brand new implementation . Thanks, Pranav -Original Message- From: SuichII, Christ

Review Request: Add GSoC proposal to docs

2013-06-13 Thread Shiva Teja Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11866/ --- Review request for cloudstack and Sebastien Goasguen. Description --- Adde

Re: Mac OS X Terminal Question

2013-06-13 Thread Chip Childers
On Thu, Jun 13, 2013 at 02:41:35PM -0500, Joe Brockmeier wrote: > On Thu, Jun 13, 2013, at 02:21 PM, Mike Tutkowski wrote: > > If so, do you know how to fix this issue? > > I have not noticed it there, but other programs do this. If you get > this, try typing "reset" and hitting enter, that will s

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

2013-06-13 Thread John Burwell
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

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

2013-06-13 Thread Wei Zhou
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

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

2013-06-13 Thread John Burwell
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

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

2013-06-13 Thread John Burwell
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

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

2013-06-13 Thread Wei Zhou
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

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

2013-06-13 Thread Wei Zhou
> On June 13, 2013, 6:01 p.m., John Burwell wrote: > > > > Wei Zhou wrote: > John, > > The validation of input fields is in CreateDiskOfferingCmd.java and > CreateServiceOfferingCmd.java, like: > public long getBytesReadRate() { > return (bytesReadRate == null)

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Mike Tutkowski
Actually, I am noticing some new behavior around picking a storage pool for zone-wide storage. The current implementation that I've brought down from master no longer finds storage for me because my plug-in is zone wide and not associated with a hypervisor. Edison? On Thu, Jun 13, 2013 at 1:13

Re: Mac OS X Terminal Question

2013-06-13 Thread Mike Tutkowski
Thanks, guys Unfortunately it didn't respond properly with either option. :( On Thu, Jun 13, 2013 at 1:43 PM, Chip Childers wrote: > On Thu, Jun 13, 2013 at 02:41:35PM -0500, Joe Brockmeier wrote: > > On Thu, Jun 13, 2013, at 02:21 PM, Mike Tutkowski wrote: > > > If so, do you know how to fix t

Re: Test halting build every now and then

2013-06-13 Thread Mike Tutkowski
Not that this is a long-term solution or anything, but I just commented out the test. :) On Wed, Jun 12, 2013 at 3:57 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > This is what nslookup 127.0.0.1 brings up for me: > > Server: 172.16.1.2 > Address: 172.16.1.2#53 > > 1.0.0.127.in-add

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread John Burwell
Mike, Please see my comment in-line below. Thanks, -John On Jun 13, 2013, at 1:22 AM, Mike Tutkowski wrote: > Hi John, > > I've put comments below in red. > > Thanks! > > > On Wed, Jun 12, 2013 at 10:51 PM, John Burwell wrote: > >> Mike, >> >> First and foremost, we must ensure that th

Re: Hack Day at CloudStack Collaboration Conference

2013-06-13 Thread Mike Tutkowski
Hi, I was wondering if we have the following documentation (below). If not, I was thinking it might be a good session to discuss and start in (at a high level) on developing such documentation. 1) Class diagrams highlighting the main classes that make up the Compute, Networking, and Storage compo

RE: UI Development

2013-06-13 Thread Soheil Eizadi
There is not a use case, but from the UI Plugin design description, looks like it is aimed at extending the CS UI after it is in production. I was thinking that there might be a requirement for having some Server side code running in CS to support this UI Plugin. The current CS Plugins are buil

Re: [ACS42] Release Status Update

2013-06-13 Thread Chiradeep Vittal
So, "open" features are theoretically "in progress" but simply not updated in Jira? On 6/12/13 4:46 PM, "Animesh Chaturvedi" wrote: >Folks > > >The status for features or improvement is depicted in table below > >|-+---+---| >| New Features / Improveme

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
Comments below in red. Thanks On Thu, Jun 13, 2013 at 2:54 PM, John Burwell wrote: > Mike, > > Please see my comment in-line below. > > Thanks, > -John > > On Jun 13, 2013, at 1:22 AM, Mike Tutkowski > wrote: > > > Hi John, > > > > I've put comments below in red. > > > > Thanks! > > > > > > O

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

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

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
My thinking is, for 4.2, while not ideal, we will need to put some burden on the admin to configure a Disk Offering in a way that makes sense. For example, if he wants to use storage QoS with supported Min and Max values, he'll have to put in a storage tag that references the SolidFire primary stor

Re: Review Request: Protect VNC port with password on KVM.

2013-06-13 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11058/#review21882 --- Ship it! Ship It! - edison su On May 13, 2013, 10:32 p.m., Fang

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
So, here's my suggestion for 4.2: Accept the values as they are currently required (four new fields for Wei's feature or two new fields for mine). The Add Disk Offering dialog needs three new radio buttons: 1) No QoS 2) Hypervisor QoS 3) Storage Qos The admin needs to specify storage tags tha

RE: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Edison Su
How about set hypervisorType to Any? Haven't take a look at the master change yet. From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Thursday, June 13, 2013 1:41 PM To: dev@cloudstack.apache.org Cc: Edison Su Subject: Re: [MERGE] disk_io_throttling to MASTER (Second Round) Actuall

Re: [MERGE] disk_io_throttling to MASTER (Second Round)

2013-06-13 Thread Mike Tutkowski
Yeah, I did that. :) I had to change some allocator code, too, because it didn't like zone-wide storage being set to hypervisor any. On Thu, Jun 13, 2013 at 3:45 PM, Edison Su wrote: > How about set hypervisorType to Any? Haven’t take a look at the master > change yet. > > ** ** > > *From

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread John Burwell
Mike, See my comments in-line below. Thanks, -John On Jun 13, 2013, at 5:31 PM, Mike Tutkowski wrote: > My thinking is, for 4.2, while not ideal, we will need to put some burden > on the admin to configure a Disk Offering in a way that makes sense. For > example, if he wants to use storage Qo

RE: Review Request: Change getListAll() within the LDAPConfigCmd.java file to return a boolean instead of a string.

2013-06-13 Thread Musayev, Ilya
Ian, Hope all is well, Please consider a case when you get a very large userbase - i.e. what would happen when you have 1000+ users in ldap/active directory? In past I've seen other products crash because the logic could not handle a where large user base. Regards ilya > -Original Messa

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread John Burwell
Mike, Overall, I agree with the steps to below for 4.2. However, we may want to throw an exception if we can not fulfill a requested QoS. If the user is expecting that the hypervisor will provide a particular QoS, and that is not possible, it seems like we should inform them rather than silen

Re: Review Request: Change getListAll() within the LDAPConfigCmd.java file to return a boolean instead of a string.

2013-06-13 Thread Ian Duffy
Hi, As far as I'm aware listAll being passed to LDAPConfigCmd only returns the LDAP configuration stored within the cloudstack database, not user accounts so it shouldn't affect it. I am looking at adding an LDAPUserListCmd with my GSoC project(Ldap user provisioning) in order to enable AJAX call

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
Comments below in red. Thanks On Thu, Jun 13, 2013 at 3:58 PM, John Burwell wrote: > Mike, > > Overall, I agree with the steps to below for 4.2. However, we may want to > throw an exception if we can not fulfill a requested QoS. If the user is > expecting that the hypervisor will provide a p

RE: [ACS42] Release Status Update

2013-06-13 Thread Animesh Chaturvedi
> -Original Message- > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] > Sent: Thursday, June 13, 2013 2:18 PM > To: dev@cloudstack.apache.org > Subject: Re: [ACS42] Release Status Update > > So, "open" features are theoretically "in progress" but simply not > updated in Jira

Re: Review Request: SolidFire storage plug-in and enhancements to the storage framework and GUI

2013-06-13 Thread Mike Tutkowski
> On June 12, 2013, 11:15 p.m., John Burwell wrote: > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java, > > line 6494 > > > > > > To reduce the conditional nesting, check if (!c

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

2013-06-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11582/#review21883 --- Commit 4612656de86a4a0258852c3fb89724bda6b612b3 in branch refs/heads

Re: Review Request: Fix a logging issue in my previsous hypervisor template code.

2013-06-13 Thread Fang Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11052/ --- (Updated June 13, 2013, 10:55 p.m.) Review request for cloudstack, edison su an

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
> > Overall, I agree with the steps to below for 4.2. However, we may want to > throw an exception if we can not fulfill a requested QoS. If the user is > expecting that the hypervisor will provide a particular QoS, and that is > not possible, it seems like we should inform them rather than silen

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
Alternatively, we could always add a new table, storage_features. It could contain three columns: Our typical ID column, a foreign key that maps back to the storage_pool table, and a column for the feature's name (this string would map to an enumeration in the codebase). Ex. storage_pool ID

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread John Burwell
Mike, Please see my comments in-line below. Thanks, -John On Jun 13, 2013, at 6:09 PM, Mike Tutkowski wrote: > Comments below in red. > > Thanks > > > On Thu, Jun 13, 2013 at 3:58 PM, John Burwell wrote: > >> Mike, >> >> Overall, I agree with the steps to below for 4.2. However, we may

Using "In Progress" status in JIRA

2013-06-13 Thread Animesh Chaturvedi
Folks It seems that we do not use "In Progress" status in JIRA as often as we should. Issues seem to change from "Open" to "Resolved" directly. IMHO marking an issue "In Progress" provides much better visibility and helps communicate to community that you are working on that item. If for whate

Re: ACS 4.1.1 release - bugfixes to backport

2013-06-13 Thread Ryan Lei
How about CLOUDSTACK-2337: [UI] Resize Volume option is enabled only for ROOT Admin? The affected version says 4.2.0, but I experienced the same UI bug in the 4.1.0 release, as well. According to Pranav's comments, "I have made a fix from the UI side the better fix would be to fix it in the API as

RE: Object_Store storage refactor Meeting Notes

2013-06-13 Thread Animesh Chaturvedi
> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Thursday, June 13, 2013 10:55 AM > To: dev@cloudstack.apache.org > Subject: Re: Object_Store storage refactor Meeting Notes > > On Thu, Jun 13, 2013 at 05:52:01PM +, Animesh Chaturvedi wrote: > > >

Re: [MERGE] disk_io_throttling to MASTER

2013-06-13 Thread Mike Tutkowski
Yeah, I'm not sure I could come up with anything near an accurate assessment of how many IOPS are currently available on the SAN (or even a total number that are available for volumes). Not sure if there's yet an API call for that. If I did know this number (total number of IOPS supported by the S

Re: committer wanted for review

2013-06-13 Thread Hiroaki KAWAI
I'd suggest: - fix the generation of double slash itself - auto-fix may happen where it is really required - and if auto-fix happens, it should log it with WARN level. (2013/06/13 21:15), Daan Hoogland wrote: H, Can someone look at Review Request #11861 fo

  1   2   >