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

build: fix kubelet issue#60987 #1134

Closed
wants to merge 1 commit into from
Closed

Conversation

chengyu-l
Copy link

When a Pod which has used PVC is deleted from kubuernetes cluster
we can get a error message repeatedly(Orphaned pod "1f415e-...-4b38"
found, but volume paths are still present on disk) from kubelet log.
this error message is caused by kubelet because it cannot clean the
parent path of targetPath when Pod is deleted from cluster
(targetPath like:
/var/lib/kubelet/pods/9cda187a-7fb3-11ea-80b3-246e968d4b38/volumes/kubernetes.io~csi/pvc-9ae9405c-7fb3-11ea-80b3-246e968d4b38/mount
). Ceph CSI can remove the parent path of targetPath first instead of
waiting kubelet to fix this issue.

Signed-off-by: chengyu-l chengyu_l@126.com

When a Pod which has used PVC is deleted from kubuernetes cluster
we can get a error message repeatedly(Orphaned pod "1f415e-...-4b38"
found, but volume paths are still present on disk) from kubelet log.
this error message is caused by kubelet because it cannot clean the
parent path of targetPath when Pod is deleted from cluster
(targetPath like:
/var/lib/kubelet/pods/9cda187a-7fb3-11ea-80b3-246e968d4b38/volumes/kubernetes.io~csi/pvc-9ae9405c-7fb3-11ea-80b3-246e968d4b38/mount
). Ceph CSI can remove the parent path of targetPath first instead of
waiting kubelet to fix this issue.

Signed-off-by: chengyu-l <chengyu_l@126.com>
@humblec humblec added the DNM DO NOT MERGE label Jun 5, 2020
@humblec
Copy link
Collaborator

humblec commented Jun 5, 2020

There are few things which I would like to discuss here, so adding DNM for now.

@chengyu-l
Copy link
Author

is this method work? @humblec

@chengyu-l chengyu-l closed this Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM DO NOT MERGE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants