----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10766/ -----------------------------------------------------------
(Updated April 25, 2013, 12:39 a.m.) Review request for cloudstack and Chip Childers. Changes ------- does this look better? if storage is NFS and we're fetching pool by URI, call LibvirtStorageAdaptor's createStoragePool instead of KVMStoragePoolManager's createStoragePool. Testing now... will update status when I can. Description ------- the KVM HA runner uses any NFS secondary storage resource available to a host to store it's HA data. This causes template deletes to fail because it cannot delete KVMHA, which is a directory that is not empty. So if KVMHA directory is found, delete it's contents before trying to delete it. This addresses bug CLOUDSTACK-2173. Diffs ----- core/src/com/cloud/storage/resource/NfsSecondaryStorageResource.java e65cbe1 Diff: https://reviews.apache.org/r/10766/diff/ Testing ------- Had a bunch of templates that the scavenger was failing to clean up. Applied this patch, installed the code, and the templates all cleaned up successfully. Thanks, Marcus Sorensen