Ceph » CephFS

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).

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.

Where is the fix that is under review? @Rishabh Dave