Bug 720944 - Properties of Configuration in update report are not usable
Summary: Properties of Configuration in update report are not usable
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Configuration
Version: 4.1
Hardware: Unspecified
OS: Unspecified
urgent
high vote
Target Milestone: ---
: ---
Assignee: Lukas Krejci
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-07-13 10:11 UTC by Heiko W. Rupp
Modified: 2012-02-07 19:18 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-07 19:18:11 UTC


Attachments (Terms of Use)

Description Heiko W. Rupp 2011-07-13 10:11:07 UTC
[11:56:15] <pilhuhn> Configuration conf = report.getConfiguration();  // From a config update report in the agent
[11:56:29] <pilhuhn> this has 38 properties defined
[11:56:30] <pilhuhn> for (Property prop  : conf.getProperties()) {
[11:56:45] <pilhuhn> will NPE, as conf.getProperties() returns null
[11:58:32] <lkrejci> do you deserialize this before you use it like this?
[11:59:12] <pilhuhn> well... I get the report passed in the agent in the 
                org.rhq.core.pluginapi.configuration.ConfigurationFacet#updateResourceConfiguration method
[12:01:08] <pilhuhn> In the debugger i see 38 properties and propertyProxy being null

With this issue, updating any resource configuration is likely to fail.

Comment 1 Lukas Krejci 2011-07-13 11:46:38 UTC
To test this, you need to try and update the resource configuration in AS7 plugin.

commit c8dc1d812b5a3dd9c2cd8ffd55f2ec803d6b81e6
Author: Lukas Krejci <lkrejci>
Date:   Wed Jul 13 12:35:29 2011 +0200

    BZ 720944 - lazy init the propertiesProxy to prevent problems after deserialization of a Configuration instance.

Comment 2 Heiko W. Rupp 2011-07-13 11:50:02 UTC
This fix works for me.

Comment 3 Mike Foley 2011-07-28 19:41:23 UTC
verified as follows:  
RHQ 4.1 master, 7/28 build
imported AS7 plug-in
auto-discovery of my AS7 CR1 "white rabbit"
resource configuration ... edit'd and saved values.

Comment 4 Mike Foley 2012-02-07 19:18:11 UTC
changing status of VERIFIED BZs for JON 2.4.2 and JON 3.0 to CLOSED/CURRENTRELEASE


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