zfs_mgmt 0.4.3 → 0.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 14fbd30f0a3d93dff6442c011b7a613b9639fd5855a9efd635b3b9b14fa336c7
-  data.tar.gz: 826699e0d0bca72bf87a779a87b14b9bcc01dffe34596eff471264b07b683d62
+  metadata.gz: 1022834076fa1482f627f9c5b7f8a476dd3e83d3358495d40f6695b23e842a88
+  data.tar.gz: ddc4f5f6e87c00f5cb0c0adbe2b4ddc47f78670c520b701d359b471b19c081c4
 SHA512:
-  metadata.gz: fbe275a0b964b664d08346def641abff1c63467e70edb47b60accbd519a3e2df0bf81f2e000f09b28c7cf4db2cd71aa67580c8b570ffc5171835f818b3783d3c
-  data.tar.gz: 4b326c73524719f3e103781532228d40fe09f0ebada718701d444d6296dfa5e310e2bd2a4035ad8cb6b08791ff5c8ddf7d0a1ecb6102d305957af853683ba356
+  metadata.gz: 06eab68af755becb316e438c5e027e78166952c7a37dc1c32343193f140e0a12463784b6a42ada068c0a90571641a11c7971143349d0dff6cc1a2581f34cbe9b
+  data.tar.gz: 9fa676ecf9837096a5e97649f01dfa398610898019f4abd0694fee241e75733767193898e26e618516c23cbf394dcdf1bdb8d9c4639a70a77d49db5b34c2ede4

data/Gemfile.lock

@@ -1,36 +1,39 @@
 PATH
   remote: .
   specs:
-    zfs_mgmt (0.4.3)
+    zfs_mgmt (0.4.6)
+      filesize (~> 0.2)
+      text-table (~> 1.2)
+      thor (~> 1.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    diff-lcs (1.3)
+    diff-lcs (1.5.0)
     filesize (0.2.0)
-    json (2.3.0)
-    rake (13.0.1)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.1)
-      rspec-support (~> 3.9.1)
-    rspec-expectations (3.9.0)
+    json (2.6.2)
+    rake (13.0.6)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.2)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
     text-table (1.2.4)
-    thor (1.0.1)
+    thor (1.2.1)
 
 PLATFORMS
-  ruby
+  x86_64-linux
 
 DEPENDENCIES
-  bundler (~> 1.16)
+  bundler (>= 2.2.33)
   filesize
   json
   rake (>= 12.3.3)
@@ -40,4 +43,4 @@ DEPENDENCIES
   zfs_mgmt!
 
 BUNDLED WITH
-   1.16.6
+   2.3.21

data/bin/zfsmgr

@@ -17,7 +17,7 @@ class ZfsMgr < Thor
                :enum => ['debug','error','fatal','info','warn'],
                :desc => 'set logging level to specified severity'
   desc "zfsget [ZFS]", "execute zfs get for the given properties and types and parse the output into a nested hash"
-  method_option :properties, :type => :array, :default => ['name'], :desc => "List of properties passed to zfs get"
+  method_option :properties, :type => :string, :default => ['name'], :desc => "List of properties passed to zfs get", repeatable: true
   method_option :types,      :type => :array, :default => ['filesystem','volume'], enum: ['filesystem','volume','snapshot'], :desc => "list of types"
   def zfsget(zfs)
     ZfsMgmt.set_log_level(options[:loglevel])

data/lib/zfs_mgmt/restic.rb

@@ -70,7 +70,7 @@ module ZfsMgmt::Restic
                                           'zfsmgmt:restic_repository',
                                           'userrefs',
                                          ],
-                             property_match: { 'zfsmgmt:restic_backup' => 'true' }).each do |blob|
+                             property_match: { 'zfsmgmt:restic_backup' => ['on','true'] }).each do |blob|
       zfs,props,zfs_snapshots = blob
       last_zfs_snapshot = zfs_snapshots.keys.sort { |a,b| zfs_snapshots[a]['creation'] <=> zfs_snapshots[b]['creation'] }.last
       zfs_snap_time = Time.at(zfs_snapshots[last_zfs_snapshot]['creation'])

