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.

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

Jdkorb #7008

Merged
merged 41 commits into from Jan 13, 2015
Merged

Jdkorb #7008

merged 41 commits into from Jan 13, 2015

Conversation

tadamski
Copy link
Contributor

@tadamski tadamski commented Dec 1, 2014

IIOP-OpenJDK subsystem

Based on openjdk 8 orb. Orb code is added as openjdk-orb-8.0.0.Beta1 project. Orb code is an hg/git repo which can be merged with current openjdk. Idls containing orb classes used by jdkorb subsystem were added and the code is automatically generated during maven build. Openjdk-orb replaces rmiapi project. All changes from rmiapi project were merged. Rmiapi module is still used but it only exports classes form openjdk-orb.
Wildfly with jdkorb subsystem passes all integration tests, tck rmiiop tests and tck interop tests. EAP6 (jacorb) -> Wildfly (openjdk) iiop comunication is working correctly for simple tests.

@wildfly-ci
Copy link

Windows Build 691 is now running using a merge of 44c3161

@wildfly-ci
Copy link

Linux Build 5581 is now running using a merge of 44c3161

@wildfly-ci
Copy link

Linux with security manager Build 567 is now running using a merge of 44c3161

@wildfly-ci
Copy link

Linux with security manager Build 567 outcome was SUCCESS using a merge of 44c3161
Summary: Tests passed: 807, ignored: 245 Build time: 0:06:55

@wildfly-ci
Copy link

Windows Build 691 outcome was SUCCESS using a merge of 44c3161
Summary: Tests passed: 3021, ignored: 241 Build time: 0:51:28

@wildfly-ci
Copy link

Linux Build 5581 outcome was SUCCESS using a merge of 44c3161
Summary: Tests passed: 3021, ignored: 241 Build time: 0:53:29

@tadamski
Copy link
Contributor Author

tadamski commented Dec 3, 2014

Transaction integration tests pass correctly. Description of tests in JIRA: https://issues.jboss.org/browse/WFLY-1077.

private AttributeConstants() {
}

static final ModelNode DEFAULT_DISABLED_PROPERTY = new ModelNode().set("off");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if we really want to stay with the on/off boolean values used by the JacORB subsystem, as every other subsystem uses true/false instead.

@stuartwdouglas
Copy link
Contributor

I think this looks good for the most part. Have you done any TCK testing?

@tadamski
Copy link
Contributor Author

tadamski commented Dec 3, 2014

Yes. But I made a couple of changes since that so I plan to rerun TCK to check if anything has been broken. Will fix above issues and rerun TCK.

@scottmarlow
Copy link
Contributor

I think that we should run all of the EE TCK tests against this change, not just the interoperabilty tests.

@stuartwdouglas
Copy link
Contributor

Why? the interoperability tests are the only ones that use the ORB.

@tadamski
Copy link
Contributor Author

tadamski commented Dec 3, 2014

I have rerun rmiiop and interop tests - both test groups pass correctly

@scottmarlow
Copy link
Contributor

@stuartwdouglas, I was thinking that other TCK tests might be impacted in unexpected ways (e.g. JTS could be impacted). I think that the Narayana team has done some testing as well, so we might of tested enough already.

@wildfly-ci
Copy link

Windows Build 710 is now running using a merge of 8ebc4a0

@wildfly-ci
Copy link

Linux with security manager Build 586 is now running using a merge of 8ebc4a0

@wildfly-ci
Copy link

Linux Build 5600 is now running using a merge of 8ebc4a0

@wildfly-ci
Copy link

Linux with security manager Build 586 outcome was SUCCESS using a merge of 8ebc4a0
Summary: Tests passed: 806, ignored: 245 Build time: 0:07:07

@wildfly-ci
Copy link

Windows Build 710 outcome was SUCCESS using a merge of 8ebc4a0
Summary: Tests passed: 3019, ignored: 241, muted: 1 Build time: 0:53:00

@wildfly-ci
Copy link

Linux Build 5600 outcome was SUCCESS using a merge of 8ebc4a0
Summary: Tests passed: 3020, ignored: 241 Build time: 0:56:27

@wildfly-ci
Copy link

Linux with security manager Build 590 is now running using a merge of 9160e1f

@wildfly-ci
Copy link

Linux Build 5604 is now running using a merge of 9160e1f

@wildfly-ci
Copy link

Windows Build 714 is now running using a merge of 9160e1f

@wildfly-ci
Copy link

Linux Build 5754 is now running using a merge of 55b7fc1

@wildfly-ci
Copy link

Linux with security manager Build 741 is now running using a merge of 55b7fc1

@wildfly-ci
Copy link

Windows Build 865 is now running using a merge of 55b7fc1

@wildfly-ci
Copy link

Linux with security manager Build 741 outcome was SUCCESS using a merge of 55b7fc1
Summary: Tests passed: 795, ignored: 371 Build time: 0:05:39

@wildfly-ci
Copy link

Windows Build 865 outcome was SUCCESS using a merge of 55b7fc1
Summary: Tests passed: 2913, ignored: 366 Build time: 0:50:40

@wildfly-ci
Copy link

Linux Build 5754 outcome was FAILURE using a merge of 55b7fc1
Summary: Tests failed: 1 (1 new), passed: 2912, ignored: 366 Build time: 0:53:12

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.ejb.stateful.passivation.PassivationTestCase.testPassivationMaxSize: javax.ejb.EJBException: java.lang.IllegalStateException: WFLYCLEJBINF0008: Stateful session bean {[-67, 39, 104, 117, -73, -38, 77, -75, -117, -46, -91, -65, -118, -4, -126, 78]} refers to an invalid bean group 1f51b7af-f2f3-498f-a293-ab2bb8d3c477
    at org.wildfly.clustering.ejb.infinispan.bean.InfinispanBeanFactory.createBean(InfinispanBeanFactory.java:75)
    at org.wildfly.clustering.ejb.infinispan.InfinispanBeanManager.findBean(InfinispanBeanManager.java:266)

n1hility added a commit that referenced this pull request Jan 13, 2015
@n1hility n1hility merged commit 6f75025 into wildfly:master Jan 13, 2015
@tadamski tadamski deleted the jdkorb branch May 21, 2015 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants