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

add option patronizemenot aka do_prerun_fs_checks=false #21

Closed
wants to merge 2 commits into from

Conversation

btittelbach
Copy link

This is a workaround for issue #18 and allows the script to be used in a setup where the script does not have write permissions on a directory but the remove-command does. It also allows using the script with zfs snapshots, which are stored in a hidden .zfs/snapshot directory for which the existence check fails.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 90.463% when pulling e4cd922 on btittelbach:master into 611c72b on xolox:master.

@btittelbach
Copy link
Author

looks like flake8-docstring fails on python2

xolox added a commit that referenced this pull request Feb 11, 2020
The mentioned issue / pull request made it clear to me that its
very well possible for these (well intentioned) sanity checks
to fail. Nevertheless I'm not inclined to just remove them.
I do believe the changes here and in daa41a2 suitably
accommodate the use case described in issue 18.
@xolox
Copy link
Owner

xolox commented Feb 12, 2020

Please see my feedback and the new rotate-backups release in #18.

@xolox xolox closed this Feb 12, 2020
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.

None yet

3 participants