]> git.proxmox.com Git - pve-storage.git/commit
zfs: use -r parameter when listing snapshots
authorFabian Ebner <f.ebner@proxmox.com>
Mon, 10 Jan 2022 11:50:44 +0000 (12:50 +0100)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Mon, 10 Jan 2022 12:33:31 +0000 (13:33 +0100)
commit73dfe360dd434cf8aa89d9e2a5e6b3e417d5718c
tree8333e4b86aaf3c96e7f44f6ef507e536ec3a5202
parentb4616e5c4a68555e66a4210b18e436916da32aec
zfs: use -r parameter when listing snapshots

Some versions of ZFS do not automatically display the child snapshots
when '-t snapshot' is used, but require '-r' to be present
additionally[1]. And in general, it's cleaner to specify the flag
explicitly.

Because of that, commit ac5c1af led to a regression[0] in the context
of ZFS over iSCSI with zfs_get_sorted_snapshot_list. Fix it, by adding
a -r flag again.

The volume_snapshot_info function is currently only used in the
context of replication and that requires a local ZFS pool, but it
would be affected by the same issue if it is ever used in the context
of ZFS over iSCSI, so also add -r there.

[0]: https://forum.proxmox.com/threads/102683/
[1]: https://forum.proxmox.com/threads/102683/post-442577

Fixes: 8c20d8a ("plugin: add volume_snapshot_info function")
Fixes: ac5c1af ("zfspool: add zfs_get_sorted_snapshot_list helper")
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
PVE/Storage/ZFSPoolPlugin.pm