Bug 1482465

Summary: [RFE] Add sorting for Cluster's columns
Product: [oVirt] ovirt-engine Reporter: Michael Burman <mburman>
Component: Backend.CoreAssignee: Steven Rosenberg <srosenbe>
Status: CLOSED CURRENTRELEASE QA Contact: Michael Burman <mburman>
Severity: low Docs Contact:
Priority: unspecified    
Version: 4.2.0CC: bugs, emesika, gshereme, michal.skrivanek, mtessun, rbarry, sgoodman, srosenbe, tjelinek
Target Milestone: ovirt-4.4.0Keywords: FutureFeature
Target Release: ---Flags: rbarry: ovirt-4.4?
rule-engine: planning_ack?
pm-rhel: devel_ack+
mburman: testing_ack+
Hardware: x86_64   
OS: Linux   
Whiteboard:
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.
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-05-20 20:01:14 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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.