Skip to content

Comments

CAMEL-14321: camel-osgi-activator support pre-startup routebuilders#3435

Closed
bobpaulin wants to merge 2 commits intoapache:masterfrom
bobpaulin:feature/camel-osgi-activator-pre-startup-support
Closed

CAMEL-14321: camel-osgi-activator support pre-startup routebuilders#3435
bobpaulin wants to merge 2 commits intoapache:masterfrom
bobpaulin:feature/camel-osgi-activator-pre-startup-support

Conversation

@bobpaulin
Copy link
Contributor

No description provided.

@oscerd
Copy link
Contributor

oscerd commented Dec 23, 2019

Merged on master. @bobpaulin thanks, when you have time, can you double check the IT test with JDK11? it is failing consistently on CI.

@oscerd oscerd closed this Dec 23, 2019
@bobpaulin
Copy link
Contributor Author

bobpaulin commented Dec 23, 2019

Hi @oscerd I'm able to reproduce locally on Java 11. Looks like it's something due to the removal of the JavaEE jars occurred between 8 and 11. Will need to tweak the karaf settings to make sure those Jars are available to the runtime. I'll create a JIRA for tracking this.

@bobpaulin
Copy link
Contributor Author

bobpaulin commented Dec 23, 2019

Looks like the issue is actually PAX-EXAM ops4j/org.ops4j.pax.exam#87 . Once this is fixed in the karaf test container startup these tests pass as expected. @jbonofre @oscerd

@oscerd
Copy link
Contributor

oscerd commented Dec 23, 2019

Thanks for the heads up :-)

@jbonofre
Copy link
Member

@bobpaulin it works fine with Java11 if you extend KarafTestSupport. I will fix that, just at least adding the starting options for Java11: https://github.com/apache/karaf/blob/master/itests/common/src/main/java/org/apache/karaf/itests/KarafTestSupport.java#L184

I will work on OSGi part (as I think some cleanup would be welcome).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants