Bug 1479838 - Wrong matching for LUN disks
Summary: Wrong matching for LUN disks
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ovirt-4.2.0
: ---
Assignee: Arik
QA Contact: Kevin Alon Goldblatt
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-08-09 14:17 UTC by Arik
Modified: 2017-12-20 10:52 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-12-20 10:52:41 UTC
oVirt Team: Virt
Embargoed:
rule-engine: ovirt-4.2+
rule-engine: blocker+
ahadas: devel_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 80533 0 master MERGED core: fix matching of lun disks 2017-08-16 08:09:09 UTC

Description Arik 2017-08-09 14:17:27 UTC
Description of problem:
In 4.2, the matching of the devices reported form libvirt with those defined in the database was moved from vdsm to the engine. But the logic that was introduced for matching the disks is incomplete. It is done according to the 'path' of the disk (like VDSM used to do), but assumes the following logic: the path must contain the disk id. 
However, this logic doesn't hold for LUN disks since their path is composed of "/dev/mapper/<lun_id>". Thus, we need to extend the matching logic for LUN disks, to base on the lun_id instead.

Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. Run a VM with a plugged LUN disk
2. 
3.

Actual results:
The LUN disk is being unplugged in the engine database

Expected results:
The LUN disk should remain plugged

Additional info:
If the disk is plugged after being unplugged in a previous run of the VM, the VM will start and the LUN disk will remain plugged. So in order to reproduce this bug, we'll need to enforce the devices hash to change by, for example, plugging a NIC.

Comment 1 Arik 2017-08-09 14:19:20 UTC
The fix is supposed to be relatively easy but it needs to be verified with LUN disks so this better gets handled by the storage team.

Comment 2 Red Hat Bugzilla Rules Engine 2017-08-14 08:35:05 UTC
This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 3 Kevin Alon Goldblatt 2017-11-28 17:09:54 UTC
Verified with the following code:
-------------------------------------------
ovirt-engine-4.2.0-0.5.master.el7.noarch
vdsm-4.20.8-53.gitc3edfc0.el7.centos.x86_64

Verified with the following scenario:
-------------------------------------------
1. Created a vm, attached a direct LUN to the VM (LUN is plugged)
2. Started the vm (LUN is plugged)
3. Added nic (LUN remains plugged)


Moving to VERIFIED

Comment 4 Sandro Bonazzola 2017-12-20 10:52:41 UTC
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

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


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