diff --git a/lib/cinder b/lib/cinder index 76314c1d1f..b029fa0db4 100644 --- a/lib/cinder +++ b/lib/cinder @@ -447,10 +447,6 @@ function init_cinder { be_type=${be%%:*} be_name=${be##*:} if type init_cinder_backend_${be_type} >/dev/null 2>&1; then - # Always init the default volume group for lvm. - if [[ "$be_type" == "lvm" ]]; then - init_default_lvm_volume_group - fi init_cinder_backend_${be_type} ${be_name} fi done diff --git a/lib/nova b/lib/nova index 1420183a19..5a12da6731 100644 --- a/lib/nova +++ b/lib/nova @@ -233,6 +233,10 @@ function cleanup_nova { stop_process "n-api-meta" remove_uwsgi_config "$NOVA_UWSGI_CONF" "$NOVA_UWSGI" remove_uwsgi_config "$NOVA_METADATA_UWSGI_CONF" "$NOVA_METADATA_UWSGI" + + if [[ "$NOVA_BACKEND" == "LVM" ]]; then + clean_lvm_volume_group $DEFAULT_VOLUME_GROUP_NAME + fi } # configure_nova() - Set config files, create data dirs, etc diff --git a/unstack.sh b/unstack.sh index d9dca7c107..4b57b6e344 100755 --- a/unstack.sh +++ b/unstack.sh @@ -176,10 +176,6 @@ fi # enabled backends. So if Cinder is enabled, and installed successfully we are # sure lvm2 (lvremove, /etc/lvm/lvm.conf, etc.) is here. if is_service_enabled cinder && is_package_installed lvm2; then - # Using /bin/true here indicates a BUG - maybe the - # DEFAULT_VOLUME_GROUP_NAME doesn't exist? We should - # isolate this further down in lib/cinder cleanup. - clean_lvm_volume_group $DEFAULT_VOLUME_GROUP_NAME || /bin/true clean_lvm_filter fi