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

[csi-cephfs] cache entries are not cleaned up when provision="false" #275

Closed
LiGeNvidia opened this issue Mar 21, 2019 · 2 comments
Closed

Comments

@LiGeNvidia
Copy link

When using csi-cephfs with provision="false", the cache entries are not being deleted (1, 2). This would cause the number of configmaps to store csi-cephfs to grow indefinitely:

> kubectl get cm | grep csi-cephfs
csi-cephfs-pvc-0037de92-45e2-11e9-a076-988609043c26   1      7d21h
csi-cephfs-pvc-155adb4f-468c-11e9-8a88-9886098cc473   1      7d
csi-cephfs-pvc-2dddceae-45e7-11e9-bbf0-988609bd0a19   1      7d20h
csi-cephfs-pvc-5bbd81db-45e3-11e9-bbf0-988609bd0a19   1      7d20h
csi-cephfs-pvc-6ac96392-45e7-11e9-8a88-9886098cc473   1      7d20h
csi-cephfs-pvc-766d74ee-45e6-11e9-8a88-9886098cc473   1      7d20h
csi-cephfs-pvc-838e24ee-4bed-11e9-8a88-9886098cc473   1      4h24m
csi-cephfs-pvc-bbd3c8df-45e8-11e9-8a88-9886098cc473   1      7d20h

1: https://github.com/ceph/ceph-csi/blob/csi-v1.0/pkg/cephfs/controllerserver.go#L131-L135
2: https://github.com/ceph/ceph-csi/blob/csi-v1.0/pkg/cephfs/controllerserver.go#L166

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Mar 22, 2019

@LiGeNvidia Thanks for pointing out the issue, would you like to send a PR to fix this one?

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jul 16, 2019

as we are moved configmap to rados omap to store volume and snapshot information, this issue is no more valid

@Madhu-1 Madhu-1 closed this as completed Jul 16, 2019
nixpanic pushed a commit to nixpanic/ceph-csi that referenced this issue Mar 27, 2024
Syncing latest changes from upstream devel for ceph-csi
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