Change in ovirt-engine[master]: core: fix matching of lun disks

Code Review gerrit at ovirt.org
Wed Aug 16 08:09:06 UTC 2017


>From Arik Hadas <ahadas at redhat.com>:

Arik Hadas has submitted this change and it was merged. ( https://gerrit.ovirt.org/80533 )

Change subject: core: fix matching of lun disks
......................................................................


core: fix matching of lun disks

The path of LUN disks is as follow:
/dev/mapper/<lun id>

Previously we matched disks by checking if their path contains
the disk UUID. Therefore, this check is wrong for LUN disks.

This patch extends the matching of disks to cover LUN disks.

Change-Id: Iad890241b7aa1be1aa70ffdfbb8c8e5115a256c4
Bug-Url: https://bugzilla.redhat.com/1479838
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/libvirt/VmDevicesConverter.java
1 file changed, 22 insertions(+), 3 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified; Looks good to me, approved
  Idan Shaby: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/80533
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iad890241b7aa1be1aa70ffdfbb8c8e5115a256c4
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list