Bug 848354

Summary: [webadmin][setupNetworks] Unsynced networks boot parameters can be changed
Product: Red Hat Enterprise Virtualization Manager Reporter: Mike Kolesnik <mkolesni>
Component: ovirt-engine-webadmin-portalAssignee: Alona Kaplan <alkaplan>
Status: CLOSED CURRENTRELEASE QA Contact: Meni Yakove <myakove>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.1.0CC: dyasny, ecohen, iheim, lpeer, Rhev-m-bugs, sgrinber, ykaul
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: network
Fixed In Version: si17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 20:01:41 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screenshot none

Description Mike Kolesnik 2012-08-15 11:26:10 UTC
Description of problem:
If an unsynched network exists on a Host's NIC, in setup networks the 'edit network' dialog will all be greyed out until the "sync network" checkbox is checked.
When checked, all fields become editable, but if unchecked after that then they become greyed out and still contain the modified values.

Attached is a screenshot where the original state was 'boot protocol = none'.

I believe that in this state, if the user will click OK in the setup networks dialog he will get a can do action failure from the engine side which claims that unsynced networks cannot be edited unless synced.


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


How reproducible:
Always


Steps to Reproduce:
1. Have an unsynced network on a host's nic:
1.1. Attach network to a NIC.
1.2. Change the network MTU to something different.
2. In setup networks dialog for the host's NICs, click the pencil on the unsynched network.
3. Check the 'sync network' checkbox.
4. Change the boot protocol to something different.
5. Uncheck the 'sync network' checkbox.
  
Actual results:
The modified boot protocol values will remain in the edit dialog.


Expected results:
When unchecked, the values before checking the box should return.


Additional info:
This should also be the case if the checkbox is unchecked after the dialog was closed and open again.

Comment 1 Mike Kolesnik 2012-08-15 11:47:14 UTC
Created attachment 604578 [details]
screenshot

Comment 2 Simon Grinberg 2012-08-15 16:19:34 UTC
(In reply to comment #0)

> 
> I believe that in this state, if the user will click OK in the setup
> networks dialog he will get a can do action failure from the engine side
> which claims that unsynced networks cannot be edited unless synced.

If that happens this is indeed wrong - but this is the only real problem which you are unsure yet. This needs to be verified 

> 
> Expected results:
> When unchecked, the values before checking the box should return.

This is not a must as long as the changes are ignored, though I admit it looks better.  

There are many applications that will not restore old value but just ignore changes. So if it keeps to that, this is not urgent to fix (if at all)

Comment 3 Alona Kaplan 2012-08-30 12:56:21 UTC
upstream commit 144c89384f8bc44c0d7cf41d60f008f7699b5367

Comment 4 Alona Kaplan 2012-09-02 15:43:24 UTC
the first commit caused a bug- fixed in upstream commit c286544f8167a7e1d5dfcb8ee8c38adc065514c6

Comment 5 Alona Kaplan 2012-09-02 16:32:24 UTC
phase 3- upstream coomit 91e8ed3a0dae71dd364c6aeeacee907d5f46b3cf

Comment 7 Meni Yakove 2012-09-05 13:46:16 UTC
Verified on rhevm-3.1.0-15.el6ev.noarch