I looked in both 4.0.0 and HEAD source, and the call to zpool uses
def->source.name.  I think it would work fine if it stripped out
everything from the first slash.

(Aside: if the zpool command fails, virStorageBackendZFSRefreshPool()
just goes straight to cleanup and returns zero, as if nothing bad had
happened)

It then goes on to call zfs list, which would do the following:

root@beaver:~# zfs list -Hp -t volume -r -o name,volsize,refreservation 
zfs/images
zfs/images/vol1 1073741824      1109393408

That looks perfectly OK to me.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1767997

Title:
  virt-manager destroys all volumes in libvirt zfs pool

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/virt-manager/+bug/1767997/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to