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

sanoid #912: sanoid --prune-snapshots performance boost by removing u… #915

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sdettmer
Copy link

…nneeded iszfsbusy()

for #912

Option 3) dev/sde/remove-force-prune - If we think the whole functionality is not needed, best would to remove it and keep things more clean. Maybe this could be done in a later version, when experiences shows, that no one ever used --no-force-prune or --double-check-prune. Doing this later probably is too much, but I work in an industry with 20 years warranty periods so I'm used to think this way :)

sanoid Outdated Show resolved Hide resolved
@phreaker0
Copy link
Collaborator

@jimsalterjrs made the necessary changes, this is now ready to be merged

@lckarssen
Copy link

Just adding a reminder that the Wiki page needs to be updated as well once this is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants