Bug 720944

Summary: Properties of Configuration in update report are not usable
Product: [Other] RHQ Project Reporter: Heiko W. Rupp <hrupp>
Component: ConfigurationAssignee: Lukas Krejci <lkrejci>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: high Docs Contact:
Priority: urgent    
Version: 4.1CC: hrupp, lkrejci
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-07 19:18:11 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

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