data/lib/zfs_mgmt/version.rb

@@ -1,3 +1,3 @@
 module ZfsMgmt
-  VERSION = "0.4.3"
+  VERSION = "0.4.6"
 end

data/lib/zfs_mgmt/zfs_mgr/send.rb

@@ -25,8 +25,8 @@ class ZfsMgmt::ZfsMgr::Send < Thor
 
   method_option :noop, :aliases => :'-n', :desc => "pass -n (noop) option to zfs send", :type => :boolean
   method_option :unmount, :aliases => :'-u', :desc => "pass -u (unmount) option to zfs receive", :type => :boolean
-  method_option :exclude, :aliases => :'-x', :desc => "passed to -x option of receive side", :type => :array
-  method_option :option, :aliases => :'-o', :desc => "passed to -o option of receive side", :type => :array
+  method_option :exclude, :aliases => :'-x', :desc => "passed to -x option of receive side", :type => :string, repeatable: true
+  method_option :option, :aliases => :'-o', :desc => "passed to -o option of receive side", :type => :string, repeatable: true
   method_option :drop_holds, :desc => "pass the -h option to zfs recv, indicating holds should be ignored", :type => :boolean
 
   method_option :mbuffer, :desc => "insert mbuffer between send and recv", :default => true, :type => :boolean
@@ -35,29 +35,6 @@ class ZfsMgmt::ZfsMgr::Send < Thor
     ZfsMgmt.set_log_level(options[:loglevel])
     ZfsMgmt.global_options = options
 
-    [
-      { 'zfsmgmt:send' => 'true' },
-      # {
-      #   'zfsmgmt:send' => 'replicate',
-      #   'zfsmgmt:send@source' => 'local'
-      # },
-    ].each do |match|
-      ZfsMgmt.zfs_managed_list(filter: options[:filter],
-                               property_match: match).each do |zfs,props,snaps|
-        if props['zfsmgmt:send@source'] == 'received'
-          $logger.debug("skipping received filesystem: #{zfs}")
-          next
-        end
-        if props.has_key?('zfsmgmt:send_replicate') and props['zfsmgmt:send_replicate'] == 'true' and props['zfsmgmt:send_replicate@source'] != 'local'
-          $logger.debug("skipping descendant of replicated filesystems: #{zfs}")
-          next
-        end
-        unless props['zfsmgmt:destination']
-          $logger.error("#{zfs}: you must specify a destination zfs path via the user property zfsmgmt:destination, even if using --destination on the command line, skipping")
-          next
-        end
-        ZfsMgmt.zfs_send(options,zfs,props,snaps)
-      end
-    end
+    ZfsMgmt.zfs_send_all(options)
   end
 end

data/lib/zfs_mgmt.rb

@@ -238,7 +238,7 @@ module ZfsMgmt
     }
     return saved,saved_snaps,deleteme
   end
-  def self.zfs_managed_list(filter: '.+', properties: ['all'], property_match: { 'zfsmgmt:manage' => 'true' } )
+  def self.zfs_managed_list(filter: '.+', properties: ['all'], property_match: { 'zfsmgmt:manage' => method(:prop_on?) } )
     zfss = [] # array of arrays
     zfsget(properties: properties).each do |zfs,props|
       unless /#{filter}/ =~ zfs
@@ -246,7 +246,7 @@ module ZfsMgmt
       end
       managed = true
       property_match.each do |k,v|
-        unless props.has_key?(k) and props[k] == v
+        unless key_comp?(props,k,v)
           managed = false
           break
         end
@@ -311,7 +311,7 @@ module ZfsMgmt
         $logger.debug("delete: #{snap_name} #{local_epoch_to_datetime(snaps[snap_name]['creation']).strftime('%F %T')}")
       end
 
-      com_base = ['zfs', 'destroy']
+      com_base = [ZfsMgmt.global_options[:zfs_binary], 'destroy']
       com_base.push('-d') if deleteme.length > 0 # why?
       com_base.push('-n') if noop
       com_base.push('-v') if verbose
