backup 4.3.0 → 5.0.0.beta.3

data/lib/backup/notifier/zabbix.rb

@@ -1,9 +1,6 @@
-# encoding: utf-8
-
 module Backup
   module Notifier
     class Zabbix < Base
-
       attr_accessor :zabbix_host
 
       attr_accessor :zabbix_port
@@ -19,10 +16,10 @@ module Backup
         instance_eval(&block) if block_given?
 
         @zabbix_host  ||= Config.hostname
-        @zabbix_port  ||= 10051
-        @service_name ||= "Backup #{ model.trigger }"
+        @zabbix_port  ||= 10_051
+        @service_name ||= "Backup #{model.trigger}"
         @service_host ||= Config.hostname
-        @item_key     ||= 'backup_status'
+        @item_key     ||= "backup_status"
       end
 
       private
@@ -45,18 +42,18 @@ module Backup
       # : Notification will be sent if `on_warning` or `on_success` is `true`.
       #
       def notify!(status)
-        send_message(message.call(model, :status => status_data_for(status)))
+        send_message(message.call(model, status: status_data_for(status)))
       end
 
       def send_message(message)
         msg = [service_host, service_name, model.exit_status, message].join("\t")
-        cmd = "#{ utility(:zabbix_sender) }" +
-              " -z '#{ zabbix_host }'" +
-              " -p '#{ zabbix_port }'" +
-              " -s #{ service_host }"  +
-              " -k #{ item_key }"      +
-              " -o '#{ msg }'"
-        run("echo '#{ msg }' | #{ cmd }")
+        cmd = utility(:zabbix_sender).to_s +
+          " -z '#{zabbix_host}'" \
+          " -p '#{zabbix_port}'" \
+          " -s #{service_host}"  \
+          " -k #{item_key}"      \
+          " -o '#{msg}'"
+        run("echo '#{msg}' | #{cmd}")
       end
     end
   end

data/lib/backup/package.rb

@@ -1,8 +1,5 @@
-# encoding: utf-8
-
 module Backup
   class Package
-
     ##
     # The time when the backup initiated (in format: 2011.02.20.03.29.59)
     attr_accessor :time
@@ -29,8 +26,8 @@ module Backup
 
     def initialize(model)
       @trigger = model.trigger
-      @extension = 'tar'
-      @chunk_suffixes = Array.new
+      @extension = "tar"
+      @chunk_suffixes = []
       @no_cycle = false
       @version = VERSION
     end
@@ -39,17 +36,16 @@ module Backup
       if chunk_suffixes.empty?
         [basename]
       else
-        chunk_suffixes.map {|suffix| "#{ basename }-#{ suffix }" }
+        chunk_suffixes.map { |suffix| "#{basename}-#{suffix}" }
       end
     end
 
     def basename
-      "#{ trigger }.#{ extension }"
+      "#{trigger}.#{extension}"
     end
 
     def time_as_object
-      Time.strptime(time, '%Y.%m.%d.%H.%M.%S')
+      Time.strptime(time, "%Y.%m.%d.%H.%M.%S")
     end
-
   end
 end

data/lib/backup/packager.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Backup
   module Packager
     class Error < Backup::Error; end
@@ -22,7 +20,7 @@ module Backup
           Logger.info "Packaging Complete!"
         else
           raise Error, "Failed to Create Backup Package\n" +
-              @pipeline.error_messages
+            @pipeline.error_messages
         end
       end
 
@@ -43,8 +41,8 @@ module Backup
         # or the Splitter (if no Encryptor), or through `cat` into the final
         # output file if neither are configured.
         @pipeline.add(
-          "#{ utility(:tar) } -cf - " +
-          "-C '#{ Config.tmp_path }' '#{ @package.trigger }'",
+          "#{utility(:tar)} -cf - " \
+          "-C '#{Config.tmp_path}' '#{@package.trigger}'",
           tar_success_codes
         )
 
@@ -75,26 +73,27 @@ module Backup
         #
         # If no Splitter was configured, the final file output will be
         # piped through `cat` into the final output file.
-        if @splitter
-          stack << lambda do
-            @splitter.split_with do |command|
-              @pipeline << command
+        stack <<
+          if @splitter
+            lambda do
+              @splitter.split_with do |command|
+                @pipeline << command
+                stack.shift.call
+              end
+            end
+          else
+            lambda do
+              outfile = File.join(Config.tmp_path, @package.basename)
+              @pipeline << "#{utility(:cat)} > #{outfile}"
               stack.shift.call
             end
           end
