Bug 1297877

Summary: [GSS](6.4.z) ServletContainerInitializers from jboss-deployment-structure.xml module dependencies are not invoked
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Aaron Ogburn <aogburn>
Component: WebAssignee: Aaron Ogburn <aogburn>
Status: CLOSED CURRENTRELEASE QA Contact: Radim Hatlapatka <rhatlapa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.4.6CC: bmaxwell, chaowan, dpospisi, jtruhlar, mcada, rhatlapa, rmaucher
Target Milestone: CR1   
Target Release: EAP 6.4.7   
Hardware: Unspecified   
OS: Unspecified   
URL: https://github.com/jbossas/jboss-eap/pull/2718
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-17 11:57:01 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:
Bug Depends On:    
Bug Blocks: 1279552    
Attachments:
Description Flags
WFLY5962test.zip none

Description Aaron Ogburn 2016-01-12 16:57:18 UTC 
Description of problem:

If a module includes a ServletContainerInitializer, that ServletContainerInitializer is not invoked for any applications that include the module in its jboss-deployment-structure.xml module dependencies. It looks like the ServletContainerInitializerDeploymentProcessor only checks server provided dependencies.

How reproducible:

Always.

Steps to Reproduce:
1. Add "com/redhat/testSCI" contents to $EAP_HOME/modules/system/layers/base
2. Deploy SCItest.war
3. This should produce a message like "Initializer invoked for ...", indicating the SCI is invoked.  But it won't occur.

Actual results:

SCI is not invoked when included from a module.


Expected results:

SCI is invoked when included from a module.

Additional info:
Comment 1 Aaron Ogburn 2016-01-12 16:59:58 UTC 
Created attachment 1114043 [details]
WFLY5962test.zip
Comment 2 Aaron Ogburn 2016-01-12 19:01:26 UTC 
PR: https://github.com/jbossas/jboss-eap/pull/2682
Comment 3 JBoss JIRA Server 2016-01-31 03:49:22 UTC 
Stuart Douglas <stuart.w.douglas> updated the status of jira JBEAP-2744 to Resolved
Comment 6 Dominik Pospisil 2016-02-22 09:29:36 UTC 
Dev acking, PR ready, merged upstream.
Comment 7 Michael Cada 2016-03-01 14:24:54 UTC 
Verified on EAP 6.4.7.CP.CR2
Comment 8 JBoss JIRA Server 2016-06-14 11:36:46 UTC 
Jiri Pallich <jpallich> updated the status of jira JBEAP-2744 to Closed
Comment 9 Petr Penicka 2017-01-17 11:57:01 UTC 
Retroactively bulk-closing issues from released EAP 6.4 cumulative patches.