Bug 769861

Summary: [as4 and as5 plugins] add maxThreads trait to JBossAS Embedded JBossWeb Connector Resources
Product: [Other] RHQ Project Reporter: Mohamed Hamza Ben Mansour <mbenmans>
Component: PluginsAssignee: Larry O'Leary <loleary>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: low Docs Contact:
Priority: high    
Version: 3.0.1CC: ccrouch, dsteigne, hrupp, ian.springer, loleary, sappleto
Target Milestone: ---   
Target Release: JON 3.1.0, RHQ 4.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 794765 (view as bug list) Environment:
Last Closed: 2013-03-14 15:06:26 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 782579, 794765    
Attachments:
Description Flags
the patch to add the feature (against 3.0.1.GA) none

Description Mohamed Hamza Ben Mansour 2011-12-22 15:05:51 UTC
Description of problem:

The maxThreads information for tomcat connectors is present in the case of a standalone Tomcat server. However it is not fetched for the JBoss embedded Tomcat servers. This information can be very useful particularly for my customer EUROPCAR. Indeed, he wants to have different groups of connector thread pools grouped by their respective maxthreads. 

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

How reproducible:
Always

Steps to Reproduce:
1. modify the rhq-plugin.xml for the jopr-jboss-as-plugin and add for the Connector service :
<!-- jboss.web:name=http-0.0.0.0-8080,type=ThreadPool:maxThreads -->
            <metric property="jboss.web:name=%schema%%dash%%address%-%port%,type=ThreadPool:maxThreads"
                    displayName="Threadpool Max Threads."
                    defaultOn="true" category="utilization" displayType="detail" dataType="trait" measurementType="dynamic"/>

2. Perform a plugin update for agents
3.
  
Actual results:
"Not found" displayed in the Trait sub-tab

Expected results:
The actual maxThreads value

Additional info:
Attached the patch against the 3.0.1.GA wich adds the information both for the AS 5 and AS plugins

Comment 1 Mohamed Hamza Ben Mansour 2011-12-22 15:07:07 UTC
Created attachment 549207 [details]
the patch to add the feature (against 3.0.1.GA)

Comment 3 Larry O'Leary 2012-02-02 19:04:25 UTC
*** Bug 786165 has been marked as a duplicate of this bug. ***

Comment 4 Charles Crouch 2012-02-15 13:18:43 UTC
Lets see if we can use the patch Mohamed has provided, assuming he can confirm he has signed the Fedora CLA

Comment 5 Shaun Appleton 2012-02-15 13:46:13 UTC
As Mohamed is a Red Hat employee I think he has signed it by default.

Comment 6 Charles Crouch 2012-02-16 15:45:29 UTC
(In reply to comment #5)
> As Mohamed is a Red Hat employee I think he has signed it by default.

My bad, I should have checked his email address. Thanks for the pointer Shaun.

Lets see if this patch applies cleanly in master.
This feature should also be added to the as7 plugin for consistency.

Comment 7 Ian Springer 2012-02-17 15:09:41 UTC
Merged patch into as4 and as5 plugins:

http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=bf2b0ec

I will make a clone of this BZ to track the equivalent enhancement to the as7 plugin.

Comment 9 dsteigne 2012-05-14 14:18:39 UTC
Installed EAP 5.1 server and verified Trait was now available