Bug 895468

Summary:	Lexicographic sorting by IP when searching for VMs
Product:	[Retired] oVirt	Reporter:	Lior Vernia <lvernia>
Component:	ovirt-engine-webadmin	Assignee:	Lior Vernia <lvernia>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	low
Version:	unspecified	CC:	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:
Clones:	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)