wip: test

tests/test_zfsautobackup34.py

@@ -12,6 +12,7 @@ def test_no_bookmark_source_support(self):
 
         subprocess.check_call("zpool destroy test_source1", shell=True)
         subprocess.check_call("zpool create -d test_source1 /dev/ram0", shell=True)
+        shelltest("zpool get all test_source1")
         subprocess.check_call("zfs create -p test_source1/fs1/sub", shell=True) # recreate with no features at all
         subprocess.check_call("zfs set autobackup:test=true test_source1/fs1", shell=True)
 

zfs_autobackup/ZfsDataset.py

@@ -1333,7 +1333,7 @@ def sync_snapshots(self, target_dataset, features, show_progress, filter_propert
                     prev_target_snapshot.release()
 
             #bookmark common snapshot on source, or use holds if bookmarks are not enabled.
-            if 'bookmarks' in features:
+            if 'bookmark_v2' in features:
                 source_snapshot.bookmark()
             else:
                 if holds:

zfs_autobackup/ZfsPool.py

@@ -1,3 +1,6 @@
+from logging import warning
+
+
 class ZfsPool():
     """a zfs pool"""