Bug 1639360 - Separate lvm activation from other lvm commands
Summary: Separate lvm activation from other lvm commands
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: vdsm
Version: 4.3.0
Hardware: Unspecified
OS: Unspecified
Target Milestone: ovirt-4.4.0
: ---
Assignee: Vojtech Juranek
QA Contact: Ilan Zuckerman
Depends On: 1411103
TreeView+ depends on / blocked
Reported: 2018-10-15 14:53 UTC by Nir Soffer
Modified: 2020-08-04 13:27 UTC (History)
16 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Previously, mixing the Logical Volume Manager (LVM) activation and deactivation commands with other commands caused possible undefined LVM behavior and warnings in the logs. The current release fixes this issue. It runs the LVM activation and deactivation commands separately from other commands. This produces resulting well-defined LVM behavior and clear errors in case of failure.
Clone Of:
Last Closed: 2020-08-04 13:26:06 UTC
oVirt Team: Storage
Target Upstream Version:
lsvaty: testing_plan_complete-

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2020:3246 0 None None None 2020-08-04 13:27:05 UTC
oVirt gerrit 95161 0 master MERGED storage: Separate lvm commands 2021-02-20 22:00:12 UTC
oVirt gerrit 101378 0 'None' MERGED lvm: Suppress "combining activation" warnings 2021-02-20 22:00:11 UTC
oVirt gerrit 104882 0 master MERGED lvm: allow changeLVTags() to accept tuple of LVs 2021-02-20 22:00:11 UTC
oVirt gerrit 104883 0 master MERGED storage: don't use changelv() if there's equivalent lvm function 2021-02-20 22:00:11 UTC
oVirt gerrit 104884 0 master MERGED storage: activate/deactivate LVs before/after zeroing 2021-02-20 22:00:11 UTC
oVirt gerrit 109249 0 master MERGED lvm: Unsuppress warning for mixing lv tags changes with activation 2021-02-20 22:00:11 UTC

Description Nir Soffer 2018-10-15 14:53:14 UTC
Description of problem:

Vdsm is mixing lvm activation with adding and removing tags in the the same lvm 
command. This show a warning with recent LVM versions.

Here is an example command:

Here is example command we use:

/usr/sbin/lvm lvchange --config ' devices { preferred_names = ["^/dev/mapper/"] ignore_suspended_devices=1 write_cache_state=0 disable_after_error_count=3 filter = [ '\''a|/dev/mapper/3600a098038304437415d4b6a59676c56'\'', '\''r|.*|'\'' ] }  global {  locking_type=1  prioritise_write_locks=1  wait_for_locks=1  use_lvmetad=0 }  backup {  retain_min = 50  retain_days = 0 } ' --autobackup n -a y --deltag IU_4181c731-8b1a-4929-9c30-f3bfa011c129 --addtag IU__remove_me_ZERO_4181c731-8b1a-4929-9c30-f3bfa011c129 c80a5bd0-809e-4c4e-97a8-111611105b3e/885511bd-7164-458f-a1ac-1f8ad4355972

And we get these warning:

  WARNING: Combining activation change with other commands is not advised.

This command did:

- activate the lv
- delete tag IU_4181c731-8b1a-4929-9c30-f3bfa011c129
- add tag IU__remove_me_ZERO_4181c731-8b1a-4929-9c30-f3bfa011c129

According to David Teigland response on linux-lvm:

Mixing activation with other commands can lead to undefined behavior.

Version-Release number of selected component (if applicable):
LVM since Feb 2017.

We should split the internal APIs for activating and deactivating logical volumes
and editing logical volume tags, and issue separate commands.

Comment 1 Sandro Bonazzola 2019-01-28 09:36:43 UTC
This bug has not been marked as blocker for oVirt 4.3.0.
Since we are releasing it tomorrow, January 29th, this bug has been re-targeted to 4.3.1.

Comment 2 Nir Soffer 2019-06-30 22:37:02 UTC
https://gerrit.ovirt.org/c/101378/ does not fix the underlying issue, but
suppress the warnings in vdsm log. The warnings are still available when
enabling DEBUG log level.

Comment 3 Nir Soffer 2019-11-19 14:24:07 UTC
I would raise the priority of this. We are using lvm incorrectly and
we must fix our ways.

Comment 6 Nir Soffer 2020-02-18 19:37:39 UTC
Checking which patches can be backported:

- https://gerrit.ovirt.org/95161 - easy to backport
- https://gerrit.ovirt.org/104882 - big change, not suitable for backport
- https://gerrit.ovirt.org/104883 - easy change but needs to be reimplemented
  without depending on https://gerrit.ovirt.org/104882

If we have a proof that mixing activation and tag changes can cause 
corruption we can fix this in 4.3.

Comment 8 Nir Soffer 2020-03-04 15:34:39 UTC
We don't have any data supporting the claim that this is related to data corruption.

Therefore I don't any reason to backport the fix.

Tal, this bug should be 4.4 bug, and move to ON_QA.

Comment 15 Ilan Zuckerman 2020-03-23 09:11:56 UTC
To verify this BZ, i used storage ART automation test suit.

The test case is TestCase10443 to be precise.
Here is what it does:

1. Create a disk with wipe after delete attribute 
2. Create a VM and install OS on it 
3. Attach the disk to the VM => disk is created with 'wipe_after_delete' = True
4. Create a file from guest 
5. Remove the disk => so the disk data should be deleted/wiped

Actual result:
I grepped "Combining activation change with other commands is not advised" in the appropriate vdsm log (turned it to DEBUG just in case). As suggested by Vojtek. This string wasnt found in the log.

Comment 27 errata-xmlrpc 2020-08-04 13:26:06 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (RHV RHEL Host (ovirt-host) 4.4), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.