@@ -422,8 +422,8 @@ module ZfsMgmt
       com = []
       source = sorted[0]
       if options[:initial_snapshot] == 'newest' or
-        ( options.has_key?('replicate') and options['replicate'] == true ) or
-        ( props.has_key?('zfsmgmt:send_replicate') and props['zfsmgmt:send_replicate'] == 'true' )
+        key_comp?(options, 'replicate') or
+        key_comp?(props, 'zfsmgmt:send_replicate')
         source = sorted[-1]
       end
       com += zfs_send_com(options,
@@ -445,7 +445,7 @@ module ZfsMgmt
       # should be resumable!
       com = [ ]
       com.push( ZfsMgmt.global_options[:zfs_binary], 'send', '-t', remote_zfs_state )
-      com.push('-v','-P') if options[:verbose] and options[:verbose] == 'send'
+      com.push('-v','-P') if key_comp?(options, 'verbose', 'send')
       com.push('|')
       e = zfs_send_estimate(com) if options[:verbose] == 'pv'
       com += mbuffer_command(options) if options[:mbuffer]
@@ -498,7 +498,15 @@ module ZfsMgmt
       if snaps.has_key?(rsnap.sub(destination_path,zfs))
         $logger.debug("process #{rsnap} to #{sorted[-1]}")
         com = []
-        com += zfs_send_com(options,props,[(options[:intermediary] ? '-I' : '-i'),dq(rsnap.split('@')[1])],sorted[-1])
+        i_opt = '-i'
+        # allow the command line option for intermediary to override the property
+        if key_comp?(options,'intermediary',[true,false])
+          i_opt = '-I' if key_comp?(options, 'intermediary', true)
+        elsif key_comp?(props, 'zfsmgmt:send_intermediary')
+          i_opt = '-I'
+        end
+
+        com += zfs_send_com(options,props,[i_opt, dq('@' + rsnap.split('@')[1])], sorted[-1])
         e = zfs_send_estimate(com) if options[:verbose] == 'pv'
         com += mbuffer_command(options) if options[:mbuffer]
         com += pv_command(options,e) if options[:verbose] == 'pv'
@@ -521,7 +529,7 @@ module ZfsMgmt
   end
   def self.zfs_send_com(options,props,extra_opts,target)
     zfs_send_com = [ ZfsMgmt.global_options[:zfs_binary], 'send' ]
-    zfs_send_com.push('-v','-P') if options[:verbose] and options[:verbose] == 'send'
+    zfs_send_com.push('-v','-P') if key_comp?(options,'verbose','send')
     send_opts = {
       'backup'      => '-b',
       'compressed'  => '-c',
@@ -533,10 +541,11 @@ module ZfsMgmt
       'replicate'   => '-R',
     }
     send_opts.each do |p,o|
-      if options.has_key?(p)
-        zfs_send_com.push(o) if options[p] == true
-      elsif props.has_key?("zfsmgmt:send_#{p}")
-        zfs_send_com.push(o) if props["zfsmgmt:send_#{p}"] == 'true'
+      # allow the command line options to override the properties value
+      if key_comp?(options,p,[true,false])
+        zfs_send_com.push(o) if key_comp?(options,p,true)
+      elsif key_comp?(props,"zfsmgmt:send_#{p}")
+        zfs_send_com.push(o)
       end
     end
     zfs_send_com + extra_opts + [dq(target),'|']
@@ -544,18 +553,21 @@ module ZfsMgmt
   def self.zfs_recv_com(options,extra_opts,props,target)
     zfs_recv_com = [ ZfsMgmt.global_options[:zfs_binary], 'recv', '-F', '-s' ]
     recv_opts = {
-      'noop'       => '-n',
-      'drop_holds' => '-h',
-      'unmount'    => '-u',
+      'noop'          => '-n',
+      'drop_holds'    => '-h',
+      'unmount'       => '-u',
+      #'discard_last'  => '-e',
+      #'discard_first' => '-d',
     }
     recv_opts.each do |p,o|
-      if options.has_key?(p)
-        zfs_recv_com.push(o) if options[p] == true
-      elsif props.has_key?("zfsmgmt:recv_#{p}")
-        zfs_recv_com.push(o) if props["zfsmgmt:recv_#{p}"] == 'true'
+      # allow the command line options to override the properties value
+      if key_comp?(options,p,[true,false])
+        zfs_recv_com.push(o) if key_comp?(options,p,true)
+      elsif key_comp?(props,"zfsmgmt:recv_#{p}")
+        zfs_recv_com.push(o)
       end
     end
-    zfs_recv_com.push('-v') if options[:verbose] and ( options[:verbose] == 'receive' or options[:verbose] == 'recv' )
+    zfs_recv_com.push('-v') if key_comp?(options, 'verbose', ['receive', 'recv'])
     if options[:exclude]
       options[:exclude].each do |x|
         zfs_recv_com.push('-x',x)
@@ -628,14 +640,13 @@ module ZfsMgmt
       return v.include?(h[p])
     elsif v.kind_of?(Hash)
       return v.keys.include?(h[p])
-    elsif v.kind_of?(String)
-      return h[p] == v
     elsif v.kind_of?(Method)
       return v.call(h[p])
     elsif v.kind_of?(Regexp)
       return v =~ h[p]
     else
-      raise ArgumentError
+      # string, boolean, numbers?
+      return h[p] == v
     end
   end
   def self.set_log_level(sev)
@@ -652,4 +663,23 @@ module ZfsMgmt
       $logger.level = Logger::FATAL
     end
   end
+  def self.zfs_send_all(options)
+    zfs_managed_list(filter: options[:filter],
+                     property_match: { 'zfsmgmt:send' => method(:prop_on?) }).each do |zfs,props,snaps|
+      
+      if props['zfsmgmt:send@source'] == 'received'
+        $logger.debug("skipping received filesystem: #{zfs}")
+        next
+      end
+      if key_comp?(props,'zfsmgmt:send_replicate') and props['zfsmgmt:send_replicate@source'] != 'local'
+        $logger.debug("skipping descendant of replicated filesystems: #{zfs}")
+        next
+      end
+      unless props['zfsmgmt:destination']
+        $logger.error("#{zfs}: you must specify a destination zfs path via the user property zfsmgmt:destination, even if using --destination on the command line, skipping")
+        next
+      end
+      zfs_send(options,zfs,props,snaps)
+    end
+  end
 end

data/zfs_mgmt.gemspec

@@ -36,11 +36,11 @@ Gem::Specification.new do |spec|
   spec.executables   = ['readsnaps','zfssendman','zfssnapman','zfsrecvman','zfs-list-snapshots','zfsmgr']
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.16"
+  spec.add_development_dependency "bundler", ">= 2.2.33"
   spec.add_development_dependency "rake", ">= 12.3.3"
   spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "thor", "~> 1.0"
-  spec.add_development_dependency "text-table", "~> 1.2"
-  spec.add_development_dependency "filesize", "~> 0.2"
+  spec.add_runtime_dependency "thor", "~> 1.0"
+  spec.add_runtime_dependency "text-table", "~> 1.2"
+  spec.add_runtime_dependency "filesize", "~> 0.2"
   
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: zfs_mgmt
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.4.6
 platform: ruby
 authors:
 - Aran Cox
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-11 00:00:00.000000000 Z
+date: 2022-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.16'
+        version: 2.2.33
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.16'
+        version: 2.2.33
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -59,7 +59,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
-  type: :development
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -73,7 +73,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.2'
-  type: :development
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -87,14 +87,14 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
-  type: :development
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
-description: 
+description:
 email:
 - arancox@gmail.com
 executables:
@@ -140,7 +140,7 @@ metadata:
   homepage_uri: https://github.com/aranc23/zfs_mgmt
   source_code_uri: https://github.com/aranc23/zfs_mgmt
   changelog_uri: https://github.com/aranc23/zfs_mgmt/commits/
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -155,8 +155,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
 summary: Misc. helpers regarding snapshots and send/recv.
 test_files: []