diff --git a/lib/cinder_backends/lvm b/lib/cinder_backends/lvm index d369c0c840..35ad209db7 100644 --- a/lib/cinder_backends/lvm +++ b/lib/cinder_backends/lvm @@ -39,6 +39,7 @@ function cleanup_cinder_backend_lvm { # Campsite rule: leave behind a volume group at least as clean as we found it clean_lvm_volume_group $VOLUME_GROUP_NAME-$be_name + clean_lvm_filter } # configure_cinder_backend_lvm - Set config files, create data dirs, etc diff --git a/lib/lvm b/lib/lvm index 54976a36cd..1fe2683e65 100644 --- a/lib/lvm +++ b/lib/lvm @@ -145,6 +145,13 @@ function init_default_lvm_volume_group { fi } +# clean_lvm_filter() Remove the filter rule set in set_lvm_filter() +# +# Usage: clean_lvm_filter() +function clean_lvm_filter { + sudo sed -i "s/^.*# from devstack$//" /etc/lvm/lvm.conf +} + # set_lvm_filter() Gather all devices configured for LVM and # use them to build a global device filter # set_lvm_filter() Create a device filter @@ -154,7 +161,7 @@ function init_default_lvm_volume_group { # # Usage: set_lvm_filter() function set_lvm_filter { - local filter_suffix='"r|.*|" ]' + local filter_suffix='"r|.*|" ] # from devstack' local filter_string="global_filter = [ " local pv local vg @@ -167,6 +174,7 @@ function set_lvm_filter { done filter_string=$filter_string$filter_suffix + clean_lvm_filter sudo sed -i "/# global_filter = \[*\]/a\ $global_filter$filter_string" /etc/lvm/lvm.conf echo_summary "set lvm.conf device global_filter to: $filter_string" } diff --git a/unstack.sh b/unstack.sh index 30981fd3c6..ed7e6175ca 100755 --- a/unstack.sh +++ b/unstack.sh @@ -192,3 +192,4 @@ fi # BUG: maybe it doesn't exist? We should isolate this further down. clean_lvm_volume_group $DEFAULT_VOLUME_GROUP_NAME || /bin/true +clean_lvm_filter