Bug 982804 - AS7 reload behavior includes hardcoded timeout of 20 seconds
Summary: AS7 reload behavior includes hardcoded timeout of 20 seconds
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Operations Network
Classification: JBoss
Component: Plugin -- JBoss EAP 6
Version: JON 3.1.2
Hardware: All
OS: All
unspecified
high
Target Milestone: ER01
: JON 3.2.0
Assignee: Thomas Segismont
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-09 23:14 UTC by Marc Shirley
Modified: 2018-12-02 16:54 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 966777 0 unspecified CLOSED EAP 6 plug-in is using a hard-coded operation timeout for start and stop instead of using the operation timeout or agent... 2021-02-22 00:41:40 UTC
Red Hat Knowledge Base (Solution) 420483 0 None None None Never

Internal Links: 966777

Description Marc Shirley 2013-07-09 23:14:40 UTC
Description of problem:
AS7 reload operation fails if the management interface takes longer than 20 seconds to come back up after the reload due to a hardcoded timeout in org/rhq/modules/plugins/jbossas7/StandaloneASComponent.java in the waitUntilReloaded() method.

Version-Release number of selected component (if applicable):
JBoss ON 3.1.2


Steps to Reproduce:
Issue will present itself only if there is a delay in the startup that prevents the management interface from coming up in less than 20 seconds.  In that case, the issue can be reproduced by issuing the "Reload" operation, waiting 20 seconds, and observing the failure of the operation in the operation history.

Actual results:
Reloads that take longer than 20 seconds are reported as failures.

Expected results:
Reload operation should report the correct result.  This should most likely be through a configurable timeout or a repurpose of the  BaseServerComponent.waitForServerToStart() method.

Additional info:
This is similar to the issue reported in BZ 966777.

Comment 1 Thomas Segismont 2013-07-16 13:12:28 UTC
Fixed in master

commit 6ad3c87c08631a00cb960a4ffa4c4929329aa6d8
Author: Thomas Segismont <tsegismo>
Date:   Tue Jul 16 13:42:50 2013 +0200

Comment 2 Larry O'Leary 2013-09-06 14:30:30 UTC
As this is MODIFIED or ON_QA, setting milestone to ER1.

Comment 3 Libor Zoubek 2013-11-15 10:22:15 UTC
verified on ER5. I was not able to reproduce when EAP6 takes more than 20s to reload, although fix oboviously removes hardcoded timeout and relies on general operation timeout .


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