Bug 1349498

Summary:	When a VM is started, attached disks can't be edited anymore
Product:	[oVirt] ovirt-engine	Reporter:	Roman Mohr <rmohr>
Component:	Frontend.WebAdmin	Assignee:	Tal Nisan <tnisan>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Kevin Alon Goldblatt <kgoldbla>
Severity:	medium	Docs Contact:
Priority:	high
Version:	4.0.0	CC:	acanan, amureini, aopetrov86, bugs, ishaby, rgolan, tnisan, ylavi
Target Milestone:	ovirt-4.0.2	Flags:	rule-engine: ovirt-4.0.z+ rule-engine: planning_ack+ tnisan: devel_ack+ acanan: testing_ack+
Target Release:	4.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:	2016-08-12 14:28:08 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 Roman Mohr 2016-06-23 14:39:26 UTC

Description of problem:
Prior to 4.0 it was always possible to change disk properties on running disks (e.g. Disk Profile) when going to the disks subtab of a VM. There you can select the disk and click "Edit". Now everything except of 'bootbale' is greyed out.

Either accidently or deliberately the whole dialogue is disabled now for Disks but there are editable fields on it which should be changeable when the VM is running.

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


How reproducible:
always

Steps to Reproduce:
1. Create a VM
2. Create a Disk
3. Attach the disk to the VM
4. Start the VM
5. Try to change disk properties on the running VM  in the Disk subtab of the VM

Actual results:
Everything is greyed out

Expected results:
Everthing which should be live changeable should not be greyed out.


Additional info:

Comment 1 Allon Mureinik 2016-07-04 10:38:53 UTC

Idan/Tal, haven't we seen thus one already?

Comment 2 Idan Shaby 2016-07-05 05:34:35 UTC

Don't know about Tal, but I haven't seen it before.
Indeed everything is greyed out except for the bootable checkbox.

Comment 3 Tal Nisan 2016-07-05 08:13:29 UTC

I haven't as well, but looking into it soon

Comment 4 Yaniv Kaul 2016-07-15 19:07:19 UTC

(In reply to Tal Nisan from comment #3)
> I haven't as well, but looking into it soon

Any updates?

Comment 5 Roy Golan 2016-07-17 10:42:24 UTC

This is blocking the disk profile live update blocker Bug 1328731 and Bug 1201482 (the dependent)

Comment 6 Roy Golan 2016-07-17 10:45:59 UTC

Tal is this due to 5f56c2f0124af51a0e33f0557bb99eaf943aed49 ?

Comment 7 Tal Nisan 2016-07-17 15:29:18 UTC

(In reply to Roy Golan from comment #6)
> Tal is this due to 5f56c2f0124af51a0e33f0557bb99eaf943aed49 ?

No, this patch disabled only one checkbox and only in attach disk popup

Comment 8 Tal Nisan 2016-07-20 13:43:15 UTC

*** Bug 1358177 has been marked as a duplicate of this bug. ***

Comment 9 Kevin Alon Goldblatt 2016-07-25 12:51:38 UTC

Tested with the following code:
-------------------------------------
rhevm-4.0.2-0.1.rc.el7ev.noarch
vdsm-4.18.8-1.el7ev.x86_64

Verified with the following scenario:
-------------------------------------
Steps to reproduce:
1. Create a VM
2. Create a Disk
3. Attach the disk to the VM
4. Start the VM
5. Try to change disk properties on the running VM  in the Disk subtab of the VM

Actual results:
The Extend Size by(GB) can now be changed!
The Alias can now be changed!
The description can now be changed!
The Disk Profile can now be changed!


Expected results:
Everthing which should be live changeable IS NO LONGER greyed out but EDITABLE!


Moving to VERIFIED!