Bug 961470

Summary: Regression: CDI portable extensions with missing dependencies fail to deploy
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Ron Šmeral <rsmeral>
Component: CDI/WeldAssignee: Stuart Douglas <sdouglas>
Status: CLOSED CURRENTRELEASE QA Contact: Ron Šmeral <rsmeral>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: amelicha, cdewolf, istudens, maschmid, myarboro
Target Milestone: ER1Keywords: Regression
Target Release: EAP 6.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-15 16:20:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Server log
none
Reproducer none

Description Ron Šmeral 2013-05-09 16:50:36 UTC 
Created attachment 745732 [details]
Server log

Description of problem:
CDI portable extensions fail to deploy if the extension class is invalid -- has a dependency on a class unavailable to the classloader.
Note: In ER5, the extension would deploy with just a warning from Weld.
Note: This commit https://github.com/weld/core/commit/34bcce3f does not fix the extension deployment error.
The problem seems to be in the AS/Weld integration.
Possible workaround is to catch the LinkageError in org.jboss.as.weld.deployment.processors.WeldPortableExtensionProcessor, as in https://github.com/rsmeral/jboss-as/commit/014c5bbb.

Version-Release number of selected component (if applicable):
6.1.0.ER6

How reproducible:
Deploy the attached reproducer.
  
Actual results:
The deployment fails with an error (LinkageError).

Expected results:
The archive deploys with a warning.
Comment 1 Ron Šmeral 2013-05-09 16:51:28 UTC 
Comment on attachment 745732 [details]
Server log

First AS run: ER7
Second run: ER7 patched: https://github.com/rsmeral/jboss-as/commit/014c5bbb
Comment 2 Ron Šmeral 2013-05-09 16:55:50 UTC 
Created attachment 745734 [details]
Reproducer
Comment 5 Stuart Douglas 2013-05-10 01:52:12 UTC 
Pull Request:

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

Upstream PR:

https://github.com/wildfly/wildfly/pull/4489
Comment 9 Ivo Studensky 2013-08-15 08:38:37 UTC 
This one was missed in 6.1.1 planning, thus moving it to 6.2.0.
Comment 10 Ron Šmeral 2013-09-11 10:27:40 UTC 
Verified on 6.2.0.ER1.