lvmsync 3.2.0 → 3.2.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: f78e30fa37ccf5150643fabe206713fe18750271
+  data.tar.gz: 0c0416476d7005349864889700f76541ad1eda45
+SHA512:
+  metadata.gz: 8bdf6db2b0e37ce28af97557c0c3580b227379991b2abc0cdec656ca88333a51c48b75d2d68fd8cf826b4a0947791ea63ffebf25fb55efc11f94c641b3328741
+  data.tar.gz: 98f9474e839e61d8f6a17be2b9bd8976715cafa594c43ba066a1a9b5f6c18ae265b7da3577e415b476f6c7812fc521e02b9cfe4c43d17d2217c6333306080402

data/README.md

@@ -107,7 +107,7 @@ Let's say you've got an LV, named `vmsrv1/somevm`, and you'd like to
 synchronise it to a new VM server, named `vmsrv2`.  Assuming that `lvmsync` is
 installed on `vmsrv2` and `vmsrv2` has an LV named `vmsrv2/somevm` large
 enough to take the data, the following will do the trick rather nicely (all
-commands should be run on `vmsrv1`:
+commands should be run on `vmsrv1`):
 
     # Take a snapshot before we do anything, so LVM will record all changes
     # made while we're doing the initial sync
@@ -168,7 +168,7 @@ each hour):
 
 This will produce files in /var/snapbacks named `somevm.<date-time>`.  You
 need to create the `somevm-snapback-new` snapshot before you start
-`lvmsync`, so that you can guarantee no changes won't get noticed.
+`lvmsync`, so that you can guarantee no changes will go unnoticed.
 
 There are some fairly large caveats to this method -- the LV will still be
 collecting writes while you're transferring the snapshots, so you won't get

data/bin/lvmsync

@@ -43,10 +43,13 @@ def main()
 		end
 
 		opts.on("-v", "--[no-]verbose",
-		        "Run verbosely") { |v| options[:verbose] = v }
+		        "Run verbosely") { |v| $verbose = v }
+
+		opts.on("-d", "--[no-]debug",
+		        "Print debugging information") { |v| $debug = v }
 
 		opts.on("-q", "--[no-]quiet",
-		        "Run quietly") { |v| options[:quiet] = v }
+		        "Run quietly") { |v| $quiet = v }
 
 		opts.on("-b <file>", "--snapback <file>",
 		        "Make a backup snapshot file on the destination") do |v|
@@ -75,49 +78,42 @@ def main()
 				puts "lvmsync #{GVB.version}"
 				exit 0
 			rescue GVB::VersionUnobtainable
-				$stderr.puts "Unable to determine lvmsync version."
-				$stderr.puts "Install lvmsync as a gem, or run it from within a git checkout"
-				exit 1
+				fatal "Unable to determine lvmsync version.\n" +
+				      "Install lvmsync as a gem, or run it from within a git checkout"
 			end
 		end
 	end.parse!
 
-	if options[:quiet] and options[:verbose]
-		$stderr.puts "I can't run quietly *and* verbosely at the same time!"
-		exit 1
+	if $quiet and ($verbose or $debug)
+		fatal "I can't run quietly *and* verbosely at the same time!"
 	end
 
 	if options[:apply]
 		if ARGV[0].nil?
-			$stderr.puts "No destination device specified."
-			exit 1
+			fatal "No destination device specified."
 		end
 		options[:device] = ARGV[0]
 		run_apply(options)
 	elsif options[:server]
-		$stderr.puts "--server is deprecated; please use '--apply -' instead" unless opts[:quiet]
+		info "--server is deprecated; please use '--apply -' instead"
 		if (ARGV[0].nil?)
-			$stderr.puts "No destination block device specified.  WTF?"
-			exit 1
+			fatal "No destination block device specified.  WTF?"
 		end
 		options[:apply] = '-'
 		options[:device] = ARGV[0]
 		run_apply(options)
 	else
 		if ARGV[0].nil?
-			$stderr.puts "ERROR: No snapshot specified.  Exiting."
-			exit 1
+			fatal "No snapshot specified.  Exiting.  Do you need --help?"
 		end
 		options[:snapdev] = ARGV[0]
 
 		if options[:stdout] and options[:snapback]
