Bug 1482465 - [RFE] Add sorting for Cluster's columns
Summary: [RFE] Add sorting for Cluster's columns
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 4.2.0
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ovirt-4.4.0
: ---
Assignee: Steven Rosenberg
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-08-17 10:52 UTC by Michael Burman
Modified: 2020-05-20 20:01 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
With this update, when viewing clusters, you can sort by the Cluster CPU Type and Compatibility Version columns.
Clone Of:
Environment:
Last Closed: 2020-05-20 20:01:14 UTC
oVirt Team: Virt
Embargoed:
rbarry: ovirt-4.4?
rule-engine: planning_ack?
pm-rhel: devel_ack+
mburman: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 100853 0 master MERGED engine: Add Sorting to Cluster Columns 2020-05-12 08:14:29 UTC

Description Michael Burman 2017-08-17 10:52:50 UTC
Description of problem:
[UI] - Add sorting for Cluster's columns.
The Cluster's columns are missing the sorting option as we have for all columns in the system.
The only sorting option which is available for Cluster column is 'Description' and 'Name'.
We should add sorting for 'Cpu type', 'Data Center', 'Compatibility version', 'Host count' and 'VM count'

Version-Release number of selected component (if applicable):
4.2.0-0.0.master.20170815155420.gitaed789a.el7.centos

How reproducible:
100%

Steps to Reproduce:
1. Have several clusters with different comparability versions
2. Go to 'Clusters' and try to sort clusters by comparability version

Actual results:
Missing sorting for most columns under Clusters

It's not new thing as on older versions it behaved the same.

Comment 1 Tomas Jelinek 2017-08-28 07:59:20 UTC
The reason is that the search does not support sorting by this columns, so it can not be exposed to the UI.

Changing to an RFE.

Comment 2 Steven Rosenberg 2019-06-16 15:34:17 UTC
Notes:

I addressed the sorting for the following fields in the UI Cluster View:

1. CPU Type
2. Data Center
3. Compatibility Version

These fields are included in the sql cluster view.

Support for sorting the Host Count and VM Count will be more complex because they are fetched separately and are not included in the sql cluster view.

To add them to the sql cluster view may have a performance impact because when the UI is connected, the sql cluster view is called to refresh the the cluster data
and these are fields that are calculated by counting the number of vds_static and vm_static entries correlating to each cluster.

Another option is to create another sql cluster view just for the sorting and make sure that for sorting only this is called, 
but this will require more infrastructure changes.

The question is if it is worth the cost and if so, we may consider creating a separate RFE for this.

Comment 3 Martin Tessun 2019-07-03 15:58:19 UTC
Let's do Host count and VM count in a separate RFE then.

Comment 4 Sandro Bonazzola 2019-08-19 11:24:22 UTC
Moving back to post since no target milestone has been set for this bug and it's not clear if it needs backport or not.
Please target this bug.

Comment 5 RHEL Program Management 2019-08-19 14:15:00 UTC
This request has been proposed for two releases. This is invalid flag usage. The ovirt-future release flag has been cleared. If you wish to change the release flag, you must clear one release flag and then set the other release flag to ?.

Comment 21 Michael Burman 2020-04-01 10:55:48 UTC
Verified on - rhvm-4.4.0-0.29.master.el8ev.noarch

Comment 22 Sandro Bonazzola 2020-05-20 20:01:14 UTC
This bugzilla is included in oVirt 4.4.0 release, published on May 20th 2020.

Since the problem described in this bug report should be
resolved in oVirt 4.4.0 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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