RE: CloudStack Server Memory Requirements

2013-09-27 Thread Geoff Higginbottom
All, Thanks for all the replies, now a simple question, what do I need to change to test it? Regards Geoff Higginbottom D: +44 20 3603 0542 | S: +44 20 3603 0540 | M: +447968161581 geoff.higginbot...@shapeblue.com -Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com]

Re: Libvirt-java 0.5.0 has been released

2013-09-27 Thread Wei ZHOU
Mike, cloudstack-agent-upgrade is missing in cloudstack-agent installation. I added it in commit a6bfd9602129d8ae308ba58f36623c04826e15ca [root@weizhou-centos master(4.2)]# git show a6bfd9602129d8ae308ba58f36623c04826e15ca commit a6bfd9602129d8ae308ba58f36623c04826e15ca Author: Wei Zhou Date: F

Re: LocalHostEndPoint seems to get called

2013-09-27 Thread Daan Hoogland
Min, I started the MS with the hosts and the ssvm running and while it was registered as running in the db. It still keeps reporting the ssvm as not running: INFO [c.c.h.x.d.XcpServerDiscoverer] (AgentTaskPool-1:ctx-e0e62e7a) Host: mccdxen2 connected with hypervisor type: XenServer. Checking CIDR

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread SuichII, Christopher
Brian - The new style looks great, but I'd like to repeat someone else's question: Are we getting rid of the icons on the nav bar? As a plugin dev, it would be really nice to keep our company logo by our UI plugin. Shiva & Sebastien - What impact would this angular.js project have on UI plugins

ovirt/RHEV-M support

2013-09-27 Thread Jithin Raju
Hi, Is there any plan to support ovirt/RHEV-M ? Thanks, Jithin

Re: Libvirt-java 0.5.0 has been released

2013-09-27 Thread Mike Tutkowski
Great - thanks, Wei! On Fri, Sep 27, 2013 at 2:34 AM, Wei ZHOU wrote: > Mike, > cloudstack-agent-upgrade is missing in cloudstack-agent installation. > I added it in commit a6bfd9602129d8ae308ba58f36623c04826e15ca > > [root@weizhou-centos master(4.2)]# git show > a6bfd9602129d8ae308ba58f36623c0

Re: ovirt/RHEV-M support

2013-09-27 Thread David Nalley
I don't think that anyone is currently working on it, but we'd love to see it if you are interested in working on it. --David On Fri, Sep 27, 2013 at 6:14 AM, Jithin Raju wrote: > Hi, > > Is there any plan to support ovirt/RHEV-M ? > > Thanks, > Jithin

Re: ovirt/RHEV-M support

2013-09-27 Thread Laszlo Hornyak
Hi, Exactly what support do you mean? Migration from rhevm to CS? :-) On Fri, Sep 27, 2013 at 3:14 PM, Jithin Raju wrote: > Hi, > > Is there any plan to support ovirt/RHEV-M ? > > Thanks, > Jithin > -- EOF

Re: Review Request 13901: fix long cast to double

2013-09-27 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13901/#review26432 --- ping? - Laszlo Hornyak On Aug. 29, 2013, 12:26 p.m., Laszlo Horny

Re: ovirt/RHEV-M support

2013-09-27 Thread Jithin Raju
Hi, I meant managing ovirt managed resources using cs. Thanks, Jithin On Fri, Sep 27, 2013 at 8:51 PM, Laszlo Hornyak wrote: > Hi, > > Exactly what support do you mean? Migration from rhevm to CS? :-) > > > On Fri, Sep 27, 2013 at 3:14 PM, Jithin Raju wrote: > > > Hi, > > > > Is there any pl

Re: ovirt/RHEV-M support

2013-09-27 Thread Laszlo Hornyak
Ok, so you want to keep your rhevm but you want to use (start/stop/create/delete/etc) its resources (CS, storage, networks, vms) in CS? On Fri, Sep 27, 2013 at 6:13 PM, Jithin Raju wrote: > Hi, > > I meant managing ovirt managed resources using cs. > > Thanks, > Jithin > > > > On Fri, Sep 27, 2

