Bug 848354 - [webadmin][setupNetworks] Unsynced networks boot parameters can be changed
Summary: [webadmin][setupNetworks] Unsynced networks boot parameters can be changed
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.1.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Alona Kaplan
QA Contact: Meni Yakove
URL:
Whiteboard: network
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-08-15 11:26 UTC by Mike Kolesnik
Modified: 2016-02-10 19:53 UTC (History)
7 users (show)

Fixed In Version: si17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-04 20:01:41 UTC
oVirt Team: Network
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot (432.22 KB, image/png)
2012-08-15 11:47 UTC, Mike Kolesnik
no flags Details

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


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