Bug 1334651

Summary: [RFE] Should not change the virtual machines count every times when enter dashboard page
Product: [oVirt] ovirt-node Reporter: wanghui <huiwa>
Component: UIAssignee: Ryan Barry <rbarry>
Status: CLOSED CURRENTRELEASE QA Contact: cshao <cshao>
Severity: low Docs Contact:
Priority: low    
Version: 4.0CC: bugs, cshao, fdeutsch, huzhao, leiwang, mgoldboi, rbarry, weiwang, yaniwang, ycui
Target Milestone: ovirt-4.0.0-rcKeywords: FutureFeature
Target Release: 4.0Flags: rule-engine: ovirt-4.0.0+
ycui: testing_plan_complete?
mgoldboi: planning_ack+
fdeutsch: devel_ack+
cshao: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: cockpit-ovirt-dashboard-0.9.1-0.0.1.el7ev Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-05 07:51:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Node RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description wanghui 2016-05-10 08:53:52 UTC
Description of problem:
It should not change the virtual machines count every time when enter dashboard page even if there is no new changes. It always changes from 0 Running to X Running when enter the dashboard page. It should be better just stay on X Running when there is no changes.

Version-Release number of selected component (if applicable):
rhev-hypervisor7-ng-20160506.0.el7
imgbased-0.6-0.1.el7ev.noarch
ovirt-hosted-engine-setup-1.3.6.1-1.el7ev.noarch
ovirt-host-deploy-1.4.1-1.el7ev.noarch
ovirt-hosted-engine-ha-1.3.5.5-1.el7ev.noarch

How reproducible:
100%

Steps to Reproduce:
1. Install rhev-hypervisor7-ng-3.6-20160506.0.el7
2. Disable NetworkManager service.
3. Create a virtual machine.
4. Check the dashboard.

Actual results:
1. It always change from 0 Running to 1 Running when enter the dashboard page.

Expected results:
1. It should keep 1 Running until other changes came.

Additional info:

Comment 1 Ryan Barry 2016-05-11 00:31:22 UTC
There is no practical way to fix this.

It's a stateless application which updates the count when a dbus call returns. We cannot keep the state inbetween different pages.

Without blocking the entire page load until it does (bad, and Javascript doesn't like to do this anyway, since the dbus object returns promises), the available solutions are:

Use some placeholder text (but that will also get updated almost immediately).

Use redux as a backend, but that's low priority.

Fabian, can you devel_ack- this, or retarget to 4.1 (where we'll have time to implement redux)?

Comment 2 Ryan Barry 2016-05-11 02:42:05 UTC
I tested -- we can block showing that element until we get a response. This way the count of VMs will not change from 0->$number (though starting/stopping VMs will still change the count)

I'd really rather not block the entire page, though, for a variety of reasons.

Does that work?

Comment 3 Fabian Deutsch 2016-05-11 07:33:49 UTC
Just for considerations, what about showing a placeholder until the count is available, i.e.

- ?
- ⋯

But only showing the element once the count is available works for me too.

Comment 4 wanghui 2016-06-02 05:22:44 UTC
Test version:
rhev-hypervisor7-ng-4.0-20160527.0
cockpit-ovirt-dashboard-0.10.1-0.0.1.el7ev.noarch

Test steps:
1. Install rhev-hypervisor7-ng-3.6-20160527.0
2. Create a virtual machine.
3. Check the dashboard.

Test result:
1. The Virtual Machines's summary page is blocked before get the latest status.

Hi Ryan,

For this issue, the vm's whole summary page is blocked before get the latest status. So every time when enter this page, the page will changed from without VM's column to have it. Do you think it's better to add a loading icon as a placeholder when waiting of a update?

Thanks!

Comment 5 Ryan Barry 2016-06-02 11:52:29 UTC
This should be a small UX change.

I am ok with adding a small loading icon (I'll push a patch up), but it shouldn't be necessary for verification of this bug, in my opinion.

Comment 6 wanghui 2016-06-03 03:21:49 UTC
Verify this issue due to comment#4&5.

Comment 9 Sandro Bonazzola 2016-07-05 07:51:20 UTC
oVirt 4.0.0 has been released, closing current release.