-			$stderr.puts "--snapback cannot be used with --stdout"
-			exit 1
+			fatal "--snapback cannot be used with --stdout"
 		end
 
 		if (options[:stdout].nil? and ARGV[1].nil?)
-			$stderr.puts "No destination specified."
-			exit 1
+			fatal "No destination specified."
 		end
 		if options[:stdout].nil?
 			dev, host = ARGV[1].split(':', 2).reverse
@@ -143,18 +139,19 @@ end
 def process_dumpdata(instream, destdev, snapback = nil, opts = {})
 	handshake = instream.readline.chomp
 	unless handshake == PROTOCOL_VERSION
-		$stderr.puts "Handshake failed; protocol mismatch? (saw '#{handshake}' expected '#{PROTOCOL_VERSION}'"
-		exit 1
+		fatal "Handshake failed; protocol mismatch? (saw '#{handshake}' expected '#{PROTOCOL_VERSION}'"
 	end
 
 	snapback.puts handshake if snapback
 
+	verbose "Writing changed data to #{destdev.inspect}"
 	File.open(destdev, 'w+') do |dest|
 		while header = instream.read(12)
 			offset, chunksize = header.unpack("QN")
 			offset = ntohq(offset)
 
 			begin
+				debug "Seeking to #{offset}"
 				dest.seek offset
 			rescue Errno::EINVAL
 				# In certain rare circumstances, we want to transfer a block
@@ -164,6 +161,8 @@ def process_dumpdata(instream, destdev, snapback = nil, opts = {})
 				# if you didn't notice that your dd shit itself, it's unlikely
 				# you're going to notice now.
 
+				info "Write occured past end of device"
+
 				# Skip the chunk of data
 				instream.read(chunksize)
 				# Go to the next chunk
@@ -173,9 +172,12 @@ def process_dumpdata(instream, destdev, snapback = nil, opts = {})
 			if snapback
 				snapback.write(header)
 				snapback.write dest.read(chunksize)
+				# Got to back to where we were before, since the read from dest
+				# has advanced the file pointer by `chunksize`
 				dest.seek offset
 			end
 			dest.write instream.read(chunksize)
+			debug "Wrote #{chunksize} bytes at #{offset}"
 		end
 	end
 end
@@ -189,13 +191,11 @@ def run_client(opts)
 	lv = begin
 		LVM::LogicalVolume.new(snapshot)
 	rescue RuntimeError => e
-		$stderr.puts "#{snapshot}: could not find logical volume (#{e.message})"
-		exit 1
+		fatal "#{snapshot}: could not find logical volume (#{e.message})"
 	end
 
 	unless lv.snapshot?
-		$stderr.puts "#{snapshot}: Not a snapshot device"
-		exit 1
+		fatal "#{snapshot}: Not a snapshot device"
 	end
 
 	# Since, in principle, we're not supposed to be reading from snapshot
@@ -208,14 +208,16 @@ def run_client(opts)
 	snapback = opts[:snapback] ? "--snapback #{opts[:snapback]}" : ''
 
 	source = opts[:source] || lv.origin.path
-	$stderr.puts "Data source: #{source}" if opts[:verbose]
+	verbose "Data source: #{source}"
 
 	if opts[:stdout]
 		dump_changes(lv, source, $stdout, opts)
 	else
-		verbose = opts[:verbose] ? '-v' : ''
+		verbose = $verbose ? '-v' : ''
+		debug   = $debug   ? '-d' : ''
+
 		server_cmd = if desthost
-			"#{opts[:rsh]} #{desthost} lvmsync --apply - #{snapback} #{verbose} #{destdev}"
+			"#{opts[:rsh]} #{desthost} lvmsync --apply - #{snapback} #{verbose} #{debug} #{destdev}"
 		else
 			"#{$0} --apply - #{snapback} #{verbose} #{destdev}"
 		end
@@ -232,7 +234,7 @@ def run_client(opts)
 				until (active_fds = IO.select(fds, [], [], 0)).nil?
 					active_fds[0].each do |fd|
 						begin
-							$stderr.puts "\e[2K\rremote:#{fd.readline}" unless opts[:quiet]
+							info "\e[2K\rremote:#{fd.readline}"
 						rescue EOFError, Errno::EPIPE
 							fd.close
 							fds.delete(fd)
