Bug 974436 - Edit alert condition -> metric value is not converted to target units
Summary: Edit alert condition -> metric value is not converted to target units
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.8
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: RHQ 4.8
Assignee: Jirka Kremser
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-14 08:00 UTC by Heiko W. Rupp
Modified: 2013-09-11 09:53 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-09-11 09:53:23 UTC
Embargoed:

Attachments (Terms of Use)
Screenshot (34.80 KB, image/png)
2013-06-14 08:00 UTC, Heiko W. Rupp 		no flags Details
View All

Description Heiko W. Rupp 2013-06-14 08:00:36 UTC 
Created attachment 761132 [details]
Screenshot

Add an alert definition with e.g.

Free Memory < 1GB

save

try to edit again.
Wizard now shows

Free Memory < 

10203234113  (or whatever, the equivalent of 1GB is)

See screenshot fo details.

The UI should try to convert that numeric value back into "1GB" before
presenting to the user
Comment 1 Heiko W. Rupp 2013-06-18 15:22:13 UTC 
Worse:

when you initially enter 50% and then edit, the UI shows "0.5". When you
modify that to 0.49 and save, you end up with "0.5%"
Comment 2 Jirka Kremser 2013-06-20 14:03:45 UTC 
branch:  master
link:    http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=b45eb6516
time:    2013-06-20 15:55:48 +0200
commit:  b45eb6516643622a97360700ecd52a899f90c8be
author:  Jirka Kremser - jkremser
message: [BZ 974436] - Edit alert condition -> metric value is not converted to
         target units - Converting correctly the value based on it's
         measurement units family.
Comment 3 Heiko W. Rupp 2013-09-11 09:53:23 UTC 
Bulk closing of old issues now that HRQ 4.9 is in front of the door.

If you think the issue has not been solved, then please open a new bug and mention this one in the description.
Note You need to log in before you can comment on or make changes to this bug.