Bug 959028 - Availability TimeoutException: Unclear root cause of timeouts, should display 'root cause'
Summary: Availability TimeoutException: Unclear root cause of timeouts, should display...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Agent
Version: 4.5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: GA
: RHQ 4.10
Assignee: Elias Ross
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-03 03:04 UTC by Elias Ross
Modified: 2014-04-23 12:32 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-04-23 12:32:13 UTC
Embargoed:


Attachments (Terms of Use)
patch on master (more or less) (6.06 KB, patch)
2013-07-26 23:16 UTC, Elias Ross
no flags Details | Diff

Description Elias Ross 2013-05-03 03:04:00 UTC
Plugin container for 4.5.1

I occasionally see this for some components:

org.rhq.core.pc.inventory.TimeoutException: Call to [com....Component.getAvailability()] with args [] timed out after 5000 milliseconds - invocation thread will be interrupted.
	at org.rhq.core.pc.inventory.ResourceContainer$ResourceComponentInvocationHandler.invokeInNewThreadWithLock(ResourceContainer.java:574)
	at 

It could very well be a problem in my plugin, but rather than the stack trace of the invocation handler, can the plugin container simply display the executing thread instead? Then I can understand exactly the root cause of the thread being interrupted. (Is it trying to connect? Read data? Not doing anything?)

Alternatively, the plugin container could output a stack trace as a warning before  interrupting that thread.

Comment 1 Elias Ross 2013-06-12 18:15:19 UTC
I rolled in a fix for this for Bug 971556. If 971556 is fixed, then this is resolved as well. My fix also pretty much allows availability checking to go on for quite a while.

Comment 2 Elias Ross 2013-07-26 18:56:17 UTC
My fix for 971556 is only for availability timeouts, this fix works for other sorts of timeout issues.

Comment 3 Elias Ross 2013-07-26 23:16:25 UTC
Created attachment 778907 [details]
patch on master (more or less)

Comment 4 John Mazzitelli 2013-11-19 20:47:38 UTC
I peer reviewed Elias' patch and it looks good to me. Commited to master:

git commit to master: 82ce5f7d17b09e96bc053c19c1af5439899d8c80

Comment 5 Heiko W. Rupp 2014-04-23 12:32:13 UTC
Bulk closing of 4.10 issues.

If an issue is not solved for you, please open a new BZ (or clone the existing one) with a version designator of 4.10.


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