cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcus Sorensen" <>
Subject Review Request: Allow for custom storage adaptors based on pool type in KVM
Date Tue, 11 Dec 2012 22:31:25 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for cloudstack and edison su.


This is a refactor of a few things to allow pluggable storage adaptors for KVM. Instead of
using LibvirtStorageAdaptor for everything, you can create your own storage adaptor and use
it. We select storage adaptor based on storage pool type, thus we needed to adjust LibvirtComputingResource
to pass pool type to everything in KVMStoragePoolManager. This in turn required that we pass
the info necessary to LibvirtComputingResource as well, so a few agent Commands were modified.

Note this patch in and of itself shouldn't change any existing behavior, just allow for new
storage adaptors to be selected based on storage pool type.


  api/src/com/cloud/agent/api/ 9476d7d 
  api/src/com/cloud/agent/api/ bf6f260 
  api/src/com/cloud/agent/api/ b589ed5 
  api/src/com/cloud/agent/api/ f006329 
  api/src/com/cloud/agent/api/ 892db37 
  api/src/com/cloud/agent/api/ e451768 
  api/src/com/cloud/agent/api/ 0586f8c 
  api/src/com/cloud/agent/api/storage/ b563177 
  server/src/com/cloud/storage/ e252633 
  server/src/com/cloud/storage/snapshot/ 32e37e6 
  server/src/com/cloud/template/ 1e87de2 
  server/src/com/cloud/vm/ 02d630f 



tested the following on a KVM cloudstack install:

add primary storage
delete primary storage
add vm to CLVM primary storage
add vm to NFS primary storage
create data disk
create snapshot
create template
attach datadisk to VM
detach datadisk from VM
delete datadisk
delete template
delete snapshot
stop vm
delete vm
register iso
attach iso
detach iso


Marcus Sorensen

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message