I got it working for zfs

[kryptt]

If you are interested I can submit a patch to use j-keck/zfs-snap-diff to get this working with zfs if you are interested I am probably prepare a pull request for it...

[priyadarshan]

I opened #1 back in 2016, and I am still very much interested in using snapshot-timemachine with zfs. Would you give access to your fork, or a patch? Currently I am using zfs on illumos servers (mainly smartOS).

[kryptt]

https://github.com/kryptt/snapshot-timemachine

I need to add more error handling, but it currently works on my machine.. let me know

[priyadarshan]

Thank you. I am currently setting up a new test SmartOS server, and will test it there and report back in a few days.

[priyadarshan]

A new version of zfs-snap-diff just came out. I will test your patch with latest version.

[mrBliss]

https://github.com/kryptt/snapshot-timemachine

I need to add more error handling, but it currently works on my machine.. let me know

I'm open to a PR whenever you think your patches are mature 🙂

[j-keck]

I'm the author of zfs-snap-diff and was pointed by @priyadarshan to this.

I can change zsd to accept a new flag (maybe -snapshot-timemachine-output?) to
provide a special / stable output for this package.

Just ping me and i can implement the missing parts on my side.

[kryptt]

@j-keck yeah, that would be great; right now I'm running it in debug to get a couple of parameters from there.

I will write an issue for you in zfs-snap-diff, yeah?

[j-keck]

@kryptt

I will write an issue for you in zfs-snap-diff, yeah?

yes, pelase

[kryptt]

I created a separate pull request #5 with 'proper' zfs support