@@ -251,7 +253,7 @@ def run_client(opts)
 			until (active_fds = IO.select(fds, [], [], 0.1)).nil?
 				active_fds[0].each do |fd|
 					begin
-						$stderr.puts "\e[2K\rremote:#{fd.readline}" unless opts[:quiet]
+						info "\e[2K\rremote:#{fd.readline}"
 					rescue EOFError, Errno::EPIPE
 						fd.close
 						fds.delete(fd)
@@ -262,7 +264,7 @@ def run_client(opts)
 		end
 
 		if (exit_status or $?).exitstatus != 0
-			$stderr.puts "APPLY FAILED."
+			fatal "APPLY FAILED."
 		end
 	end
 end
@@ -282,8 +284,7 @@ def dump_changes(snapshot, source, outfd, opts)
 			chunk_size = r.last - r.first + 1
 			xfer_size  += chunk_size
 
-			$stderr.puts "Sending chunk #{r.to_s}..." if opts[:verbose]
-			$stderr.puts "Seeking to #{r.first} in #{source}" if opts[:verbose]
+			debug "Sending chunk #{r.to_s}..."
 
 			origindev.seek(r.first, IO::SEEK_SET)
 
@@ -297,7 +298,7 @@ def dump_changes(snapshot, source, outfd, opts)
 			end
 
 			# Progress bar!
-			if xfer_count % 100 == 50 and !opts[:quiet]
+			if xfer_count % 100 == 50 and !$quiet
 				$stderr.printf "\e[2K\rSending chunk %i of %i, %.2fMB/s",
 									xfer_count,
 									change_count,
@@ -311,7 +312,7 @@ def dump_changes(snapshot, source, outfd, opts)
 		total_size = origindev.tell
 	end
 
-	unless opts[:quiet]
+	unless $quiet
 		$stderr.printf "\rTransferred %i bytes in %.2f seconds\n",
 		               xfer_size, Time.now - start_time
 
@@ -336,4 +337,21 @@ def parse_snapshot_name(origname)
 	end
 end
 
+def debug(s)
+	$stderr.puts s if $debug
+end
+
+def verbose(s)
+	$stderr.puts s if $verbose or $debug
+end
+
+def info(s)
+	$stderr.puts s unless $quiet
+end
+
+def fatal(s, status=1)
+	$stderr.puts "FATAL ERROR: #{s}"
+	exit status
+end
+
 main

data/lib/vgcfgbackup.treetop

@@ -24,7 +24,7 @@ grammar VgCfgBackup
 	end
 
 	rule list
-		"[" space? ((string / integer) ", " / (string / integer))* space? "]" <List>
+		"[" space? ((string / integer) "," space? / (string / integer))* space? "]" <List>
 	end
 
 	rule space

metadata

@@ -1,206 +1,181 @@
 --- !ruby/object:Gem::Specification
 name: lvmsync
 version: !ruby/object:Gem::Version
-  version: 3.2.0
-  prerelease: 
+  version: 3.2.1
 platform: ruby
 authors:
 - Matt Palmer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-14 00:00:00.000000000 Z
+date: 2015-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: git-version-bump
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
 - !ruby/object:Gem::Dependency
   name: treetop
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: github-release
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-spork
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: plymouth
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry-debugger
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rb-inotify
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: 
@@ -211,47 +186,40 @@ extensions: []
 extra_rdoc_files:
 - README.md
 files:
-- README.md
 - LICENCE
+- README.md
 - bin/lvmsync
 - lib/lvm.rb
-- lib/lvm/lv_config.rb
+- lib/lvm/helpers.rb
 - lib/lvm/logical_volume.rb
-- lib/lvm/thin_snapshot.rb
+- lib/lvm/lv_config.rb
+- lib/lvm/pv_config.rb
 - lib/lvm/snapshot.rb
+- lib/lvm/thin_snapshot.rb
 - lib/lvm/vg_config.rb
-- lib/lvm/helpers.rb
-- lib/lvm/pv_config.rb
-- lib/vgcfgbackup.treetop
 - lib/vgcfgbackup.rb
+- lib/vgcfgbackup.treetop
 homepage: http://theshed.hezmatt.org/lvmsync
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -3943591993715626870
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -3943591993715626870
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Efficiently transfer changes in LVM snapshots
 test_files: []