-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Continue streaming kubelet logs when runtime is unavailable #124025
base: master
Are you sure you want to change the base?
Continue streaming kubelet logs when runtime is unavailable #124025
Conversation
PTAL @kubernetes/sig-node-pr-reviews |
75f259d
to
ba5dc44
Compare
Is this patch just related to kubelet logs? |
I researched this. The code that @saschagrunert added is only used in https://github.com/saschagrunert/kubernetes/blob/7ea3d0245a63fbbba698f1cb939831fe8143db3e/pkg/kubelet/kuberuntime/logs/logs.go#L442 So this change is actually only limited to logs. |
/triage accepted |
/lgtm |
LGTM label has been added. Git tree hash: 6224fb14640c25fc82923a9615cdac751e487253
|
ba5dc44
to
e2db70f
Compare
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@kubernetes/sig-node-pr-reviews PTAL |
e2db70f
to
3011b84
Compare
Container runtimes are able to run existing containers even when their main CRI server is not available for any reason. The call to the container status RPC happens quite frequently during log parsing, means that a single CRI interruption will also abort streaming the logs. We now check that specific use case and continue following the log streaming if the CRI is unavailable. We still abort the streaming accordingly if the CRI comes back and the container status reports that the workload has exited. Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
3011b84
to
ff95ae0
Compare
Rebased, @kubernetes/sig-node-pr-reviews PTAL |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Container runtimes are able to run existing containers even when their main CRI server is not available for any reason. The call to the container status RPC happens quite frequently during log parsing, means that a single CRI interruption will also abort streaming the logs.
We now check that specific use case and continue following the log streaming if the CRI is unavailable. We still abort the streaming accordingly if the CRI comes back and the container status reports that the workload has exited.
Which issue(s) this PR fixes:
Refers to cri-o/cri-o#7826
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: