Bug 1362152

Summary:	Code change - move is_using_scsi_reservation to DiskVmElement
Product:	[oVirt] ovirt-engine	Reporter:	Tal Nisan <tnisan>
Component:	BLL.Storage	Assignee:	Tal Nisan <tnisan>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Raz Tamir <ratamir>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	4.0.0	CC:	bugs, ratamir, tnisan
Target Milestone:	ovirt-4.1.0-alpha	Flags:	rule-engine: ovirt-4.1+ rule-engine: planning_ack+ tnisan: devel_ack+ ratamir: testing_ack+
Target Release:	4.1.0.2
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-02-01 14:47:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Tal Nisan 2016-08-01 12:06:55 UTC

The property is_using_scsi_reservation in the vm_device table is relevant only for disks and was placed in the vm_device since there was no other entity to describe the relation between a disk and a VM at the time this RFE was written.
Now when we have disk_vm_element this property should be moved there.

Comment 2 Tal Nisan 2016-11-08 10:00:09 UTC

UI remains the same, in the REST this field will be visible and editable via DiskAttachment as well as Disk

Comment 3 Raz Tamir 2016-11-27 21:52:51 UTC

Verified on - 
ovirt-engine-4.1.0-0.0.master.20161126211319.gitae69c34.el7.centos.noarch

Comment 4 Sandro Bonazzola 2016-12-12 13:54:26 UTC

The fix for this issue should be included in oVirt 4.1.0 beta 1 released on December 1st. If not included please move back to modified.

Comment 5 Raz Tamir 2016-12-13 17:52:49 UTC

Verified on 4.1.0-0.2.master.20161212071212.git8a015dd.el7.centos

<uses_scsi_reservation>false</uses_scsi_reservation> is an attribute of diskAttachment object