-
Notifications
You must be signed in to change notification settings - Fork 188
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(camel-k): add options to override some aspects of a route #5792
Conversation
@davsclaus mind having a look ? |
9e12430
to
4809f76
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
model hacker ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from my comment about the sources location it LGTM.
integration-tests/camel-k-runtime-model-reifier/src/test/resources/conf.properties
Outdated
Show resolved
Hide resolved
"camel.k.routes.overrides[4].input.with", "direct:r5invalid", | ||
"camel.k.routes.overrides[5].id", "r5", | ||
"camel.k.routes.overrides[5].input.with", "direct:r5override", | ||
"camel.k.sources[0].location", "file:src/test/resources/routes.yaml", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file:src/test/resources/routes.yaml
wont work when the test is run in the Quarkus Platform since the test is run from the test-jar. Better to copy any required resources into target
or a temporary location.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me fix that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved to classpath reolver, would that work on Quarkus Platform @jamesnetherton ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it should be ok
4809f76
to
f0ba93e
Compare
I guess we'd want this backported to 3.8.x? |
if that is not an issue, yes |
f0ba93e
to
25ee0ee
Compare
@jamesnetherton should I backport ? |
oh you did it already |
No description provided.