Feature #61903
openpybind/mgr/volumes: add config to turn off subvolume deletion
0%
Description
Sometimes we want to be able to turn off asynchronous subvolume deletion during cluster recovery scenarios. Add a config to turn the threads off.
Updated by Venky Shankar 10 months ago
Patrick Donnelly wrote:
Sometimes we want to be able to turn off asynchronous subvolume deletion during cluster recovery scenarios. Add a config to turn the threads off.
Maybe add functionality to scale down the purge threads and then setting it to 0 would disable async purge.
Also, how about doing the same for async threads? Those are heavy on the file system too.
Updated by Patrick Donnelly 10 months ago
Venky Shankar wrote:
Patrick Donnelly wrote:
Sometimes we want to be able to turn off asynchronous subvolume deletion during cluster recovery scenarios. Add a config to turn the threads off.
Maybe add functionality to scale down the purge threads and then setting it to 0 would disable async purge.
I'm not a fan of doing that because "restoring" the setting to a sane default is left to the admin who doesn't know better.
Also, how about doing the same for async threads? Those are heavy on the file system too.
You mean the cloner threads? i thought we already have a config for that.
Updated by Venky Shankar 9 months ago
Patrick Donnelly wrote:
Venky Shankar wrote:
Patrick Donnelly wrote:
Sometimes we want to be able to turn off asynchronous subvolume deletion during cluster recovery scenarios. Add a config to turn the threads off.
Maybe add functionality to scale down the purge threads and then setting it to 0 would disable async purge.
I'm not a fan of doing that because "restoring" the setting to a sane default is left to the admin who doesn't know better.
Also, how about doing the same for async threads? Those are heavy on the file system too.
You mean the cloner threads? i thought we already have a config for that.
I meant supporting turning it off (by setting it to zero).
Updated by Venky Shankar 9 months ago
From GChat
Rishabh Dave, 1:40 PM
https://tracker.ceph.com/issues/61903
Should I add an option to turn off threading or add an option to allow specifying number of thread user wants?
Right - an option that disables async subvolume deletion. As Patrick mentioned in note-3, there isn't a need to allow scaling the number of purge threads.
Updated by Venky Shankar 9 months ago
Venky Shankar wrote:
From GChat
Rishabh Dave, 1:40 PM
https://tracker.ceph.com/issues/61903
Should I add an option to turn off threading or add an option to allow specifying number of thread user wants?Right - an option that disables async subvolume deletion. As Patrick mentioned in note-3, there isn't a need to allow scaling the number of purge threads.
Rishabh, another point to add to this - One the operator disables async subvolume deletions, there needs to be a way for the operator to check if the purge threads have backed out performing the deletes and nothing related to async deletions are in-progress. This is required IMO as disabling async deletions is essentially asking the purge threads to voluntarily back-out doing purges.
Updated by Rishabh Dave 4 months ago
- Status changed from New to Fix Under Review
Updated by Patrick Donnelly about 23 hours ago
Where is the fix that is under review? @Rishabh Dave
Updated by Patrick Donnelly about 23 hours ago
- Target version changed from v19.0.0 to v20.0.0