-
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
added logging traits documentation. #3231
Conversation
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.
Good work, thanks.
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.
Still some change is needed
examples/traits/logging/README.md
Outdated
|
||
Logs are enabled by default on an integration. Just run this example: | ||
|
||
$ kamel run logging.java |
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.
Logging with L uppercase...
examples/traits/logging/README.md
Outdated
|
||
$ kubectl get pods | ||
|
||
$ kubecttl logs <pod name> |
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.
Probably kubectl
examples/traits/logging/README.md
Outdated
|
||
manual log setting example using `logging trait`: | ||
|
||
$ kamel run ./logging.java --trait logging.enabled=true --trait logging.json=true --trait logging.level=info |
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.
Logging with L
examples/traits/logging/README.md
Outdated
## Using modeline | ||
An example of using a `modeline` to set the `logging traits` : | ||
|
||
$ kamel run ./loggingModeline.java |
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.
LoggingModeline with L
The logging traits needed a documentation. This fixes issue #3143
Release Note