That's a good point about the "managed" property being storable in the storage_pool_details table. We don't need another column for it in the storage_pool table. In the current url field is where this kind of information can be passed along and it can be stored in the storage_pool_details table, if the plug-in wants to do so.
Let's decide on how the hypervisor code should operate, though. Should we pass in info to the "attach" command to have it create an SR if need be? This is sort of how I did it in the patch code I submitted (but I used the StoragePoolType.Dynamic field, which we don't have to use).
