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

Implement Issue #245 Snapshot exclude patterns #264

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

exi
Copy link
Contributor

@exi exi commented Aug 18, 2024

Issue #245

This is mostly a first draft without tests, but it seems to work and i'm using these changes on top of v3.2.2 on my server.

The primary point why I needed this is because in the time zfs_autobackup runs, my zfs-autosnapshot has taken and removed multiple "frequent" snapshots, which leads to autobackup failures.
I just want autobackup the snapshots with "monthly", "weekly" and "daily" in the name.

@exi
Copy link
Contributor Author

exi commented Aug 18, 2024

@psy0rz I just wanted to get early feedback if this approach seems reasonable. I know you don't have much time to implement it yourself, so worst case i'll just use my own patched version on my server for the time being.

@psy0rz
Copy link
Owner

psy0rz commented Aug 20, 2024

looks good, but "is_exluded" is too generic a name i think?

@psy0rz
Copy link
Owner

psy0rz commented Aug 20, 2024

uh oohh some tests failed?

@psy0rz
Copy link
Owner

psy0rz commented Aug 20, 2024

oh nevermind they were already broken, i'll have to fix that first

@psy0rz
Copy link
Owner

psy0rz commented Sep 17, 2024

i fixed all the tests, can you rebase on master so that we can run the tests?

@exi exi force-pushed the snapshot_exclude_245 branch from c433c18 to 67f6f37 Compare September 17, 2024 19:17
@exi
Copy link
Contributor Author

exi commented Sep 17, 2024

@psy0rz rebased ✅

@coveralls
Copy link

Coverage Status

coverage: 86.145% (-0.3%) from 86.462%
when pulling 67f6f37 on exi:snapshot_exclude_245
into 9e2476a on psy0rz:master.

@psy0rz psy0rz merged commit c5f1e38 into psy0rz:master Sep 18, 2024
4 of 5 checks passed
@psy0rz
Copy link
Owner

psy0rz commented Sep 18, 2024

thanks!

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.

3 participants