Skip to content

Allow ReclaimSpace to be disabled from StorageClasses#1020

Open
black-dragon74 wants to merge 3 commits intocsi-addons:mainfrom
black-dragon74:enable-sc-rs
Open

Allow ReclaimSpace to be disabled from StorageClasses#1020
black-dragon74 wants to merge 3 commits intocsi-addons:mainfrom
black-dragon74:enable-sc-rs

Conversation

@black-dragon74
Copy link
Copy Markdown
Member

@black-dragon74 black-dragon74 commented Apr 14, 2026

This patch allows a user to disable reclaimspace from the annotations
on the StorageClass.

This is required for RWX block mode volumes used in VMs (to prevent sparsify op from blacklisting the actual client) .

This patch allows a user to disable reclaimspace from the annotations
on the StorageClass.

This is required for RWX block mode volumes used in VMs.

Signed-off-by: Niraj Yadav <niryadav@redhat.com>
Signed-off-by: Niraj Yadav <niryadav@redhat.com>
Copy link
Copy Markdown
Member

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this makes sense. Only need to add some user facing documentation for the new annotation.

Signed-off-by: Niraj Yadav <niryadav@redhat.com>
@black-dragon74
Copy link
Copy Markdown
Member Author

I guess this makes sense. Only need to add some user facing documentation for the new annotation.

Done :)

Comment thread docs/reclaimspace.md
kubectl annotate storageclass <SC_NAME> "reclaimspace.csiaddons.openshift.io/enable=false"
```

This action will disable reclaim space across all PVCs in that StorageClass and remove any existing `ReclaimSpaceCronJob` CRs for PVCs within it.
Copy link
Copy Markdown
Member

@nixpanic nixpanic Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what happens when there is an annotation on a Namespace?

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