----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12348/ -----------------------------------------------------------
(Updated July 9, 2013, 10:15 p.m.) Review request for cloudstack, Chip Childers, Kelven Yang, rajeshbabu chintaguntla, and Sateesh Chodapuneedi. Changes ------- Observed the same issue on master branch, so uploading the diff for master. Testing showed up other issues in volume migration, but they are unrelated to the issue this patch addresses. The eligible storage pools are listed as expected out of this fix, so requesting that this fix be applied on top of master. Bugs: CLOUDSTACK-3264 Repository: cloudstack-git Description ------- When listing zone wide primary storages for volume migration, a null pointer exception is encountered because the listing of the eligible storage pools missed taking into account the absence of cluster id for zone wide primary pools. Fixing the same and putting in appropriate checks for other storage type allocators. Note: This fix is for the master-6-17-stable branch. It is to be seen whether this fix is required for master and if so, needs to be ported/tested on the same. Diffs (updated) ----- engine/storage/src/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java e16703e engine/storage/src/org/apache/cloudstack/storage/allocator/ClusterScopeStoragePoolAllocator.java 0933adc engine/storage/src/org/apache/cloudstack/storage/allocator/LocalStoragePoolAllocator.java ef9e84e plugins/storage-allocators/random/src/org/apache/cloudstack/storage/allocator/RandomStoragePoolAllocator.java 76ce663 server/src/com/cloud/server/ManagementServerImpl.java da9d6a2 Diff: https://reviews.apache.org/r/12348/diff/ Testing ------- Created multiple zone wide primary storages and confirmed that they show up as eligible destinations when attempting to migrate a volume currently resident on a zone wide primary storage pool. Created mixed zone-wide and cluster-wide primary storages to ensure nothing broke there. Observed that volumes on a zone wide primary can only be moved to other zone wide primary pools and similarly with cluster wide pools. Created new guest VM to confirm that existing primary code paths work correctly with the changes in place. Thanks, Venkata Siva Vijayendra Bhamidipati