-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
CRI Logging Docs & Spec Inaccurate Example for Partial Logs #4643
Comments
/sig node |
Is the desired intent here to fix the documentation to better reflect the reality? |
@haircommander Yea, the docs need to be updated to reflect clearly and specifically how it actually works. I could open a PR to edit them if you link me the repo... |
so that repo is archived and I find it unlikely it will be updated. You could add a blurb to https://kubernetes.io/docs/concepts/cluster-administration/logging/ describing the CRI format and maybe adding a note that serves as an errata to the archived repo? |
@haircommander you mean this repo right? https://github.com/kubernetes/website I will submit a PR. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
Enhancement Description
Hello, I apologize if this is not the right repo. I'm a maintainer of the popular k8s log collector Fluent BIt.
The spec for the CRI log format could be improved.
https://github.com/kubernetes/design-proposals-archive/blob/main/node/kubelet-cri-logging.md
Specifically, this example log file is incorrect and can not be parsed:
The reason is that the " Last line of the log entry 2" is marked as stderr, but the previous lines were stdout. This means it must be a separate partial message. Also, i believe the log time stamp should be the same for all partial messages.
The spec also lacks information and examples on whether or not partial lines from stdout and stderr can be interleaved.
Please see my Fluent Bit example/troubleshooting guidance here which shows how an example k8s log file with partial log messages is read and recombined: https://github.com/aws/aws-for-fluent-bit/pull/823/files
The text was updated successfully, but these errors were encountered: