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: | UI | Assignee: | Ryan Barry <rbarry> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | cshao <cshao> |
Severity: | low | Docs Contact: | |
Priority: | low | ||
Version: | 4.0 | CC: | bugs, cshao, fdeutsch, huzhao, leiwang, mgoldboi, rbarry, weiwang, yaniwang, ycui |
Target Milestone: | ovirt-4.0.0-rc | Keywords: | FutureFeature |
Target Release: | 4.0 | Flags: | 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
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)? 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? 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. 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! 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. oVirt 4.0.0 has been released, closing current release. |