foreman-tasks-core 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2d33e3c510526b078873ccdddbb5f164f9b6d0df
-  data.tar.gz: 4d6693cf71fd52035ed119e0ae71bf3d2bbd6dfe
+  metadata.gz: 7182c93db320880e8ab630c2edcfef6baa2f6ce0
+  data.tar.gz: b578ed3de2672e41910921ad1b5cfe0cc1872f6b
 SHA512:
-  metadata.gz: bd9a840a4ea81bdb19fd9f2538fd3ca943e21eeca329d08001ccbf99d01f8bc45ee2750eac1b10fb6c73dc0d56557d2309cf4458197327dacc3b9e44eef6710e
-  data.tar.gz: a1c757b94c63caaea74ce464e52150c64e305909900ab87afae20b0deca18a834ac8c8dbdd5e4f006b1d2a915229b04c374b47691bcb6b7a2187766c17ebf92e
+  metadata.gz: 4f8d31d820535c4ed018bad60c64710579f31761f3c0cd16477ee776233a0b98498f57821e72899275c20f9b2de8d3a780d5b4dcfa4ccd4e5db2a51b801a2d54
+  data.tar.gz: 9c6c676b311f5d9507765409df16d1c49c72f3da1fea223d9c3496312a62048219cf89197f81aa5787a505b4f7204eb8f6ee6d0e8fd7631badddb08d6e7d346a

data/lib/foreman_tasks_core.rb

@@ -5,7 +5,7 @@ require 'foreman_tasks_core/settings_loader'
 
 module ForemanTasksCore
   def self.dynflow_world
-    raise "Dynflow world not set. Call initialize first" unless @dynflow_world
+    raise 'Dynflow world not set. Call initialize first' unless @dynflow_world
     @dynflow_world
   end
 

data/lib/foreman_tasks_core/continuous_output.rb

@@ -8,7 +8,7 @@ module ForemanTasksCore
     end
 
     def add_raw_output(raw_output)
-      missing_args = %w[output_type output timestamp] - raw_output.keys
+      missing_args = %w(output_type output timestamp) - raw_output.keys
       unless missing_args.empty?
         raise ArgumentError, "Missing args for raw output: #{missing_args.inspect}"
       end

data/lib/foreman_tasks_core/runner/action.rb

@@ -17,12 +17,12 @@ module ForemanTasksCore
         end
       rescue => e
         action_logger.error(e)
-        process_update(Runner::Update.encode_exception("Proxy error", e))
+        process_update(Runner::Update.encode_exception('Proxy error', e))
       end
 
       def finalize
         # To mark the task as a whole as failed
-        error! "Script execution failed" if failed_run?
+        error! 'Script execution failed' if failed_run?
       end
 
       def rescue_strategy_for_self

data/lib/foreman_tasks_core/runner/base.rb

@@ -6,7 +6,7 @@ module ForemanTasksCore
       attr_reader :id
       attr_accessor :logger
 
-      def initialize(*args)
+      def initialize(*_args)
         @id = SecureRandom.uuid
         @continuous_output = ::ForemanTasksCore::ContinuousOutput.new
       end
@@ -16,7 +16,7 @@ module ForemanTasksCore
       end
 
       def run_refresh
-        logger.debug("refreshing runner")
+        logger.debug('refreshing runner')
         refresh
         new_data = @continuous_output
         @continuous_output = ForemanTasksCore::ContinuousOutput.new

data/lib/foreman_tasks_core/runner/command_runner.rb

@@ -17,7 +17,7 @@ module ForemanTasksCore
           else
             close_io
             Process.wait(@command_pid)
-            publish_exit_status($?.exitstatus)
+            publish_exit_status($CHILD_STATUS.exitstatus)
           end
           publish_data(lines, 'stdout') if lines && !lines.empty?
         end

data/lib/foreman_tasks_core/runner/dispatcher.rb

@@ -3,8 +3,8 @@ module ForemanTasksCore
     class Dispatcher
       def self.instance
         return @instance if @instance
