-
Notifications
You must be signed in to change notification settings - Fork 345
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
Integration logging configuration #580
Comments
Yeah lets get this into 1.1. For centralized logging (such as FluentD, ES and Kibana) then we may need to output in a format that it can import such as json. |
Yes, we need to have an option for structured logging. |
I guess this is a good candidate when moving to quarkus as default runtime |
Hello! Is there a way to use a custom logging appender and stuff? I have my own logback.xml file which I would like to use for camel k integration. |
No, the logging engine is now the one provided by quarkus, see https://quarkus.io/guides/logging |
it supports SLF4J APIs but the configuration happens through properties as described int he logging guide (at least that is my understanding) |
@lburgazzoli thanks for the clarification! I want to try https://github.com/quarkiverse/quarkus-logging-json and its custom log handler. I am not sure how to use CDI to inject JsonProvider.. Does not seem to work for me.. Can you please advise? |
In such case you need to bring the handler as part of a custom artifact as camel-k does not expose CDI facilities (at least for now). Be aware that you need to index the artifact using Jandex or add a beans.xml to get quarkus be able to find your handler. |
I'm looking at this. |
These PRs mentioned throughout this discussion have added the support for the logging trait. As discussed w/ @astefanutti, now that the documentation is in place, we can close this. |
We could add a trait that enables to configure logging for integration, e.g. format, level. It should ideally be agnostic from the logging backend and the integration runtime.
The text was updated successfully, but these errors were encountered: