----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10227/#review18619 -----------------------------------------------------------
Commit 3541aa86e34605e5ac4a99df84c47e6f6e7c71ee in branch refs/heads/4.0 from Chip Childers <chip.child...@gmail.com> [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3541aa8 ] CLOUDSTACK-1846, CLOUDSTACK-1845 - KVM Storage, sometimes KVMHA will remount deleted NFS pools, causing failures when defining new storage pools. Sometimes a storage pool has never been used on a host, and getStoragePool fails when copying templates or in storage migration. deleteStoragePool(pool) often fails silently, leaving no pool defined in libvirt, but a mountpoint left behind. This patch handles some of these exceptions and brings forward any issues via logging. Signed-off-by: Chip Childers <chip.child...@gmail.com> - ASF Subversion and Git Services On April 1, 2013, 8:29 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10227/ > ----------------------------------------------------------- > > (Updated April 1, 2013, 8:29 p.m.) > > > Review request for cloudstack and Joe Brockmeier. > > > Description > ------- > > CLOUDSTACK-1846, CLOUDSTACK-1845 - KVM Storage, sometimes KVMHA will remount > deleted NFS pools, causing failures when defining new storage pools. > Sometimes > a storage pool has never been used on a host, and getStoragePool fails > when > copying templates or in storage migration. deleteStoragePool(pool) often > fails > silently, leaving no pool defined in libvirt, but a mountpoint left > behind. > This patch handles some of these exceptions and brings forward any issues > via > logging. > > > This addresses bugs CLOUDSTACK-1845 and CLOUDSTACK-1846. > > > Diffs > ----- > > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java > 131c37e > > Diff: https://reviews.apache.org/r/10227/diff/ > > > Testing > ------- > > Deployed new 4.0 zone in devcloud-kvm, tested volume downloads, template > creation, storage migration > > > Thanks, > > Marcus Sorensen > >
