Bug 1067311 - Refresh interval button disappears from Network interfaces subtab of specific host
Summary: Refresh interval button disappears from Network interfaces subtab of specific...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: 3.4
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
: 3.4.0
Assignee: Lior Vernia
QA Contact: GenadiC
URL:
Whiteboard: network
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-20 07:57 UTC by GenadiC
Modified: 2015-05-05 01:44 UTC (History)
11 users (show)

Fixed In Version: ovirt-3.4.0-ga
Clone Of:
Environment:
Last Closed: 2014-03-31 12:28:59 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 24835 0 None MERGED webadmin: revert refresh synchornization Never

Description GenadiC 2014-02-20 07:57:49 UTC
Description of problem:
When choosing Network interfaces subtab for specific Host there is a Refresh interval icon. It appears and disappears once in several seconds

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


How reproducible:
Always

Steps to Reproduce:
1. Choose specific Host under Hosts on UI
2. Pick for that Host Network Interfaces subtab
3. Check for a refresh button

Actual results:
After several seconds the refresh button disappears and comes back only after you move to another tab and come back to Network Interfaces subtab ( and then it disappears again)
Expected results:
The refresh button should be presented all the time

Additional info:

Comment 1 Einav Cohen 2014-02-20 15:21:44 UTC
this is a straight forward bug, no input/advice from the RHEV User Experience stakeholders is required -> removing the UserExperience Keyword.

Comment 2 Alexander Wels 2014-02-20 15:22:52 UTC
I believe the problem lies in SubTabHostInterfacePresenter.java:

line 76: getView().setRefreshButtonVisibility(!getModelProvider().getModel().getTimer().isActive());

Should be:

getView().setRefreshButtonVisibility(getModelProvider().getModel().getTimer().isActive());

You will notice part of the refresh button shown in 3.1 DC hosts, while it not be there at all for > 3.1 DC hosts.

Comment 3 Alexander Wels 2014-02-20 15:25:21 UTC
(In reply to Alexander Wels from comment #2)
> I believe the problem lies in SubTabHostInterfacePresenter.java:
> 
> line 76:
> getView().setRefreshButtonVisibility(!getModelProvider().getModel().
> getTimer().isActive());
> 
> Should be:
> 
> getView().setRefreshButtonVisibility(getModelProvider().getModel().
> getTimer().isActive());
> 
> You will notice part of the refresh button shown in 3.1 DC hosts, while it
> not be there at all for > 3.1 DC hosts.

This is for 3.4. In 3.5 the refresh sync patch seems to affect this a little more, there is a patch http://gerrit.ovirt.org/#/c/24796 for 3.5 to solve this issue.

Comment 4 Lior Vernia 2014-02-23 08:52:43 UTC
(In reply to Alexander Wels from comment #2)
> I believe the problem lies in SubTabHostInterfacePresenter.java:
> 
> line 76:
> getView().setRefreshButtonVisibility(!getModelProvider().getModel().
> getTimer().isActive());
> 
> Should be:
> 
> getView().setRefreshButtonVisibility(getModelProvider().getModel().
> getTimer().isActive());
> 

Logic was put there on purpose, if I remember correctly following comment by derez on the patch (on master branch) fixing Bug 1027597. For newer hosts the subtab is refreshed periodically so there's no need for the button, for older hosts it can't be refreshed periodically so the button is shown.

This appears to be another regression due to the grid timer synchronization patch, therefore reverting that patch on the 3.4 branch should fix it for 3.4. However, if that patch isn't reverted on master, this bug should probably be cloned for 3.5.

Comment 5 Sandro Bonazzola 2014-03-03 15:23:50 UTC
This is an automated message.

This BZ should be fixed in oVirt 3.4.0 RC repository, assignee please update the status of the bug to ON_QA once you've verified that the change is included in the build.

Comment 6 GenadiC 2014-03-04 14:40:29 UTC
Verified in av2

Comment 7 Sandro Bonazzola 2014-03-31 12:28:59 UTC
this is an automated message: moving to Closed CURRENT RELEASE since oVirt 3.4.0 has been released


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