-        else
-          stack << lambda do
-            outfile = File.join(Config.tmp_path, @package.basename)
-            @pipeline << "#{ utility(:cat) } > #{ outfile }"
-            stack.shift.call
-          end
-        end
 
         ##
         # Last Proc to be called runs the Pipeline the procedure built.
         # Once complete, the call stack will unwind back through the
         # preceeding Procs in the stack (if any)
-        stack << lambda { @pipeline.run }
+        stack << -> { @pipeline.run }
 
         stack.shift
       end

data/lib/backup/pipeline.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Backup
   class Pipeline
     class Error < Backup::Error; end
@@ -12,7 +10,7 @@ module Backup
       @commands = []
       @success_codes = []
       @errors = []
-      @stderr = ''
+      @stderr = ""
     end
 
     ##
@@ -51,22 +49,21 @@ module Backup
     # Use `#success?` to determine if all commands in the pipeline succeeded.
     # If `#success?` returns `false`, use `#error_messages` to get an error report.
     def run
-      Open4.popen4(pipeline) do |pid, stdin, stdout, stderr|
-        pipestatus = stdout.read.gsub("\n", '').split(':').sort
+      Open4.popen4(pipeline) do |_pid, _stdin, stdout, stderr|
+        pipestatus = stdout.read.delete("\n").split(":").sort
         pipestatus.each do |status|
-          index, exitstatus = status.split('|').map(&:to_i)
-          unless @success_codes[index].include?(exitstatus)
-            command = command_name(@commands[index])
-            @errors << SystemCallError.new(
-              "'#{ command }' returned exit code: #{ exitstatus }", exitstatus
-            )
-          end
+          index, exitstatus = status.split("|").map(&:to_i)
+          next if @success_codes[index].include?(exitstatus)
+          command = command_name(@commands[index])
+          @errors << SystemCallError.new(
+            "'#{command}' returned exit code: #{exitstatus}", exitstatus
+          )
         end
         @stderr = stderr.read.strip
       end
       Logger.warn(stderr_messages) if success? && stderr_messages
     rescue Exception => err
-      raise Error.wrap(err, 'Pipeline failed to execute')
+      raise Error.wrap(err, "Pipeline failed to execute")
     end
 
     def success?
@@ -78,9 +75,9 @@ module Backup
     # from the commands in the pipeline (if any), along with the SystemCallError
     # (Errno) message for each command which had a non-zero exit status.
     def error_messages
-      @error_messages ||= (stderr_messages || '') +
-          "The following system errors were returned:\n" +
-          @errors.map {|err| "#{ err.class }: #{ err.message }" }.join("\n")
+      @error_messages ||= (stderr_messages || "") +
+        "The following system errors were returned:\n" +
+        @errors.map { |err| "#{err.class}: #{err.message}" }.join("\n")
     end
 
     private
@@ -106,19 +103,18 @@ module Backup
     def pipeline
       parts = []
       @commands.each_with_index do |command, index|