Re: LocalHostEndPoint seems to get called

2013-09-27 Thread Min Chen
Can you file a bug and attach your db dump and ms log with it? Thanks -min On 9/27/13 3:14 AM, "Daan Hoogland" wrote: >Min, I started the MS with the hosts and the ssvm running and while it >was registered as running in the db. It still keeps reporting the ssvm >as not running: > >INFO [c.c.h.

Network.{Provider,Service,Capability} string names don't match constant names

2013-09-27 Thread Darren Shepherd
For Network.{Provider,Service,Capability} the getName() does not always match the constant names. For example Capability.SupportedVpnProtocols, the getName() is "SupportedVpnTypes". For Capability.SupportedLBAlgorithms the getName() is "SupportedLbAlgorithms". Can we just clean up all those cons

Scalable Backup and Recovery

2013-09-27 Thread SuichII, Christopher
I'd like to start a discussion around the direction of scalable backup and recovery in CloudStack. Currently, the only want to backup and recover vms is by setting up a schedule or manually snapshotting up individual vm disks or manually snapshotting vms. Unfortunately, I don't believe this is a

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread Kelcey Jamison Damage
We should make a CSS skinning bin/repo so people can easily share their skins. This could even just be a Wiki section. I have been playing with a dark(Greys) themed flatter skin that is designed for comfortable 1920 x 1080 usage. I think us having a fairly easily skin-able UI will attract a lot

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Edison Su
That's nice! Do you need help to setup a demo, or coding? Just finished read < Mastering Web Application Development with AngularJS >, so trying to mastering something:) > -Original Message- > From: Shiva Teja [mailto:shivate...@gmail.com] > Sent: Thursday, September 26, 2013 9:53 PM > T

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread sebgoa
On Sep 27, 2013, at 2:14 PM, "SuichII, Christopher" wrote: > Brian - The new style looks great, but I'd like to repeat someone else's > question: Are we getting rid of the icons on the nav bar? As a plugin dev, it > would be really nice to keep our company logo by our UI plugin. > > Shiva &

Re: Scalable Backup and Recovery

2013-09-27 Thread Kelcey Jamison Damage
I agree this needs to be approached. I find it quite frustrating to have a VM with 4-5 volumes and not be able to backup all 5 volumes as a set. Plus the restore operation gets messy when you have to restore 5 volumes and re-attach them to the VM. The user and domain sets you discuss below I ca

Review Request 14360: Make getName() match the constant names for Network Capability, and Service

2013-09-27 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14360/ --- Review request for cloudstack, Alena Prokharchyk and Chiradeep Vittal. Reposito

Re: Network.{Provider,Service,Capability} string names don't match constant names

2013-09-27 Thread Darren Shepherd
I created https://reviews.apache.org/r/14360/ for this. I found that changing the provider names would be rather impactful, but changing the Capability and Service didn't seem to matter. I can't find any reference to the string constant names anywhere. The unit test I added will ensure that futu

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Brian Federle
Yeah, I am definitely planning on JS cleanup, though not for 4.3. That will be a much more complex project, and needs to be a more iterative refactor since we have innumerable lines of complex business logic to take account for in there. I've been learning about AngularJS as well, and it would d

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Brian Federle
Right now the plan was to remove the icons, though if people think that they are important to usability then we can definitely put them back in. I'm thinking flat icons though, which would look better with the new design. I'll play around with it and maybe post a screenshot with icons included.

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread Tracy Phillips
If you could use font icons, that would be really nice.. (MIT License) http://fortawesome.github.io/Font-Awesome/icons/ (Apache License) http://getbootstrap.com/components/#glyphicons On Fri, Sep 27, 2013 at 1:53 PM, Brian Federle wrote: > Right now the plan was to remove the icons, though if

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Brian Federle
I can definitely look into that...and it wouldn't require Photoshopping a large sprite sheet, so much easier for maintenance :) -Brian From: Tracy Phillips [tracp...@mantoso.com] Sent: Friday, September 27, 2013 11:01 AM To: dev@cloudstack.apache.org Subje

NetworkElement and other *ServiceProviders often have useless return values

2013-09-27 Thread Darren Shepherd
On NetworkElement and many other *ServiceProvider interfaces (ie DhcpServiceProvider), the return value for the methods are a boolean, but those booleans are never checked. I would like to clean up all these interfaces and put the majority of the methods to return "void." Would there be any objec

RE: NetworkElement and other *ServiceProviders often have useless return values

2013-09-27 Thread Alex Huang
Shouldn't be a problem. They were Boolean previously to mean this networkelement does not know how to apply the rule, move on to the next one because previously it was really more like a broadcast to all networkelements. Whichever networkelment believes they should apply the rule, then applies

RE: Problem creating VirtualRouter on XenServer 6.2 due to problem with copy_vhd_from_secondarystorage.sh

2013-09-27 Thread Soheil Eizadi
I traced this to a problem with ListTemplateCommand() that is returning what I assumed is a cached value 10.48.15.22:/home/nfs/secondary/template/tmpl/1/1/34b74283-916e-4562-82fd-082b387b51c5.vhd, the new secondary storage has a new template now: template/tmpl/1/1/170c3425-6078-479e-8843-3f8fdc

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Brian Federle
Hey Chip, the BG is basically a layered background with a gradient and overlay pattern on top. I can send along the PNGs I used for those and a snippet of CSS, and you can apply them to the site. I'm also using Noto Sans for the font (ASF 2.0 licensed), if you want to use that as well. What I

RE: [DISCUSS] UI: New look and feel

2013-09-27 Thread Edison Su
Is there anybody like Win8 style UI: especially for the icons: http://aozora.github.io/bootmetro/docs/docs-advanced-components.html > -Original Message- > From: Brian Federle [mailto:brian.fede...@citrix.com] > Sent: Friday, September 27, 2013 10:53 AM > To: dev@cloudstack.apache.org; Son

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread Chip Childers
On Fri, Sep 27, 2013 at 06:31:10PM +, Brian Federle wrote: > Hey Chip, the BG is basically a layered background with a gradient and > overlay pattern on top. I can send along the PNGs I used for those and a > snippet of CSS, and you can apply them to the site. > > I'm also using Noto Sans fo

Re: Review Request 12228: static resource compression

2013-09-27 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review26435 --- For the next release I'm planning to refactor the stylesheets to use

Re: Review Request 13770: CLOUDSTACK-3363 UI > fix list Events/Alerts screen is in processing state forever with Delete Events/Alerts request having no filter

2013-09-27 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13770/#review26441 --- Ship it! Ship It! - Brian Federle On Aug. 28, 2013, 7:46 a.m., b

Re: Review Request 13770: CLOUDSTACK-3363 UI > fix list Events/Alerts screen is in processing state forever with Delete Events/Alerts request having no filter

2013-09-27 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13770/#review26442 --- Commit 38b94e934c38556ed3974ab16a35d5be33f4edc4 in branch refs/heads

Re: Problem creating VirtualRouter on XenServer 6.2 due to problem with copy_vhd_from_secondarystorage.sh

2013-09-27 Thread Nitin Mehta
ListTemplateCommand() just reads the content present on secondary storage through template.properties. How did you put the new template ? Was template.properties and DB updated accordingly ? On 27/09/13 11:30 AM, "Soheil Eizadi" wrote: >I traced this to a problem with ListTemplateCommand() that

Re: Review Request 12228: static resource compression

2013-09-27 Thread Laszlo Hornyak
> On Sept. 27, 2013, 6:40 p.m., Brian Federle wrote: > > For the next release I'm planning to refactor the stylesheets to use the > > SASS compiler. Will this effect integration of a SASS compiler in any way? > > > > FYI The one I'm considering is https://github.com/Jasig/sass-maven-plugin. >

Re: Libvirt-java 0.5.0 has been released

2013-09-27 Thread Mike Tutkowski
Kind of made progress here. :) Looks like we have another problem. Is it possible another item is missing from the scripts? mtutkowski@ubuntu:~$ sudo apt-get install cloudstack-agent Reading package lists... Done Building dependency tree Reading state information... Done The following NEW packages

