Bug 617718 - OID field of SNMP trap alert notification getting dropped during database upgrade
Summary: OID field of SNMP trap alert notification getting dropped during database upg...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Database
Version: 3.0.0
Hardware: All
OS: Linux
urgent
high
Target Milestone: ---
: ---
Assignee: Joseph Marques
QA Contact: John Sanda
URL:
Whiteboard:
Depends On:
Blocks: jon-sprint12-bugs
TreeView+ depends on / blocked
 
Reported: 2010-07-23 19:40 UTC by John Sanda
Modified: 2010-08-12 16:48 UTC (History)
2 users (show)

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


Attachments (Terms of Use)

Description John Sanda 2010-07-23 19:40:05 UTC
Description of problem:
When I upgraded from 2.3.1 to 2.4.0, I have a few alert definitions with SNMP trap notifications. It appears that the OID fields of the SNMP traps are getting dropped/lost during the database upgrade.

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


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Charles Crouch 2010-07-23 22:39:06 UTC
From joseph:

'this I think can be fixed by changing line 76 of the rhq-serverplugin.xml file to "oid" instead of "OID".  the data should have been properly translated during db-upgrade, but the metadata in the server-side plugin is accessing the wrong property due to case-sensitivity'

John can you try this fix out and if it works, joseph can push the change to release branch.

Comment 2 John Sanda 2010-07-25 01:28:05 UTC
Tested the change as per Joseph's suggestion locally, and it looked good. I pushed the commit to the release branch.

commit hash: c3b143eec4f67612e868a947f370f1dd7bfc7dee

I will verify when hudson has a build ready.

Comment 3 Heiko W. Rupp 2010-07-26 11:09:55 UTC
The above change is *not* enough, as the alert sender depends on the name of 
the property too.

Fix is in 0bfff55

Comment 4 Corey Welton 2010-07-27 01:56:19 UTC
QA Verified -- the values for alerts now persist across an upgrade.  As noted by others earlier, they do /not/ persist from the JON system settings (<2.4) to the JON plugin config (2.4+) level.  As long as we have a docs bug for that, I think we're good to go here.

Comment 5 Corey Welton 2010-08-12 16:48:36 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.