----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14904/ -----------------------------------------------------------
Review request for cloudstack. Summary (updated) ----------------- Support managed storage for root disks Bugs: CLOUDSTACK-6170 https://issues.apache.org/jira/browse/CLOUDSTACK-6170 Repository: cloudstack-git Description (updated) ------- Cloud environments have a need for guaranteed storage performance. By this I mean having the ability to specify a minimum and maximum number of IOPS on a volume-by-volume basis. I added support for this for XenServer and ESX in 4.2 for data disks. I added support for this for KVM in 4.3 for data disks. It is my intent to add support for this for XenServer and ESX in 4.4 for root disks (with subsequent support for root disks on KVM expected in 4.5). This will require minor changes in the SolidFire (storage) plug-in. The main changes are expected to occur in CloudStack logic that controls hypervisors and additions to the way root-volume orchestration happens. Diffs ----- api/src/com/cloud/agent/api/to/DiskTO.java 556ccd4 core/src/com/cloud/agent/api/MigrateCommand.java 5042b8c core/src/org/apache/cloudstack/storage/command/AttachCommand.java 44bce91 engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 8bf419f engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java 0817393 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java 371c4e0 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePool.java a1721e1 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java 945243a plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java 04bdd76 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java e04297e plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStoragePool.java c0e1000 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/StorageAdaptor.java 44e0691 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/iScsiAdmStorageAdaptor.java PRE-CREATION plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/iScsiAdmStoragePool.java PRE-CREATION plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java 34bfe18 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServerStorageProcessor.java 2d4c86e plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/lifecycle/SolidFirePrimaryDataStoreLifeCycle.java f1ac3b3 plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/provider/SolidFireHostListener.java PRE-CREATION plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/provider/SolidfirePrimaryDataStoreProvider.java 9c784ba server/src/com/cloud/storage/VolumeApiServiceImpl.java 00df333 server/src/com/cloud/storage/listener/StoragePoolMonitor.java 2be0c65 Diff: https://reviews.apache.org/r/14904/diff/ Testing ------- Thanks, Mike Tutkowski