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

Liveness probe memory leak #226

Closed
juliohm1978 opened this issue Feb 9, 2021 · 4 comments
Closed

Liveness probe memory leak #226

juliohm1978 opened this issue Feb 9, 2021 · 4 comments

Comments

@juliohm1978
Copy link

What happened:

After upgrading to v0.6.0, we are seeing regular pod restarts, some of them being OOMKilled.

Looking at the pod metrics, we can see that the liveness probe in the node pod seems to be leaking memory over time.

image

Environment:

  • CSI Driver version: v0.6.0
  • Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4", GitCommit:"d360454c9bcd1634cf4cc52d1867af5491dc9c5f", GitTreeState:"archive", BuildDate:"2020-11-25T13:19:56Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.5", GitCommit:"e338cf2c6d297aa603b50ad3a301f761b4173aa6", GitTreeState:"clean", BuildDate:"2020-12-09T11:10:32Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}
  • OS (e.g. from /etc/os-release): ubuntu 18.04 LTS
  • Kernel (e.g. uname -a): Linux jhempc 5.10.13-arch1-2 #1 SMP PREEMPT Sat, 06 Feb 2021 11:07:29 +0000 x86_64 GNU/Linux
  • Install tools: Kubespray
@juliohm1978
Copy link
Author

Could this be related?

kubernetes-csi/livenessprobe#102

@juliohm1978
Copy link
Author

A memory leak appears to have been fixed in the liveness probe image 2.2.0.

kubernetes-csi/livenessprobe#94

https://github.com/kubernetes-csi/livenessprobe/blob/master/CHANGELOG/CHANGELOG-2.2.md

@andyzhangx
Copy link
Member

andyzhangx commented Feb 9, 2021

the master branch is already using v2.2.0:

- name: liveness-probe
image: k8s.gcr.io/sig-storage/livenessprobe:v2.2.0

Would you kubectl edit ds csi-driver-smb-node -n kube-system to upgrade to v2.2.0 directly in your cluster?

@juliohm1978
Copy link
Author

Just changed my values.yaml and redeployed. Will be back to confirm if this fixes it.

andyzhangx added a commit to andyzhangx/csi-driver-smb that referenced this issue Aug 11, 2023
670bb0ef1 Merge pull request kubernetes-csi#229 from marosset/fix-codespell-errors
35d5e783c Merge pull request kubernetes-csi#219 from yashsingh74/update-registry
63473cc96 Merge pull request kubernetes-csi#231 from coulof/bump-go-version-1.20.5
29a5c76c7 Merge pull request kubernetes-csi#228 from mowangdk/chore/adopt_kubernetes_recommand_labels
8dd28211b Update cloudbuild image with go 1.20.5
1df23dba6 Merge pull request kubernetes-csi#230 from msau42/prow
1f92b7e7c Add ginkgo timeout to e2e tests to help catch any stuck tests
2b8b80ead fixing some codespell errors
c10b67804 Merge pull request kubernetes-csi#227 from coulof/check-sidecar-supported-versions
72984ec0a chore: adopt kubernetes recommand label
b05553510 Header
bd0a10b65 typo
c39d73c33 Add comments
f6491af0e Script to verify EOL sidecar version
4133d1df0 Merge pull request kubernetes-csi#226 from msau42/cloudbuild
8d519d237 Pin buildkit to v0.10.6 to workaround v0.11 bug with docker manifest
6e04a0301 Merge pull request kubernetes-csi#224 from msau42/cloudbuild
26fdfffdd Update cloudbuild image
6613c3980 Merge pull request kubernetes-csi#223 from sunnylovestiramisu/update
0e7ae993d Update k8s image repo url
77e47cce8 Merge pull request kubernetes-csi#222 from xinydev/fix-dep-version
155854b09 Fix dep version mismatch
8f839056a Merge pull request kubernetes-csi#221 from sunnylovestiramisu/go-update
1d3f94dd5 Update go version to 1.20 to match k/k v1.27
901bcb5a9 Update registry k8s.gcr.io -> registry.k8s.io

git-subtree-dir: release-tools
git-subtree-split: 670bb0ef135a53be44643cc34440eff22ad3ac8c
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

No branches or pull requests

2 participants