Bug 978332 - DeployableContainer kills the managed server instead of regular shutdown in stop() on Windows
Summary: DeployableContainer kills the managed server instead of regular shutdown in s...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Testsuite
Version: 6.1.0
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ER4
: EAP 6.1.1
Assignee: Ivo Studensky
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-26 11:33 UTC by Ivo Studensky
Modified: 2013-10-15 17:20 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-09-16 20:24:33 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker WFLY-126 0 Critical Closed DeployableContainer kills the managed server instead of regular shutdown in stop() on Windows 2013-10-15 17:20:03 UTC

Description Ivo Studensky 2013-06-26 11:33:15 UTC
Description of problem:
Backport of WFLY-126 to the EAP 6.1.x branch.

Upstream's description: 
This might be due to historical reasons for the TS to run faster, but this messes up clustering tests since shutdown is part of the test.

The problem is that Managed*DeployableContainer kills the managed server instead of regular shutdown in stop() on Windows. ARQ's stop is supposed to shutdown the server 'nicely', for abrupt termination there is jvmKill().

This was probably implemented like this incorrectly assuming that on windows, the process is sent SIGTERM like on *NIX-based OSes but there is no such thing on windows and the process is terminated immediately.

This shows up only in clustered tests, compared to other tests which dont rely on the shutdown sequence to be completed, but clustering testsuite relies on shutdown to disconnect the channel and leave the cluster without FD having to kick in.

Comment 1 Ivo Studensky 2013-06-26 12:01:36 UTC
Created a PR for this:

https://github.com/jbossas/jboss-eap/pull/202

Comment 2 Rostislav Svoboda 2013-06-27 12:00:12 UTC
TS improvement is welcome.

Comment 3 Richard Janík 2013-08-12 12:16:44 UTC
I do indeed see improvements in the clustering testsuite on Windows.

Comment 5 JBoss JIRA Server 2013-10-15 17:20:04 UTC
Radoslav Husar <rhusar> updated the status of jira WFLY-126 to Closed


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