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

Deprecated SNAPOID_SNAPNAME includes target #798

Open
sTywin opened this issue Feb 6, 2023 · 0 comments
Open

Deprecated SNAPOID_SNAPNAME includes target #798

sTywin opened this issue Feb 6, 2023 · 0 comments

Comments

@sTywin
Copy link

sTywin commented Feb 6, 2023

In version 2.0.3, the snapshot name environment variable did not include the target or @ symbol:

SANOID_SNAPNAME='autosnap_2023-02-06_03:00:06_hourly'

Since upgrading to 2.1.0, the now-deprecated snapshot name includes the target:

SANOID_SNAPNAME='rpool/archive/vmail@autosnap_2023-02-06_03:00:06_hourly'

I am working on updating my post-snapshot script to handle the new SANOID_SNAPNAMES (and SANOID_TARGETS) variables, but that will take a bit of time. It's not clear if this change was intentional or not. Note that the documentation in README.md indicates that the SANOID_SNAPNAME variable should not include the target:

SANOID_SNAPNAME=autosnap_2020-02-12_14:49:32_yearly

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

No branches or pull requests

1 participant