Bug 895468

Summary: Lexicographic sorting by IP when searching for VMs
Product: [Retired] oVirt Reporter: Lior Vernia <lvernia>
Component: ovirt-engine-webadminAssignee: Lior Vernia <lvernia>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: low    
Version: unspecifiedCC: acathrow, bazulay, ecohen, iheim, lpeer, lvernia, mgoldboi
Target Milestone: ---   
Target Release: 3.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: network
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 905446 (view as bug list) Environment:
Last Closed: 2013-09-23 07:32:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 905446    

Description Lior Vernia 2013-01-15 09:41:05 UTC
Description of problem:

When using the search panel to search for VMs, and asking to sort by IP address, the sorting is done lexicographically and not numerically (i.e. 192.168.0.10 would appear before 192.168.0.2).


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


How reproducible:

Always.


Steps to Reproduce:
1. Choose Virtual Machines tab.
2. Type in search panel "Vms: sortby ip asc".
  
Actual results:

IPs ordered by lexicographic sort.


Expected results:

IPs ordered by numeric sort.


Additional info:

Comment 1 Itamar Heim 2013-01-15 12:22:27 UTC
please see bug 590413

Comment 2 Lior Vernia 2013-08-04 19:46:20 UTC
This was opened as a more accurate version of https://bugzilla.redhat.com/show_bug.cgi?id=590413, as it described the table header being clickable when in the meantime we moved to GWT and lost the ability to click on table headers.

However, the RHEV version of the bug was solved long ago, for RHEV 3.2 and maybe even for oVirt 3.2: https://bugzilla.redhat.com/show_bug.cgi?id=905446

I guess we just forgot about this one or I didn't know what to do with it at the time.

Comment 3 Itamar Heim 2013-08-04 20:25:24 UTC
iirc, this missed ovirt 3.2 and is part of 3.3, hence in status modified.
any reason it should be back in post?

Comment 4 Lior Vernia 2013-08-05 07:04:40 UTC
Oh no, that was just my mistake. And yes, it seems to have missed the oVirt 3.2 release, sorry about the confusion.

Comment 5 Itamar Heim 2013-08-21 16:40:18 UTC
as RC is built, moving to ON_QA (hopefully did not catch incorrect bugs when doing this)

Comment 6 Itamar Heim 2013-09-23 07:32:25 UTC
closing as this should be in 3.3 (doing so in bulk, so may be incorrect)