Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WFLY-4491 If security subsystem is excluded don't install security inter... #7321

Merged
merged 1 commit into from Apr 21, 2015
Merged

WFLY-4491 If security subsystem is excluded don't install security inter... #7321

merged 1 commit into from Apr 21, 2015

Conversation

stuartwdouglas
Copy link
Contributor

...ceptors / handlers

@wildfly-ci
Copy link

Linux with security manager Build 1167 is now running using a merge of ac35916

@wildfly-ci
Copy link

Windows Build 1303 is now running using a merge of ac35916

@wildfly-ci
Copy link

Linux Build 6191 is now running using a merge of ac35916

@wildfly-ci
Copy link

Linux with security manager Build 1167 outcome was SUCCESS using a merge of ac35916
Summary: Tests passed: 819, ignored: 356 Build time: 0:06:25

@wildfly-ci
Copy link

Linux Build 6191 outcome was SUCCESS using a merge of ac35916
Summary: Tests passed: 2838, ignored: 416 Build time: 0:50:39

@wildfly-ci
Copy link

Windows Build 1303 outcome was FAILURE using a merge of ac35916
Summary: Tests failed: 1 (1 new), passed: 2837, ignored: 416 Build time: 0:53:24

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.ejb.mdb.ejb2x.MDB20TopicTestCase.testEjb20TopicMDBs: <no details avaliable>

@darranl
Copy link
Contributor

darranl commented Apr 9, 2015

Out of interest does this pass the TCK - are we sure the anonymous identity requirements are still met Ok?

@stuartwdouglas
Copy link
Contributor Author

This is not really intended to pass the TCK (which it won't, because the TCK requires the security subsystem). Basically if you are not using any security features in your deployment then you can exclude the subsystem, and get a performance boost.

@n1hility
Copy link
Member

retest this please

@wildfly-ci
Copy link

Windows Build 1380 is now running using a merge of ac35916

@wildfly-ci
Copy link

Linux with security manager Build 1238 is now running using a merge of ac35916

@wildfly-ci
Copy link

Linux Build 6273 is now running using a merge of ac35916

@wildfly-ci
Copy link

Linux with security manager Build 1238 outcome was SUCCESS using a merge of ac35916
Summary: Tests passed: 823, ignored: 356 Build time: 0:04:56

@wildfly-ci
Copy link

Windows Build 1380 outcome was SUCCESS using a merge of ac35916
Summary: Tests passed: 2848, ignored: 412 Build time: 0:49:03

@wildfly-ci
Copy link

Linux Build 6273 outcome was SUCCESS using a merge of ac35916
Summary: Tests passed: 2848, ignored: 412 Build time: 0:49:18

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Apr 21, 2015
bstansberry added a commit that referenced this pull request Apr 21, 2015
@bstansberry
Merge pull request #7321 from stuartwdouglas/WFLY-4491
b8aa5bd 
WFLY-4491 If security subsystem is excluded don't install security inter...
@bstansberry bstansberry merged commit b8aa5bd into wildfly:master Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready-for-merge Only for use by those with merge permissions!
Projects
None yet
Milestone
No milestone
5 participants
@stuartwdouglas @wildfly-ci @darranl @n1hility @bstansberry