Skip to content
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

Fix systemd cpu/memory problems on RedHat #230

Merged
merged 1 commit into from
Jan 4, 2019

Conversation

jorhett
Copy link
Contributor

@jorhett jorhett commented Jan 4, 2019

This solves the following problems on RedHat:

Dec 5 12:42:03 kubelet: E1205 12:42:03.527763 23059 summary.go:92] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Dec 5 12:42:03 kubelet: E1205 12:42:03.527796 23059 summary.go:92] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"
Dec 5 12:42:03 kubelet: W1205 12:42:03.527852 23059 helpers.go:847] eviction manager: no observation found for eviction signal allocatableNodeFs.available

The fix is well known, but for some reason hasn't been added to the kubelet package:

All kubernetes package installations have added this change to resolve the problem:

I can't fathom why this isn't part of the kubelet package, but it seems kubernetes has pushed this out to the distribution to solve and since Puppet is providing the implementation we have to do it.

@davejrt
Copy link
Contributor

davejrt commented Jan 4, 2019

LGTM

@davejrt davejrt merged commit 9443711 into puppetlabs:master Jan 4, 2019
@davejrt davejrt added this to the 3.2.0 milestone Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants