----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22807/#review46385 -----------------------------------------------------------
Commit bbcffbbab09905872a1b9c405a67734e457182fd in cloudstack's branch refs/heads/master from Anshul Gangwar [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=bbcffbb ] CLOUDSTACK-6965: fixed the NullPointerException introduced by fix for cloudstack 6935 in AbstractStoragePoolAllocator#filter method for Zone Wide storage - ASF Subversion and Git Services On June 23, 2014, 5:30 a.m., Anshul Gangwar wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > > engine/storage/src/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java > 2d958a6 > > Diff: https://reviews.apache.org/r/22807/diff/ > > > Testing > ------- > > > Thanks, > > Anshul Gangwar > >