Bug 593223 - Alert Notification: fails to save the command line arguments for script execution
Summary: Alert Notification: fails to save the command line arguments for script execu...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Alerts
Version: 3.0.0
Hardware: All
OS: Linux
high
high
Target Milestone: ---
: ---
Assignee: Joseph Marques
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On:
Blocks: jon24-alert-notify
TreeView+ depends on / blocked
 
Reported: 2010-05-18 08:40 UTC by Sudhir D
Modified: 2010-08-12 16:44 UTC (History)
2 users (show)

Fixed In Version: 2.4
Clone Of:
Environment:
Last Closed: 2010-08-12 16:44:19 UTC
Embargoed:


Attachments (Terms of Use)

Description Sudhir D 2010-05-18 08:40:43 UTC
Description of problem:
When an alert sender type for Resource Operations is created to execute script, once the command line arguments to be passed is entered in the Value filed, clicking save reset back to Unset(Checked).

Version-Release number of selected component (if applicable):
Build# 152. Version 2.4 Snapshot

How reproducible:
Always

Steps to Reproduce:
1.Create a new alert definition.
2.Click on Edit for the Alert Notification.
3.On the Alert notification page, click on Add New. Select the Alert sender
type as Resource Operations.
4.Select the resource selection mode as Relative Resource. 
5.Click Save
6.Select Start search from as "JBoss AS Server"
7.Click Save.
8.Select Then Filter by as "Script"
9.Provide the name of the script that should be executed upon meeting the alert condition.
10.Select operation name as "Execute Script"
11.Click Save.
12.Uncheck the Unset for Command line arguments and enter the value that needs to be passed.
13.Click Save.

  
Actual results:
clicking save reset back to Unset(Checked)

Expected results:
command line arguments that need to be passed should be retained.

Additional info:

Comment 1 Charles Crouch 2010-06-08 03:08:00 UTC
Is this still a problem Sudhir?

Comment 2 Sudhir D 2010-06-08 11:58:42 UTC
Yes. Retried it on Beta build and issue still persist.

Comment 3 Joseph Marques 2010-06-10 09:25:20 UTC
commit b848dc16492a1c34dc043c9fcd7243e9ac38cfd7
Author: Joseph Marques <joseph>
Date:   Thu Jun 10 04:27:59 2010 -0400

    BZ-593223: add ability to save/restore operationArguments for operation-based alert senders
    
    'extra parameters' base support:
    * augment AlertNotification entity store "extra parameters" configuration object
    * add dbsetup/dbupgrade parts for this "extra parameters" element
    * modify backend and UI code to do managed loading/persistence of this extra notification info
    ** update various locations in servar/jar, server/plugin/pc/alert, and portal/war for retrieval/storage
    
    operation-based alert sender improvements:
    * modify the load/save algorithms to use "extra parameters" config to retrieve/store arguments for selected operation
    * remove enum and legacy referencs to OperationInfo.Constants.ARGUMENTS_CONFIG_ID

Comment 4 Corey Welton 2010-06-10 19:02:26 UTC
QA Verified. cmdline parameters can be successfully passed.

Note:  discovered bug #602788 while verifying this.

Comment 5 Corey Welton 2010-08-12 16:44:19 UTC
Mass-closure of verified bugs against JON.


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