Re: Libvirt-java 0.5.0 has been released

2013-09-27 Thread Mike Tutkowski
Looks like listNetworks() is called a couple times in cloudstack-agent-upgrade: if __name__ == '__main__': netlib = networkConfig() bridges = netlib.listNetworks() bridges = filter(isOldStyleBridge, bridges) for br in bridges: enslavedDev = netlib.getEnslavedDev(br, 1) if

Advanced Networking Shared Network Deletion Fail due to VRouter

2013-09-27 Thread Soheil Eizadi
Here is problem I just encoutered on 4.3 Master: 1. Created a Shared Advanced network 2. Created an instance 3. Removed the instance 4. After the GUI showed the Instance was removed I tried to delete the network The network deletion fails, due to VRouter Instance being present: >> WARN [c.c.n.Net

Re: Review Request 14360: Make getName() match the constant names for Network Capability, and Service

2013-09-27 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14360/#review26447 --- Darren, changes in Network.java are going to break API compatibility

Re: Review Request 14285: Add multi-row action support to UI list views (CLOUDSTACK-1579)

2013-09-27 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14285/ --- (Updated Sept. 27, 2013, 8:34 p.m.) Review request for cloudstack. Changes --

Re: Review Request 14058: Including tests for VPC VM Lifecycle on Tagged hosts

2013-09-27 Thread Sheng Yang
> On Sept. 26, 2013, 1:30 a.m., Sheng Yang wrote: > > Please test it before submit. > > Ashutosh Kelkar wrote: > The tests have been tested and run correctly. There is a manual step > needed in setup for these tests - Adding host tags to the hosts which is why > the tests are skipped when

Cloudstack doesn't seem to recognize we added 100GB of memory to the nodes.

2013-09-27 Thread Jeronimo Garcia
Hi List. We've done some memory upgrades to our hosts but even tho we can see the new memory numbers and all that .. the cluster where this hosts belong to still showing full (in the dashboard). We've forced recalculation and still no luck. Do you guys happen to know what to do with this? Thank

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
Ok, I've got our plugin working against 4.2. Tested start vm, stop vm, migrate vm, attach volume, detach volume. Other functions that we already had in our StorageAdaptor implementation, such as copying templates to primary storage, just worked without any modification from our 4.1 version. I'll

RE: Problem creating VirtualRouter on XenServer 6.2 due to problem with copy_vhd_from_secondarystorage.sh

2013-09-27 Thread Soheil Eizadi
I followed the steps here: https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+to+build+CloudStack Under Install the SystemVM Template, I put the steps there as there was not enough detail for developers. I think it is OK but you can review. As I said I am not sure of the steps that led

Re: bug? capacity not updated when adding memory to hypervisor

2013-09-27 Thread Jeronimo Garcia
Has this bug been raised? I have this issue as well .. we are on 4.0.2 and 4.1.1 On Wed, Sep 25, 2013 at 4:43 PM, Prachi Damle wrote: > Yes, the main bug here is not updating the Total Capacity - this is > avoiding the total getting updated on host reconnects. > > Bug with the Used and Reserved

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
Sure, sounds good - let me know when it's up on Review Board and I can take a look. I made most of the changes you and I talked about: https://github.com/mike-tutkowski/incubator-cloudstack/commit/eb9b2edfc9062f9ca7961fecd5379b180ca3aed1 I have a new idea, though, that I think will simplify this

Re: Scalable Backup and Recovery

2013-09-27 Thread Chiradeep Vittal
Ah I see. You mean a "scalable user experience". The actual scalability of the snapshot process itself is limited by available disk and network bandwidth. Also certain hypervisors have various quirks which stand in the way of an efficient solution. On 9/27/13 10:27 AM, "SuichII, Christopher" wro

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
URL has changed to this: https://github.com/mike-tutkowski/incubator-cloudstack/commit/636cf78bcd9d32ae9f20c0ccd631fcf41b829d43 I've just been squashing my commits and sending them to GitHub as one commit. It should make it easier to see what's changed when comparing against what's fairly recent

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
On Fri, Sep 27, 2013 at 4:22 PM, Mike Tutkowski wrote: > Sure, sounds good - let me know when it's up on Review Board and I can take > a look. > > I made most of the changes you and I talked about: > > https://github.com/mike-tutkowski/incubator-cloudstack/commit/eb9b2edfc9062f9ca7961fecd5379b180c

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
Yeah, I can verify that my createPhysicalDisk method is not being called at all in 4.2. The create happens on the mgmt server in your driver (createAsync), then you pass the lun info on to KVM in attachVolume (or StartCommand for starting VMs), and it assumes the LUN will be there on the SAN when i

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
Well, from what I saw with XenServer and VMware, that hypervisor logic's attachVolume command also assumed a VDI/VMDK was created in advance. I had to put logic in those attachVolume methods to create the SR/VDI or datastore/VMDK. However, thinking back on it, it might have made more sense for th

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
createAsync is just for creating the SAN (or whatever storage) volume. deleteAsync is the reverse. Technically even the default plug-in should not call into the hypervisor layer. The storage layer should probably not be aware of the hypervisor layer. On Fri, Sep 27, 2013 at 5:14 PM, Mike Tutkow

Re: bug? capacity not updated when adding memory to hypervisor

2013-09-27 Thread Timothy Ehlers
I created a bug here if a developer could describe the code problem. I have merely added some mysql commands showing how to fix it. https://issues.apache.org/jira/browse/CLOUDSTACK-4755 On Fri, Sep 27, 2013 at 4:54 PM, Jeronimo Garcia wrote: > Has this bug been raised? I have this issue as wel

Re: Cloudstack doesn't seem to recognize we added 100GB of memory to the nodes.

2013-09-27 Thread Nitin Mehta
I guess you are referring to the same issue as discussed in the thread http://www.mail-archive.com/dev@cloudstack.apache.org/msg15197.html There is a bug created https://issues.apache.org/jira/browse/CLOUDSTACK-4755 For this with the workaround. On 27/09/13 1:48 PM, "Jeronimo Garcia" wrote: >H

Re: Review Request 14285: Add multi-row action support to UI list views (CLOUDSTACK-1579)

2013-09-27 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14285/#review26456 --- Ship it! Ship It! - Brian Federle On Sept. 27, 2013, 8:34 p.m.,

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
On Fri, Sep 27, 2013 at 5:16 PM, Mike Tutkowski wrote: > createAsync is just for creating the SAN (or whatever storage) volume. > deleteAsync is the reverse. Exactly. It used to be that the hypervisor created the disk/lun/file volume via createPhysicalDisk. Now it's done on the SAN if the plugin

Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-09-27 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14381/ --- Review request for cloudstack and Mike Wang. Repository: cloudstack-git Descr

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
https://reviews.apache.org/r/14381/ On Fri, Sep 27, 2013 at 6:03 PM, Marcus Sorensen wrote: > On Fri, Sep 27, 2013 at 5:16 PM, Mike Tutkowski > wrote: >> createAsync is just for creating the SAN (or whatever storage) volume. >> deleteAsync is the reverse. > > Exactly. It used to be that the hype

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
On Fri, Sep 27, 2013 at 6:03 PM, Marcus Sorensen wrote: > On Fri, Sep 27, 2013 at 5:16 PM, Mike Tutkowski > wrote: >> createAsync is just for creating the SAN (or whatever storage) volume. >> deleteAsync is the reverse. > > Exactly. It used to be that the hypervisor created the disk/lun/file > vo

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
Yeah, we should bring John Burwell into this conversation. He had a strong opposition to storage plug-ins talking to hypervisors. Perhaps he was just referring to shared storage, though (not local disks). Anyways, prior to 4.2 for XenServer and VMware, you had to preallocate your SR/datastore (the

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
The plugin itself doesn't talk to the hypervisor. On Sep 27, 2013 7:28 PM, "Mike Tutkowski" wrote: > Yeah, we should bring John Burwell into this conversation. He had a strong > opposition to storage plug-ins talking to hypervisors. Perhaps he was just > referring to shared storage, though (not l

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
Well, the default KVM one does, but only because the hypervisor is playing the part or storage API. My plugin certainly doesn't have to talk to the hypervisor, and I don't think I've instructed you to. On Sep 27, 2013 7:29 PM, "Marcus Sorensen" wrote: > The plugin itself doesn't talk to the hyper

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
Yeah, OK. That's kind of what I'm talking about. With KVM we have storage adaptors, so they can just pass the VM info to the hypervisor and it can select the right way to handle the storage on the fly, which is a bit easier, even though ultimately you still have to write the hypervisor side code. O

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
Let's see...to answer your question here: With Xen, as with any hypervisor, the storage framework invokes createAsync for you to create your volume just before it is to be attached to a VM for the first time. A call is not necessarily made from the storage plug-in to register it as an SR, however

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
I think we should get John Burwell an Edison Su involved in these discussions. John was a proponent for not having storage plug-ins talk to hypervisors and Edison felt it was fine. Until we resolve that design issue, we might just be talking in circles here. :) On Fri, Sep 27, 2013 at 7:29 PM,

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
I actually think its irrelevant to this discussion, since its really only a contention point for the default KVM plugin. I did post this with the intention of getting feedback from anyone who cares about the KVM side and how it handles things. What you describe with Xen sounds identical to what we

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
On a side note, I'm actually a proponent of the storage plugin only talking to its storage api. Best to be hypervisor agnostic. Its just in the case of the default storage plugin the storage API is the cloudstack agent. But enough about that, we can spin that off into another thread if we need to.

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-09-27 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14381/ --- (Updated Sept. 28, 2013, 4:51 a.m.) Review request for cloudstack and Mike Tutk

Re: [jira] [Created] (CLOUDSTACK-4753) Console proxy disk space fills up

2013-09-27 Thread Wei ZHOU
1.In 4.1,cloud service run before cloud-early-config,that could be an issue. 2.In 4.2,new template,the issue does not exist 2013/9/27, John Beredimas (JIRA) : > John Beredimas created CLOUDSTACK-4753: > -- > > Summary: Console proxy disk space f

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
CHAP credentials are per SolidFire/CloudStack account. A volume on the SolidFire SAN belongs to an account. I have a one-to-one mapping between a CloudStack account and a SolidFire account. As far as Volume Access Groups (VAGs) go, I'm not sure what kind of limits they have. I'd hate to have, say

Re: add connect method on KVM storage code

2013-09-27 Thread Marcus Sorensen
Yeah, maybe we just add a "Map connectDetails = cmd.getConnectDetails()" into attachVolume, and pass that along to connectPhysicalDisk. Then just set the details where you're currently setting the chap info prior to the attach command. On Sat, Sep 28, 2013 at 12:25 AM, Mike Tutkowski wrote: > CHA

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
If grantAccess/revokeAccess worked in the storage plug-in, though, I could create a VAG per KVM host and give it access to whatever volumes it needs access to. When we revoke access, I could remove the IQN of the volume from the appropriate VAG. If the VAG is empty, I could delete it. If you had,

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
That works. Right now the cmd takes in CHAP info in the form of four strings (initiator username, initiator password, target username, and target password). I like your idea of a generic list better, though. We could remove the four strings from the cmd and I could use the map in not just KVM la

Re: add connect method on KVM storage code

2013-09-27 Thread Mike Tutkowski
> I'd hate to have, say, 10,000 KVM hosts all stuck in the same VAG...each > host having access (technically) to, say, 20,000 volumes. Plus this modeling of VAGs does nothing to fence against rouge compute nodes. If we did use VAGs, I'd like to use them in a way to protect against rouge nodes. CH

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-09-27 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14381/ --- (Updated Sept. 28, 2013, 6:43 a.m.) Review request for cloudstack, edison su an