Re: pluggable storage implementation

2012-12-11 Thread Marcus Sorensen
ing userInfo, >>>>> StoragePoolType type) { >>>>> >>>>> >>>>> ** ** >>>>> >>>>> may be implemented like this: >>>>> >>>>> storageadaptor adaptor = getstorageadaptor(type);*

Re: pluggable storage implementation

2012-12-11 Thread Marcus Sorensen
t;>> ** ** >>>> >>>> may be implemented like this: >>>> >>>> storageadaptor adaptor = getstorageadaptor(type); >>>> >>>> adaptor.createPool(…) >>>> >>>> ** ** >>>> >>>> getStorageA

Re: pluggable storage implementation

2012-12-11 Thread Marcus Sorensen
t; >>> return this._storageAdaptor.getStoragePool(uuid); >>> >>> } >>> >>> ** ** >>> >>> // pluggable getStoragePool >>> >>> public KVMStoragePool getStoragePool(String storageAdaptorType, >>> S

Re: pluggable storage implementation

2012-12-11 Thread Marcus Sorensen
aptor getStorageAdaptor(StoragePoolType type) { >> >> storageadaptor adaptor = storageMapper.get(type.toString()); >> >>if (adaptor == null) { >> >> adaptor = storageMapper.get(“libvirt”); >> >>} >> &g

Re: pluggable storage implementation

2012-12-10 Thread Marcus Sorensen
n adaptor; > > } > > *From:* Marcus Sorensen [mailto:shadow...@gmail.com] > *Sent:* Monday, December 10, 2012 4:25 PM > > *To:* Edison Su > *Cc:* cloudstack-dev@incubator.apache.org > *Subject:* Re: pluggable storage implementation > > ** ** > >

RE: pluggable storage implementation

2012-12-10 Thread Edison Su
eMapper.get("libvirt"); } return adaptor; } From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Monday, December 10, 2012 4:25 PM To: Edison Su Cc: cloudstack-dev@incubator.apache.org Subject: Re: pluggable storage implementation I'm not quite sure what you mean on

Re: pluggable storage implementation

2012-12-10 Thread Marcus Sorensen
} On Mon, Dec 10, 2012 at 2:05 PM, Edison Su wrote: > ** ** > > ** ** > > *From:* Marcus Sorensen [mailto:shadow...@gmail.com] > *Sent:* Monday, December 10, 2012 3:37 PM > > *To:* Edison Su > *Cc:* cloudstack-dev@incubator.apache.org > *Subject:* Re: pluggable storage im

RE: pluggable storage implementation

2012-12-10 Thread Edison Su
From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Monday, December 10, 2012 3:37 PM To: Edison Su Cc: cloudstack-dev@incubator.apache.org Subject: Re: pluggable storage implementation Ok, that makes sense. So this will be a part of the storage refactor? If you're not already worki

Re: pluggable storage implementation

2012-12-10 Thread Marcus Sorensen
t; > ** ** > > ** ** > > ** ** > > *From:* Marcus Sorensen [mailto:shadow...@gmail.com] > *Sent:* Monday, December 10, 2012 3:06 PM > *To:* Edison Su > *Cc:* cloudstack-dev@incubator.apache.org > *Subject:* Re: pluggable storage implementation > > ** ** >

RE: pluggable storage implementation

2012-12-10 Thread Edison Su
PM To: Edison Su Cc: cloudstack-dev@incubator.apache.org Subject: Re: pluggable storage implementation Yeah, but I'm not seeing an easy/pluggable way to implement a storage adaptor, where LibvirtComputingResource relies exclusively on KVMStoragePoolManager, which is implementing LibvirtS

Re: pluggable storage implementation

2012-12-10 Thread Marcus Sorensen
ev@incubator.apache.org > *Subject:* pluggable storage implementation > > ** ** > > Just curious, hadn't thought about this before but it seems that at least > on KVM (probably similar in Xen and VMware too?), there are two separate > issues with storage in the existing code. F

RE: pluggable storage implementation

2012-12-10 Thread Edison Su
From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Monday, December 10, 2012 2:06 PM To: Edison Su Cc: cloudstack-dev@incubator.apache.org Subject: pluggable storage implementation Just curious, hadn't thought about this before but it seems that at least on KVM (probably similar i

pluggable storage implementation

2012-12-10 Thread Marcus Sorensen
Just curious, hadn't thought about this before but it seems that at least on KVM (probably similar in Xen and VMware too?), there are two separate issues with storage in the existing code. First, adding a new storage type is a matter of adding in a new 'else if' or something in a bunch of different