diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-07-14 12:53:47 +0200 |
|---|---|---|
| committer | Marian Csontos <mcsontos@redhat.com> | 2021-07-22 13:53:47 +0200 |
| commit | f29ce593ec5c19296f50a1dbd1adbced1a1a53a7 (patch) | |
| tree | 2e90f11d974d2b0e37a10aa0ba6a39c669697c48 | |
| parent | 154592db7e3dc3a7ffb4001bb9e342d10fce0a5b (diff) | |
| download | lvm2-f29ce593ec5c19296f50a1dbd1adbced1a1a53a7.tar.gz | |
thin: fix component detection of external origin
When check active componet of thinLV with external origin,
we need to check if the external origin isn't already active.
For this however we need to use layered check for -real device.
(cherry picked from commit d38fdb25e4debc035f1e4585432c2627460d5e50)
Conflicts:
WHATS_NEW
| -rw-r--r-- | WHATS_NEW | 1 | ||||
| -rw-r--r-- | lib/activate/activate.c | 5 | ||||
| -rw-r--r-- | test/shell/lvconvert-cache-thin.sh | 22 |
3 files changed, 27 insertions, 1 deletions
@@ -1,5 +1,6 @@ Version 2.03.13 - =============================== + Fix detection of active components of external origin volume. Add vdoimport tool to support conversion of VDO volumes. Support configurable allocation/vdo_pool_header_size. Fix handling of lvconvert --type vdo-pool --virtualsize. diff --git a/lib/activate/activate.c b/lib/activate/activate.c index 6bda7385b..94fc9441d 100644 --- a/lib/activate/activate.c +++ b/lib/activate/activate.c @@ -2740,7 +2740,10 @@ static int _component_cb(struct logical_volume *lv, void *data) (lv_is_thin_pool(lv) && pool_is_active(lv))) return -1; - if (lv_is_active(lv)) { + /* External origin is activated through thinLV and uses -real suffix. + * Note: for old clustered logic we would need to check for all thins */ + if ((lv_is_external_origin(lv) && lv_info(lv->vg->cmd, lv, 1, NULL, 0, 0)) || + lv_is_active(lv)) { if (!lv_is_component(lv) || lv_is_visible(lv)) return -1; /* skip whole subtree */ diff --git a/test/shell/lvconvert-cache-thin.sh b/test/shell/lvconvert-cache-thin.sh index 7dda6e6ff..9254239ee 100644 --- a/test/shell/lvconvert-cache-thin.sh +++ b/test/shell/lvconvert-cache-thin.sh @@ -67,4 +67,26 @@ fail lvconvert --yes --thinpool $vg/$lv1 --poolmetadata $vg/$lv # Thin-pool CAN use cached data LV lvconvert --yes --thinpool $vg/$lv +lvremove -f $vg + +# Check we can active snapshot of cached external origin (BZ: 1967744) +lvcreate -T -L10M $vg/pool "$dev1" + +lvcreate -L10M -n origin $vg "$dev1" +lvcreate -H -L4M -n CPOOL $vg/origin "$dev2" + +# Use cached origin as external origin +lvconvert -y -T --thinpool $vg/pool --originname extorig origin + +# Check we can easily create snapshot of such LV +lvcreate -y -kn -n snap -s $vg/origin + +# Deactivate everything and do a component activation of _cmeta volume +lvchange -an $vg +lvchange -ay -y $vg/CPOOL_cpool_cmeta + +# Now this must fail since component volume is active +not lvcreate -y -kn -n snap2 -s $vg/origin |& tee err +grep "cmeta is active" err + vgremove -f $vg |