diff options
| author | David Teigland <teigland@redhat.com> | 2023-05-17 14:15:25 -0500 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2023-05-17 14:15:25 -0500 |
| commit | 24e4b6df1182d0d41763176c175e98e5fa6153ab (patch) | |
| tree | 30d1484cc4bc4f24c7a0f4e557cb02428fbee843 /lib/metadata/lv_manip.c | |
| parent | 3a757047560d75a28d7e4c7d9a5253a72d786544 (diff) | |
| download | lvm2-24e4b6df1182d0d41763176c175e98e5fa6153ab.tar.gz | |
The recent fix 05c2b10c5d0a9 ensures that raid LV images are not
using the same devices. This was happening in the lvextend commands
used by this test, so fix the test to use more devices to ensue
redundancy.
Diffstat (limited to 'lib/metadata/lv_manip.c')
| -rw-r--r-- | lib/metadata/lv_manip.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 9a9e547be..fa3661739 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -4471,8 +4471,11 @@ static int _lv_raid_redundant(struct logical_volume *lv, for (s = 0; s < seg->area_count; s++) { struct logical_volume *slv = meta ? seg_metalv(seg, s) : seg_lv(seg, s); - if (slv && lv_is_on_pv(slv, pvl->pv) && nlvs++) + if (slv && lv_is_on_pv(slv, pvl->pv) && nlvs++) { + log_error("LV %s using PV %s is not redundant.", + display_lvname(slv), dev_name(pvl->pv->dev)); return 0; + } } } |