----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22807/ -----------------------------------------------------------
(Updated June 23, 2014, 5:30 a.m.) Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi. Changes ------- updated the patch as per Devdeep's suggestion Bugs: CLOUDSTACK-6965 https://issues.apache.org/jira/browse/CLOUDSTACK-6965 Repository: cloudstack-git Description ------- Fix for CLOUDSTACK-6935 has introduced the NullPointerException in code. Since ZoneWideStoragePoolAllocator's filter method has been removed it is using AbstractStoragePoolAllocator's filter method. In the filter method there is check for cluster's hypervisor type and and disk profile hypervisor type. But cluster is null for ZoneWideStoragePool and Hence the exception. Diffs (updated) ----- engine/storage/src/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java 2d958a6 Diff: https://reviews.apache.org/r/22807/diff/ Testing ------- Thanks, Anshul Gangwar