Skip to content

Summary#7243

Merged
davsclaus merged 9 commits into
mainfrom
summary
Mar 22, 2022
Merged

Summary#7243
davsclaus merged 9 commits into
mainfrom
summary

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

No description provided.

@davsclaus
Copy link
Copy Markdown
Contributor Author

davsclaus commented Mar 21, 2022

jbang run -Dcamel.jbang.version=3.16.0-SNAPSHOT -Dcamel.main.name=Killroy camel@apache/camel run chuck-out.yaml myapp.properties

Screenshot 2022-03-21 at 17 48 18

@davsclaus
Copy link
Copy Markdown
Contributor Author

You can now see from where all these configured options is coming from, as this can be "hard to guess" when you have many ways to configure this with ENV/SYS/properties/yaml/code and whatnot

@orpiske
Copy link
Copy Markdown
Contributor

orpiske commented Mar 22, 2022

This looks nice!

@davsclaus
Copy link
Copy Markdown
Contributor Author

The order may seem random, but its the order they are configured .. if we sort them a..z then we may loose a little bit of detail that can help the end user to trouble shoot.

@davsclaus davsclaus marked this pull request as ready for review March 22, 2022 11:09
Comment thread core/camel-api/src/main/java/org/apache/camel/spi/PropertiesComponent.java Outdated
davsclaus and others added 3 commits March 22, 2022 12:54
…mponent.java

Co-authored-by: Nicolas Filotto <essobedo@users.noreply.github.com>
@davsclaus davsclaus merged commit 2f8212d into main Mar 22, 2022
@davsclaus davsclaus deleted the summary branch March 22, 2022 12:57
vpaturet pushed a commit to vpaturet/camel that referenced this pull request Mar 29, 2022
CAMEL-17831: camel-main - Auto configuration summary (show from where the option was taken)
CAMEL-17837 camel-main - Loading properties from multiple PropertiesSource - Keys with difference case styles

* Update core/camel-api/src/main/java/org/apache/camel/spi/PropertiesComponent.java
Co-authored-by: Nicolas Filotto <essobedo@users.noreply.github.com>
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.

4 participants