You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/07/30 13:16:17 UTC
[GitHub] [camel-k] christophd opened a new issue #2538: CLI parameter does not overwrite application.properties
christophd opened a new issue #2538:
URL: https://github.com/apache/camel-k/issues/2538
Some Camel-K integration uses a modeline to define a property file like this
```
// camel-k: language=java property=file:application.properties
```
Properties defined as CLI parameters on `kamel run` should be able to overwrite these properties
```
kamel run Foo.java -p foo.bar=something
```
So `foo.bar` property in application.properties should be overwritten. This is not working as expected with Camel-K 1.5.0.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez closed issue #2538: CLI parameter does not overwrite application.properties
Posted by GitBox <gi...@apache.org>.
squakez closed issue #2538:
URL: https://github.com/apache/camel-k/issues/2538
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] squakez commented on issue #2538: CLI parameter does not overwrite application.properties
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #2538:
URL: https://github.com/apache/camel-k/issues/2538#issuecomment-896895757
I've analyzed it and it's not really a bug as the modeline just convert the configuration into a full `kamel run` command. The real problem is that we don't consider precedence of a single property over a file properties. I think that by including it we will also fix the problem described in this issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org