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
feat: add initial support for camel-k #1097
Conversation
core/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelAutoConfiguration.java
Outdated
Show resolved
Hide resolved
...-xml/src/main/java/org/apache/camel/spring/boot/xml/SpringBootXmlCamelContextConfigurer.java
Outdated
Show resolved
Hide resolved
core/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelAutoConfiguration.java
Outdated
Show resolved
Hide resolved
Looks like there are some issues related to openapi v2 rmeoval
|
Are you on old code ? |
No, rebased on main, I see that OpenApi v2 is hardcoded in some of the tests https://github.com/apache/camel-spring-boot/blob/main/components-starter/camel-openapi-java-starter/src/test/java/org/apache/camel/openapi/RestOpenApiModelApiSecurityRequirementsTest.java#L94 |
Ah sorry its those darn test copy from core, that have little value and get out of date |
fixing this now |
pushed fix |
ok so now failures are because there is no yet a -SNAPSHOT ready |
core/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelAutoConfiguration.java
Outdated
Show resolved
Hide resolved
...-xml/src/main/java/org/apache/camel/spring/boot/xml/SpringBootXmlCamelContextConfigurer.java
Outdated
Show resolved
Hide resolved
There was some leftover in the camel-spring-boot core module. If this gets reverted then this PR is only adding a new camel-k module, and then its perfect fine to merged. |
Requires apache/camel#13376