Bug 1144459 - [eap6] Warnings in agent.log - Required property 'manage' and 'delete-durable-queue' have a null value
Summary: [eap6] Warnings in agent.log - Required property 'manage' and 'delete-durable...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Operations Network
Classification: JBoss
Component: Plugin -- JBoss EAP 6
Version: JON 3.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ER04
: JON 3.3.0
Assignee: Jay Shaughnessy
QA Contact: Filip Brychta
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-09-19 12:50 UTC by Filip Brychta
Modified: 2014-12-11 14:04 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-12-11 14:04:29 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Filip Brychta 2014-09-19 12:50:24 UTC
Description of problem:
There are 'Plugin Error' warnings in agent.log

Version-Release number of selected component (if applicable):
Version :	
3.3.0.ER03
Build Number :	
4aefe39:44e33a4

How reproducible:
2/2

Steps to Reproduce:
1. import eap6 standalone
2. navigate to eap6 standalone resource


Actual results:
Warnings in agent.log:
2014-09-19 07:06:27,497 WARN  [ConfigurationManager.threadpool-1] (rhq.core.pc.configuration.ConfigurationCheckExecutor)- Plugin Error: Invalid Core Address resource configuration returned by JBossAS7 plugin - Required property 'delete-durable-queue' has a null value

2014-09-19 07:06:27,497 WARN  [ConfigurationManager.threadpool-1] (rhq.core.pc.configuration.ConfigurationCheckExecutor)- Plugin Error: Invalid Core Address resource configuration returned by JBossAS7 plugin - Required property 'manage' has a null value

Expected results:
No plugin errors

Comment 2 Jay Shaughnessy 2014-09-23 20:34:29 UTC
An unusual issue.  Certain resource config props in the 'Roles' list (of maps) were being ignored because they looked like duplicates.  This was due to them having duplicate indexes with other properties in the Map.  When trying to add them to a TreeMap, the comparator considered the keys equal, and voila, map omissions.

The bug actually ends up being an issue in the plugin descriptor, where it looks like an evil cut+paste issue gave us duplicate property names.  The duplicate names caused the issue in the order_indexes.

After some investigation it seems to repair the problem we need only to fix the plugin config and update the plugin on the server and agent.  And then when the resource config is updated (when the res config checker runs) the issue will resolve itself.

Note that GUI sessions will need to be refreshed if open during the update because it may have cached the resource type (and its config def).

This should probably be part of an upgrade test as 3.2 does have the bad config.

commit to follow...

Comment 3 Jay Shaughnessy 2014-09-23 21:10:29 UTC
Master commit 722d7a462b7848425f3fb28bf5174db0122bb2e3
Author: Jay Shaughnessy <jshaughn>
Date:   Tue Sep 23 17:01:19 2014 -0400

    An issue in the plugin descriptor resulted in an ill-defined configuration
    definition for the 'Role' "list of maps".  Fixing the descriptor should
    resolve the issue when resource config is rediscovered for an existing
    resource.  New resources should be fine from the start.

    Also, fix an issue in the config editor when rendering a lot of property
    fields. Something that showed up after the all of the properties started
    showing up.

    The lesson here? Listen to your log file warnings :)



Release/jon3.3.x commit 3d477b03a194a6f1ab61b4d63b756283cc2ddc24
Author: Jay Shaughnessy <jshaughn>
Date:   Tue Sep 23 17:01:19 2014 -0400

    (cherry picked from commit 722d7a462b7848425f3fb28bf5174db0122bb2e3)
    Signed-off-by: Jay Shaughnessy <jshaughn>

Comment 4 Simeon Pinder 2014-10-01 21:33:19 UTC
Moving to ON_QA as available for test with build:
https://brewweb.devel.redhat.com/buildinfo?buildID=388959

Comment 5 Filip Brychta 2014-10-02 07:55:40 UTC
Verified on
Version :	
3.3.0.ER04
Build Number :	
99d2107:d7c537e


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