Feature #61903
open
pybind/mgr/volumes: add config to turn off subvolume deletion
Added by Patrick Donnelly 11 months ago.
Updated 12 days ago.
Category:
Administration/Usability
Backport:
reef,quincy,pacific
Component(FS):
mgr/volumes
Labels (FS):
task(easy), task(intern)
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.
- Tracker changed from Bug to Feature
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.
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.
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).
- Assignee deleted (
Patrick Donnelly)
- Assignee set to Rishabh Dave
Rishabh, please take this one.
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.
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.
- Status changed from New to Fix Under Review
- Target version changed from v19.0.0 to v20.0.0
Also available in: Atom
PDF