-
Notifications
You must be signed in to change notification settings - Fork 13
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
kola/kubeadm/tmpl: forward docker logs to journald #228
Conversation
kola/tests/kubeadm/templates.go
Outdated
mode: 0644 | ||
contents: | ||
remote: | ||
url: data:text/plain;base64,eyJsb2ctZHJpdmVyIjogImpvdXJuYWxkIn0K` |
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.
I think inline: |
should work, too.
Another alternative is data:,…
with URL encoding.
The base64 encoding is not readable for humans^^
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 I wanted to avoid URL encoding - let's give a try to inline: |
.
kola/tests/kubeadm/templates.go
Outdated
remote: | ||
url: data:text/plain;base64,eyJsb2ctZHJpdmVyIjogImpvdXJuYWxkIn0K` |
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.
remote: | |
url: data:text/plain;base64,eyJsb2ctZHJpdmVyIjogImpvdXJuYWxkIn0K` | |
inline: | | |
{"log-driver": "journald"} |
@pothos comment has been addressed - too bad it's not today we'll start to write config fully in base64 |
this can help debugging failing tests. See also: https://docs.docker.com/config/containers/logging/journald/ Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
03aed12
to
d14cf13
Compare
this can help debugging failing tests.
See also: https://docs.docker.com/config/containers/logging/journald/
Signed-off-by: Mathieu Tortuyaux mtortuyaux@microsoft.com
For ARM64, tests are failing and only indications we have are from kubelet:
How to use
kubeadm.*
test_kola_temp/.../journal.txt
:docker containers logs are now redirected to journald.