-        @instance = self.new(ForemanTasksCore.dynflow_world.clock,
-                             ForemanTasksCore.dynflow_world.logger)
+        @instance = new(ForemanTasksCore.dynflow_world.clock,
+                        ForemanTasksCore.dynflow_world.logger)
       end
 
       class RunnerActor < ::Dynflow::Actor
@@ -34,7 +34,7 @@ module ForemanTasksCore
 
         def refresh_runner
           @logger.debug("refresh runner #{@runner.id}")
-          if update = @runner.run_refresh
+          if (update = @runner.run_refresh)
             @suspended_action << update
             finish if update.exit_status
           end
@@ -68,7 +68,7 @@ module ForemanTasksCore
         def plan_next_refresh
           if !@finishing && !@refresh_planned
             @logger.debug("planning to refresh #{@runner.id}")
-            @clock.ping(reference, Time.now + @refresh_interval, :refresh_runner)
+            @clock.ping(reference, Time.now.getlocal + @refresh_interval, :refresh_runner)
             @refresh_planned = true
           end
         end
@@ -137,7 +137,7 @@ module ForemanTasksCore
       def _finish(runner_id)
         runner_actor = @runner_actors.delete(runner_id)
         return unless runner_actor
-        @logger.debug("closing session for command [#{runner_id}]," +
+        @logger.debug("closing session for command [#{runner_id}]," \
                       "#{@runner_actors.size} actors left ")
         runner_actor.tell([:start_termination, Concurrent.future])
       ensure
@@ -149,7 +149,7 @@ module ForemanTasksCore
                       "#{exception.class} #{exception.message}:\n #{exception.backtrace.join("\n")}")
         suspended_action = @runner_suspended_actions[runner_id]
         if suspended_action
-          suspended_action << Runner::Update.encode_exception("Runner error", exception, fatal)
+          suspended_action << Runner::Update.encode_exception('Runner error', exception, fatal)
         end
         _finish(runner_id) if fatal
       end

data/lib/foreman_tasks_core/runner/update.rb

@@ -14,7 +14,7 @@ module ForemanTasksCore
       def self.encode_exception(context, exception, fatal = true)
         continuous_output = ::ForemanTasksCore::ContinuousOutput.new
         continuous_output.add_exception(context, exception)
-        return self.new(continuous_output, fatal ? 'EXCEPTION' : nil)
+        new(continuous_output, fatal ? 'EXCEPTION' : nil)
       end
     end
   end

data/lib/foreman_tasks_core/settings_loader.rb

@@ -19,7 +19,7 @@ module ForemanTasksCore
     def self.register_settings(names, object)
       names = [names] unless names.is_a? Array
       names.each do |name|
-        raise "settings name has to be a symbol" unless name.is_a? Symbol
+        raise 'settings name has to be a symbol' unless name.is_a? Symbol
         raise "settings #{name} already registered" if SettingsLoader.settings_registered?(name)
         name_to_settings[name] = object
       end
@@ -47,7 +47,7 @@ module ForemanTasksCore
     end
 
     def validate_settings!
-      raise "Only symbols expected in keys" unless @settings.keys.all? { |key| key.is_a? Symbol }
+      raise 'Only symbols expected in keys' unless @settings.keys.all? { |key| key.is_a? Symbol }
     end
   end
 end

data/lib/foreman_tasks_core/version.rb

@@ -1,3 +1,3 @@
 module ForemanTasksCore
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'.freeze
 end

metadata

@@ -1,17 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: foreman-tasks-core
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-12 00:00:00.000000000 Z
+date: 2017-03-29 00:00:00.000000000 Z
 dependencies: []
-description: |
-  Common code used both at Forman and Foreman proxy regarding tasks
+description: 'Common code used both at Forman and Foreman proxy regarding tasks
+
+'
 email:
 - inecas@redhat.com
 executables: []
@@ -49,7 +50,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Common code used both at Forman and Foreman proxy regarding tasks