Bug 1622031

Tested today with 7.5 and passed. This concrete scenario was added in 7.6 but there is older one similar to this that's testing snapshot stacking, which is passing with 7.5 but fails with locking error in 7.6. This indicates a regression.


7.5 pass:
https://beaker.cluster-qe.lab.eng.brq.redhat.com/bkr/jobs/86272

SCENARIO - [stacked_thin_snaps]
Stack snapshots on top of existing snapshots
Setting up base level origin/snapshot
Making pool volume
lvcreate --activate ey --thinpool POOL -L 1G  --zero n --poolmetadatasize 4M snapper_thinp

Sanity checking pool device (POOL) metadata
thin_check /dev/mapper/snapper_thinp-meta_swap.885
examining superblock
examining devices tree
examining mapping tree
checking space map counts


Making origin volume
lvcreate --activate ey --virtualsize 1G -T snapper_thinp/POOL -n origin
lvcreate --activate ey -V 1G -T snapper_thinp/POOL -n other1
  WARNING: Sum of all thin volume sizes (2.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate --activate ey --virtualsize 1G -T snapper_thinp/POOL -n other2
  WARNING: Sum of all thin volume sizes (3.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate --activate ey -V 1G -T snapper_thinp/POOL -n other3
  WARNING: Sum of all thin volume sizes (4.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate --activate ey --virtualsize 1G -T snapper_thinp/POOL -n other4
  WARNING: Sum of all thin volume sizes (5.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate --activate ey --virtualsize 1G -T snapper_thinp/POOL -n other5
  WARNING: Sum of all thin volume sizes (6.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate --activate ey -y -k n -s /dev/snapper_thinp/origin -n snap_level1
Creating stacked level PV/VG
Creating stacked level origin/snapshot
lvcreate --activate ey -L 100M snapper_thinp_stack -n origin
lvcreate --activate ey -s /dev/snapper_thinp_stack/origin -n snap_level2 -L 50M
Removing stacked level origin/snapshot
Removing stacked level VG/PV
Removing base level VG/PV
Removing snap volume snapper_thinp/snap_level1
lvremove -f /dev/snapper_thinp/snap_level1
Removing thin origin and other virtual thin volumes
Removing pool snapper_thinp/POOL


lvm2-2.02.177-4.el7.x86_64



=======================

7.6 fail:
https://beaker.cluster-qe.lab.eng.brq.redhat.com/bkr/jobs/86275


SCENARIO - [stacked_thin_snaps]
Stack snapshots on top of existing snapshots
Setting up base level origin/snapshot
Making pool volume
lvcreate  --thinpool POOL -L 1G --profile thin-performance --zero y --poolmetadatasize 4M snapper_thinp

Sanity checking pool device (POOL) metadata
thin_check /dev/mapper/snapper_thinp-meta_swap.759
examining superblock
examining devices tree
examining mapping tree
checking space map counts

Making origin volume
lvcreate  --virtualsize 1G -T snapper_thinp/POOL -n origin
lvcreate  --virtualsize 1G -T snapper_thinp/POOL -n other1
  WARNING: Sum of all thin volume sizes (2.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate  --virtualsize 1G -T snapper_thinp/POOL -n other2
  WARNING: Sum of all thin volume sizes (3.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate  -V 1G -T snapper_thinp/POOL -n other3
  WARNING: Sum of all thin volume sizes (4.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate  --virtualsize 1G -T snapper_thinp/POOL -n other4
  WARNING: Sum of all thin volume sizes (5.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate  -V 1G -T snapper_thinp/POOL -n other5
  WARNING: Sum of all thin volume sizes (6.00 GiB) exceeds the size of thin pool snapper_thinp/POOL (1.00 GiB).
lvcreate  -y -k n -s /dev/snapper_thinp/origin -n snap_level1
Creating stacked level PV/VG
pvcreate -y /dev/snapper_thinp/snap_level1
vgcreate  -f snapper_thinp_stack /dev/snapper_thinp/snap_level1
Creating stacked level origin/snapshot
lvcreate  -L 100M snapper_thinp_stack -n origin
  Error locking on node 2: Reading VG not found for LVID gz7Kfku214KFWCOImjOqvgyvUwS90lN7Pw4XSkB17OjBBho7yvfGd8w4m7N3Gi1b
  Failed to activate new LV.
couldn't create stacked origin


lvm2-2.02.180-5.el7.x86_64

Created attachment 1478946 [details]
lvcreate_vvvv

Created attachment 1478947 [details]
vgcfgbackup

Created attachment 1478950 [details]
lvcreate_vvvv_RHEL75

Created attachment 1478957 [details]
do not use

removing incorrect log

(In reply to Roman Bednář from comment #4)
> Created attachment 1478946 [details]
> lvcreate_vvvv

Yep - this seems to work as expected.

ThinLV is made active/present only on 1 node in cluster - while lvcreate on top of this thinLV requests cluster-wide activation, but since other cluster nodes do NOT see this thinLV as PV (not available to them) - it fails on error where remote clvmd cannot find this PV/VG present.

Correct usage would be  'lvcreate -aey' for this origin creation.

So far it more or less look like a bugfix.

Created attachment 1478962 [details]
do not use

Created attachment 1478963 [details]
lvcreate_vvvv

adding correct -vvvv logs

(In reply to Roman Bednář from comment #10)
> Created attachment 1478963 [details]
> lvcreate_vvvv
> 
> adding correct -vvvv logs

So here both logs for 7.5 & 7.6 seems to match - and complain about locking error.

Closing. This is a test issue. We're missing the needed exclusive activation flag in 7.6.