Skip to content

Commit

Permalink
Fix configuration of LVM global_filter
Browse files Browse the repository at this point in the history
As far as I could tell, the global_filter config added in change
I5d5c48e188cbb9b4208096736807f082bce524e8 wasn't actually making it
into the lvm.conf. Given the volume (or rather LVM volume) related
issues we've been seeing in the gate recently, we can give this a try
to see if the global_filter setting has any positive effect.

This also adds the contents of /etc/lvm/* to the logs collected by the
jobs, so that we can see the LVM config.

Change-Id: I2b39acd352669231d16b5cb2e151f290648355c0
  • Loading branch information
melwitt committed Aug 24, 2023
1 parent 640c318 commit ef53db7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions .zuul.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -430,6 +430,7 @@
/var/log/mysql: logs
/var/log/libvirt: logs
/etc/libvirt: logs
/etc/lvm: logs
/etc/sudoers: logs
/etc/sudoers.d: logs
'{{ stage_dir }}/iptables.txt': logs
Expand Down
2 changes: 1 addition & 1 deletion lib/lvm
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ function set_lvm_filter {
filter_string=$filter_string$filter_suffix

clean_lvm_filter
sudo sed -i "/# global_filter = \[*\]/a\ $global_filter$filter_string" /etc/lvm/lvm.conf
sudo sed -i "/# global_filter = \[.*\]/a\ $filter_string" /etc/lvm/lvm.conf
echo_summary "set lvm.conf device global_filter to: $filter_string"
}

Expand Down

0 comments on commit ef53db7

Please sign in to comment.