-        parts << %Q[{ #{ command } 2>&4 ; echo "#{ index }|$?:" >&3 ; }]
+        parts << %({ #{command} 2>&4 ; echo "#{index}|$?:" >&3 ; })
       end
-      %Q[{ #{ parts.join(' | ') } } 3>&1 1>&2 4>&2]
+      %({ #{parts.join(" | ")} } 3>&1 1>&2 4>&2)
     end
 
     def stderr_messages
-      @stderr_messages ||= @stderr.empty? ? false : <<-EOS.gsub(/^ +/, '  ')
+      @stderr_messages ||= @stderr.empty? ? false : <<-EOS.gsub(/^ +/, "  ")
         Pipeline STDERR Messages:
         (Note: may be interleaved if multiple commands returned error messages)
 
-        #{ @stderr }
+        #{@stderr}
       EOS
     end
-
   end
 end

data/lib/backup/splitter.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Backup
   class Splitter
     include Utilities::Helpers
@@ -21,8 +19,8 @@ module Backup
     # Once the packaging procedure is complete, it will return and
     # @package.chunk_suffixes will be set based on the resulting files.
     def split_with
-      Logger.info "Splitter configured with a chunk size of #{ chunk_size }MB " +
-                  "and suffix length of #{ suffix_length }."
+      Logger.info "Splitter configured with a chunk size of #{chunk_size}MB " \
+                  "and suffix length of #{suffix_length}."
       yield split_command
       after_packaging
     end
@@ -34,8 +32,8 @@ module Backup
     # multiple files, based on @chunk_size and @suffix_length, using the full
     # path to the final @package.basename, plus a '-' separator as the `prefix`.
     def split_command
-      "#{ utility(:split) } -a #{ suffix_length } -b #{ chunk_size }m - " +
-          "'#{ File.join(Config.tmp_path, package.basename + '-') }'"
+      "#{utility(:split)} -a #{suffix_length} -b #{chunk_size}m - " \
+          "'#{File.join(Config.tmp_path, package.basename + "-")}'"
     end
 
     ##
@@ -47,10 +45,10 @@ module Backup
     # remove the suffix from the filename.
     def after_packaging
       suffixes = chunk_suffixes
-      first_suffix = 'a' * suffix_length
+      first_suffix = "a" * suffix_length
       if suffixes == [first_suffix]
         FileUtils.mv(
-          File.join(Config.tmp_path, "#{ package.basename }-#{ first_suffix }"),
+          File.join(Config.tmp_path, "#{package.basename}-#{first_suffix}"),
           File.join(Config.tmp_path, package.basename)
         )
       else
@@ -62,15 +60,14 @@ module Backup
     # Returns an array of suffixes for each chunk, in alphabetical order.
     # For example: [aa, ab, ac, ad, ae] or [aaa, aab, aac aad]
     def chunk_suffixes
-      chunks.map {|chunk| File.extname(chunk).split('-').last }.sort
+      chunks.map { |chunk| File.extname(chunk).split("-").last }.sort
     end
 
     ##
     # Returns an array of full paths to the backup chunks.
     # Chunks are sorted in alphabetical order.
     def chunks
-      Dir[File.join(Config.tmp_path, package.basename + '-*')].sort
+      Dir[File.join(Config.tmp_path, package.basename + "-*")].sort
     end
-
   end
 end

data/lib/backup/storage/base.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Backup
   module Storage
     class Base
@@ -34,19 +32,19 @@ module Backup
       def initialize(model, storage_id = nil, &block)
         @model = model
         @package = model.package
-        @storage_id = storage_id.to_s.gsub(/\W/, '_') if storage_id
+        @storage_id = storage_id.to_s.gsub(/\W/, "_") if storage_id
 
         load_defaults!
         instance_eval(&block) if block_given?
       end
 
       def perform!
-        Logger.info "#{ storage_name } Started..."
+        Logger.info "#{storage_name} Started..."
         transfer!
         if respond_to?(:cycle!, true) && (keep.to_i > 0 || keep.is_a?(Time))
           cycle!
         end
-        Logger.info "#{ storage_name } Finished!"
+        Logger.info "#{storage_name} Finished!"
       end
 
       private
@@ -60,10 +58,9 @@ module Backup
       alias :remote_path_for :remote_path
 
       def storage_name
-        @storage_name ||= self.class.to_s.sub('Backup::', '') +
-            (storage_id ? " (#{ storage_id })" : '')
+        @storage_name ||= self.class.to_s.sub("Backup::", "") +
+          (storage_id ? " (#{storage_id})" : "")
       end
-
     end
   end
 end

data/lib/backup/storage/cloud_files.rb

@@ -1,5 +1,4 @@
-# encoding: utf-8
-require 'backup/cloud_io/cloud_files'
+require "backup/cloud_io/cloud_files"
 
 module Backup
   module Storage
@@ -79,8 +78,8 @@ module Backup
         @max_retries        ||= 10
         @retry_waitsec      ||= 30
 
-        @path ||= 'backups'
-        path.sub!(/^\//, '')
+        @path ||= "backups"
+        path.sub!(/^\//, "")
 
         check_configuration
       end
@@ -89,18 +88,18 @@ module Backup
 
       def cloud_io
         @cloud_io ||= CloudIO::CloudFiles.new(
-          :username           => username,
-          :api_key            => api_key,
-          :auth_url           => auth_url,
-          :region             => region,
-          :servicenet         => servicenet,
-          :container          => container,
-          :segments_container => segments_container,
-          :segment_size       => segment_size,
-          :days_to_keep       => days_to_keep,
-          :max_retries        => max_retries,
-          :retry_waitsec      => retry_waitsec,
-          :fog_options        => fog_options
+          username: username,
+          api_key: api_key,
+          auth_url: auth_url,
+          region: region,
+          servicenet: servicenet,
+          container: container,
+          segments_container: segments_container,
+          segment_size: segment_size,
+          days_to_keep: days_to_keep,
+          max_retries: max_retries,
+          retry_waitsec: retry_waitsec,
+          fog_options: fog_options
         )
       end
 
@@ -108,7 +107,7 @@ module Backup
         package.filenames.each do |filename|
           src = File.join(Config.tmp_path, filename)
           dest = File.join(remote_path, filename)
-          Logger.info "Storing '#{ container }/#{ dest }'..."
+          Logger.info "Storing '#{container}/#{dest}'..."
           cloud_io.upload(src, dest)
         end
 
@@ -118,12 +117,12 @@ module Backup
       # Called by the Cycler.
       # Any error raised will be logged as a warning.
       def remove!(package)
-        Logger.info "Removing backup package dated #{ package.time }..."
+        Logger.info "Removing backup package dated #{package.time}..."
 
         remote_path = remote_path_for(package)
         objects = cloud_io.objects(remote_path)
 
-        raise Error, "Package at '#{ remote_path }' not found" if objects.empty?
+        raise Error, "Package at '#{remote_path}' not found" if objects.empty?
 
         slo_objects, objects = objects.partition(&:slo?)
         cloud_io.delete_slo(slo_objects)
@@ -131,10 +130,10 @@ module Backup
       end
 
       def check_configuration
-        required = %w{ username api_key container }
-        raise Error, <<-EOS if required.map {|name| send(name) }.any?(&:nil?)
+        required = %w[username api_key container]
+        raise Error, <<-EOS if required.map { |name| send(name) }.any?(&:nil?)
           Configuration Error
-          #{ required.map {|name| "##{ name }"}.join(', ') } are all required
+          #{required.map { |name| "##{name}" }.join(", ")} are all required
         EOS
 
         raise Error, <<-EOS if segment_size > 0 && segments_container.to_s.empty?
@@ -152,7 +151,6 @@ module Backup
           #segment_size is too large (max 5120)
         EOS
       end
-
     end
   end
 end

data/lib/backup/storage/cycler.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 module Backup
   module Storage
     module Cycler
@@ -11,7 +9,7 @@ module Backup
       # and will remove any old package file(s) when the storage limit
       # set by #keep is exceeded.
       def cycle!
-        Logger.info 'Cycling Started...'
+        Logger.info "Cycling Started..."
 
         packages = yaml_load.unshift(package)
         cycled_packages = []
@@ -32,24 +30,22 @@ module Backup
       end
 
       def delete_package(package)
-        begin
-          remove!(package) unless package.no_cycle
-        rescue => err
-          Logger.warn Error.wrap(err, <<-EOS)
+        remove!(package) unless package.no_cycle
+      rescue => err
+        Logger.warn Error.wrap(err, <<-EOS)
             There was a problem removing the following package:
             Trigger: #{package.trigger} :: Dated: #{package.time}
-            Package included the following #{ package.filenames.count } file(s):
-            #{ package.filenames.join("\n") }
+            Package included the following #{package.filenames.count} file(s):
+            #{package.filenames.join("\n")}
           EOS
-        end
       end
 
       # Returns path to the YAML data file.
       def yaml_file
         @yaml_file ||= begin
-          filename = self.class.to_s.split('::').last
-          filename << "-#{ storage_id }" if storage_id
-          File.join(Config.data_path, package.trigger, "#{ filename }.yml")
+          filename = self.class.to_s.split("::").last
+          filename << "-#{storage_id}" if storage_id
+          File.join(Config.data_path, package.trigger, "#{filename}.yml")
         end
       end
 
@@ -65,11 +61,10 @@ module Backup
       # Stores the given package objects to the YAML data file.
       def yaml_save(packages)
         FileUtils.mkdir_p(File.dirname(yaml_file))
-        File.open(yaml_file, 'w') do |file|
+        File.open(yaml_file, "w") do |file|
           file.write(packages.to_yaml)
         end
       end
-
     end
   end
 end