Bug 994904 - Edit dialog for alert conditions loads condition parameters incorrectly
Summary: Edit dialog for alert conditions loads condition parameters incorrectly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Alerts, Core UI
Version: 4.8
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: RHQ 4.10
Assignee: Jirka Kremser
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On: 973140
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-08 08:43 UTC by Filip Brychta
Modified: 2014-04-23 12:30 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-04-23 12:30:24 UTC
Embargoed:


Attachments (Terms of Use)
alert condition (146.25 KB, image/png)
2013-08-08 08:43 UTC, Filip Brychta
no flags Details

Description Filip Brychta 2013-08-08 08:43:45 UTC
Created attachment 784254 [details]
alert condition

Description of problem:
When editing alert condition of 'Measurement Absolute Value Threshold' type, a 'Metric' field is filled incorrectly. See attached screen shot. There is a number (id of metric probably) instead of name of the metric. 

Version-Release number of selected component (if applicable):
Version: 4.9.0-SNAPSHOT
Build Number: ee56897

How reproducible:
Always

Steps to Reproduce:
1. create alert definition and add condition of type 'Measurement Absolute Value Threshold' (i.e. on RHQ agent resource)
2. select some metric, comparator, put a value
3. Save the condition
4. select the condition and pop up edit dialog

Actual results:
see attached screen shot

Expected results:
correct metric name is displayed

Additional info:
the same problem affects 'Measurement Baseline Threshold', 'Measurement Value Change', 'Measurement Value Range' types

Comment 1 Filip Brychta 2013-08-08 13:21:31 UTC
Covered by automation - com.redhat.qe.jon.sahi.tests.alerts.AlertDefinitioinTest

Comment 2 Jirka Kremser 2013-09-09 12:37:22 UTC
Works in master, it was probably fixed by commit 9c2e41ff

Comment 3 Filip Brychta 2013-10-30 15:57:04 UTC
Verified on
Version :	
4.10.0-SNAPSHOT
Build Number :	
afc0857

Comment 4 Filip Brychta 2014-01-06 12:57:45 UTC
Moving it back to ON_DEV. The issue is still there for some combinations. 
Use following conditions:
 - Metric Value Baseline [Up Time < 12.0 % of max]
 - Metric Value Threshold [Number of Commands Sent but Failed = 12.0]
 - Metric Value Range: [Number of Agent Restarts] outside [12.0] and [20.0], exclusive
 - Metric Value Change [Number of Commands Received but Failed  per Minute]

Comment 5 Jirka Kremser 2014-01-06 17:30:49 UTC
branch:  master
link:    http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=8b14a1f9c
time:    2014-01-06 18:13:19 +0100
commit:  8b14a1f9c08d9f40fd3692e1c2da5a977ce8b57d
author:  Jirka Kremser - jkremser
message: [BZ 994904] - Edit dialog for alert conditions loads condition
         parameters incorrectly - The metric name was filtered out as a
         reaction on BZ 737565. Although, it does make sense to do that
         for all _other_ conditions, it does not make sense to filter
         out the metric name of the currently modified condition. (the
         original bug was fixed before the UI was capable of editing
         existing alert conditions).

Comment 6 Jirka Kremser 2014-01-06 17:44:55 UTC
btw. only these two conditions were necessary for reproduction (on RHQ Agent resource):

 - Metric Value Threshold [Number of Commands Sent but Failed = 12.0]
 - Metric Value Range: [Number of Agent Restarts] outside [12.0] and [20.0]

Comment 7 Filip Brychta 2014-01-08 09:11:09 UTC
Verified on:
4.10.0-SNAPSHOT
Build Number :	
5450097

Comment 8 Heiko W. Rupp 2014-04-23 12:30:24 UTC
Bulk closing of 4.10 issues.

If an issue is not solved for you, please open a new BZ (or clone the existing one) with a version designator of 4.10.


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