go.mod: github.com/containerd/zfs v0.0.0-20210322090317-0e92c2247fb7 (for snapshot usage)

[AkihiroSuda]

Contains "Implements zfs usage by returning the USED field from the zfs snapshot"
(containerd/zfs#38)

[thaJeztah]

LGTM

[estesp]

LGTM

[dalbani]

Question from a total newbie on containerd's release process: I suppose this fix can't / won't be backported to the 1.4 branch?
I'm asking because this fix would be more than welcome in MicroK8s, which will use containerd 1.4.x for the next MicroK8s 1.21 release.

[dmcgowan]

@dalbani zfs is supported in 1.4 so this would be a backport-able change. I believe that would require backporting in the zfs repo as well to exclude non-backported changes when vendoring into the 1.4 branch.

[dalbani]

@dmcgowan
Shall I then open a feature request regarding my wish to have this function backported to 1.4?

[thaJeztah]

I believe that would require backporting in the zfs repo as well to exclude non-backported changes when vendoring into the 1.4 branch.

Diff between version used in 1.4 and this version; containerd/zfs@9abf673...0e92c22. At a glance, most changes look "ok" (so probably wouldn't require containerd/zfs to branch for 1.4); only change that requires local changes (afaics) is this commit: containerd/zfs@804f3e0 (containerd/zfs#31) (would depend on cherry-picking 96f4ce4 #4534)

[dalbani]

Thanks @thaJeztah for this detailed report! Let me know if I can help.