|Summary:||[GSS] (6.2.x) Logging deployment unit processor (DUP) leaking class loaders|
|Product:||[JBoss] JBoss Enterprise Application Platform 6||Reporter:||James Perkins <jperkins>|
|Component:||Logging||Assignee:||James Perkins <jperkins>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Nikoleta Hlavickova <nziakova>|
|Severity:||high||Docs Contact:||Russell Dickenson <rdickens>|
|Version:||6.2.0||CC:||cdewolf, jperkins, kkhan, klape, myarboro, smumford|
|Target Release:||EAP 6.2.1|
|Fixed In Version:||Doc Type:||Bug Fix|
In previous versions of JBoss EAP 6, when per-deployment logging or logging-profiles were used on a deployment, the undeploy was not removing the class loaders from the LogContextSelector. This was because the ClassLoaderLogContextSelector used the call stack to determine the LogContext. In this release, the correct LogContext is attached to the DeploymentUnit. During the undeploy the LogContext is now removed from the selector using the LogContext attached to the deployment rather than relying on the class loader of the deployment. Class loaders are now removed as expected on undeploy.
|Last Closed:||2014-02-24 20:15:58 UTC||Type:||Bug|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Cloudforms Team:||---||Target Upstream Version:|
|Bug Depends On:||1038305|
Description James Perkins 2014-01-06 23:28:04 UTC
+++ This bug was initially created as a clone of Bug #1038305 +++ When per-deployment logging or logging-profiles are used on a deployment the undeploy is not removing the class loaders from the LogContextSelector. The error appeared when the switch was made to use the ClassLoaderLogContextSelector which uses the call stack to determine the LogContext. The commit hash is 526c2ae7a4194dc6b787c32c5218cea003276900 that introduced the error. The bug appears in 6.1.x and 6.2.x.
Comment 3 Scott Mumford 2014-01-21 03:40:21 UTC
Marking for inclusion in 6.2.1 release notes.
Comment 4 Scott Mumford 2014-01-24 01:20:09 UTC
Set 'Version' field to ensure this ticket appears in documentation effort dashboards.
Comment 5 Scott Mumford 2014-01-24 02:25:53 UTC
Hi James. Can I ask you to supply a 'fix' for this issue in the draft release note in the Doc Text field above please? I can't see any reference as to how this was resolved. All help is appreciated.
Comment 6 James Perkins 2014-01-24 17:00:42 UTC
Updated, hopefully it makes sense.
Comment 7 Scott Mumford 2014-01-27 22:49:41 UTC
Thanks for the content James. Formatted release note text for publican and marked for inclusion in the final document.
Comment 8 Nikoleta Hlavickova 2014-01-29 07:48:18 UTC
Verified with 6.2.1.CP.CR1-patch.