foreman_remote_execution_core 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 80ae26d6b169ea807ed8b6eea817a6797dde93a11fff3ee5a29cf6794b5ea378
-  data.tar.gz: ccb07c483e5a2973c11641cb1bc6e6385fc2b33088a4fadbeb41fff60ef2fdff
+  metadata.gz: e1576329ece4e20e9506b989b77557686d030c329c71f70386c7415e93a73e24
+  data.tar.gz: 4021d34ddbaa469bb1f4152c0a3c404c976a97910422a6fd60868661992ab774
 SHA512:
-  metadata.gz: 9f9b4d32165279cc36b5c59a7fa901a239c7a67643d4542b6713d1569624107f5e79091d16916a2a9695e20d78b405d1255c6f363e6130b6e0b73903e55deae3
-  data.tar.gz: 651c3382363f5684cb2c8f62ce16582cfe8b51dbb9479addb3cd8282c2d90d2e658c2e831a563ffce1cf3f67d5ac2884e2bb1c2eca0f836b2fce1ffe85edd1f3
+  metadata.gz: b0ce710cc252baca86acb51cfe294b76f8de319b5516bc8f303252fff52a8ef1e074c34ea00f7bd179b2811dc85daf292035a2736cb421046aa46e2a37137d73
+  data.tar.gz: 65092102cb80436b6de9ce165905e8d87c387bf70256b244224d780cd2335f43b0323a06afed5c72e78fa8297566dcbe01f0624dee89b25645d8e2bf292d91a4

data/lib/foreman_remote_execution_core.rb

@@ -3,16 +3,16 @@ require 'foreman_tasks_core'
 module ForemanRemoteExecutionCore
   extend ForemanTasksCore::SettingsLoader
   register_settings([:remote_execution_ssh, :smart_proxy_remote_execution_ssh_core],
-                    :ssh_identity_key_file   => '~/.ssh/id_rsa_foreman_proxy',
-                    :ssh_user                => 'root',
-                    :remote_working_dir      => '/var/tmp',
-                    :local_working_dir       => '/var/tmp',
-                    :kerberos_auth           => false,
-                    :async_ssh               => false,
-                    # When set to nil, makes REX use the runner's default interval
-                    :runner_refresh_interval => nil,
-                    :ssh_log_level           => :fatal,
-                    :cleanup_working_dirs    => true)
+    :ssh_identity_key_file   => '~/.ssh/id_rsa_foreman_proxy',
+    :ssh_user                => 'root',
+    :remote_working_dir      => '/var/tmp',
+    :local_working_dir       => '/var/tmp',
+    :kerberos_auth           => false,
+    :async_ssh               => false,
+    # When set to nil, makes REX use the runner's default interval
+    :runner_refresh_interval => nil,
+    :ssh_log_level           => :fatal,
+    :cleanup_working_dirs    => true)
 
   SSH_LOG_LEVELS = %w(debug info warn error fatal).freeze
 
@@ -32,7 +32,9 @@ module ForemanRemoteExecutionCore
       raise "Wrong value '#{@settings[:ssh_log_level]}' for ssh_log_level, must be one of #{SSH_LOG_LEVELS.join(', ')}"
     end
 
-    current = if defined?(SmartProxyDynflowCore)
+    current = if defined?(::Proxy::SETTINGS)
+                ::Proxy::SETTINGS.log_level.to_s.downcase
+              elsif defined?(SmartProxyDynflowCore)
                 SmartProxyDynflowCore::SETTINGS.log_level.to_s.downcase
               else
                 Rails.configuration.log_level.to_s

data/lib/foreman_remote_execution_core/actions.rb

@@ -6,10 +6,10 @@ module ForemanRemoteExecutionCore
       def initiate_runner
         additional_options = {
           :step_id => run_step_id,
-          :uuid => execution_plan_id
+          :uuid => execution_plan_id,
         }
         ForemanRemoteExecutionCore.runner_class.build(input.merge(additional_options),
-                                                      suspended_action: suspended_action)
+          suspended_action: suspended_action)
       end
 
       def runner_dispatcher

data/lib/foreman_remote_execution_core/log_filter.rb

@@ -4,10 +4,11 @@ module ForemanRemoteExecutionCore
       @base_logger = base_logger
     end
 
-    def add(severity, *args)
+    def add(severity, *args, &block)
       severity ||= ::Logger::UNKNOWN
       return true if @base_logger.nil? || severity < @level
-      @base_logger.add(severity, *args)
+
+      @base_logger.add(severity, *args, &block)
     end
   end
 end

data/lib/foreman_remote_execution_core/polling_script_runner.rb

@@ -58,6 +58,7 @@ module ForemanRemoteExecutionCore
         @logger.info("Error while connecting to the remote host on refresh: #{e.message}")
       end
       return if output.nil? || output.empty?
+
       lines = output.lines
       result = lines.shift.match(/^DONE (\d+)?/)
       publish_data(lines.join, 'stdout') unless lines.empty?
@@ -90,6 +91,7 @@ module ForemanRemoteExecutionCore
 
     def upload_control_scripts
       return if @control_scripts_uploaded
+
       cp_script_to_remote(env_script, 'env.sh')
       @control_script_path = cp_script_to_remote(CONTROL_SCRIPT, 'control.sh')
       @retrieval_script = cp_script_to_remote(RETRIEVE_SCRIPT, 'retrieve.sh')

data/lib/foreman_remote_execution_core/script_runner.rb

@@ -1,11 +1,11 @@
 require 'net/ssh'
 require 'fileutils'
 
-# rubocop:disable Lint/HandleExceptions
+# rubocop:disable Lint/SuppressedException
 begin
   require 'net/ssh/krb'
 rescue LoadError; end
-# rubocop:enable Lint/HandleExceptions:
+# rubocop:enable Lint/SuppressedException:
 
 module ForemanRemoteExecutionCore
   class SudoUserMethod
@@ -32,7 +32,7 @@ module ForemanRemoteExecutionCore
     end
 
     def filter_password?(received_data)
-      !@effective_user_password.empty? && @password_sent && received_data.match(@effective_user_password)
+      !@effective_user_password.empty? && @password_sent && received_data.match(Regexp.escape(@effective_user_password))
     end
 
     def sent_all_data?
@@ -68,7 +68,8 @@ module ForemanRemoteExecutionCore
       @ssh_user = ssh_user
     end
 
-    def on_data(_, _); end
+    def on_data(_, _)
+    end
 
     def filter_password?(received_data)
       false
@@ -82,11 +83,13 @@ module ForemanRemoteExecutionCore
       "su - #{effective_user} -c "
     end
 
-    def reset; end
+    def reset
+    end
   end
 
   class NoopUserMethod
-    def on_data(_, _); end
+    def on_data(_, _)
+    end
 
     def filter_password?(received_data)
       false
@@ -96,9 +99,11 @@ module ForemanRemoteExecutionCore
       true
     end
 
-    def cli_command_prefix; end
+    def cli_command_prefix
+    end
 
-    def reset; end
+    def reset
+    end
   end
 
   class ScriptRunner < ForemanTasksCore::Runner::Base
@@ -136,10 +141,10 @@ module ForemanRemoteExecutionCore
                       NoopUserMethod.new
                     elsif effective_user_method == 'sudo'
                       SudoUserMethod.new(effective_user, ssh_user,
-                                         options.fetch(:secrets, {}).fetch(:sudo_password, nil))
+                        options.fetch(:secrets, {}).fetch(:sudo_password, nil))
                     elsif effective_user_method == 'dzdo'
                       DzdoUserMethod.new(effective_user, ssh_user,
-                                         options.fetch(:secrets, {}).fetch(:sudo_password, nil))
+                        options.fetch(:secrets, {}).fetch(:sudo_password, nil))
                     elsif effective_user_method == 'su'
                       SuUserMethod.new(effective_user, ssh_user)
                     else
@@ -182,6 +187,7 @@ module ForemanRemoteExecutionCore
 
     def refresh
       return if @session.nil?
+
       with_retries do
         with_disconnect_handling do
           @session.process(0)
@@ -295,6 +301,7 @@ module ForemanRemoteExecutionCore
     # part of calling the `refresh` method.
     def run_async(command)
       raise 'Async command already in progress' if @started
+
       @started = false
       @user_method.reset
 
@@ -350,6 +357,7 @@ module ForemanRemoteExecutionCore
         end
         ch.exec command do |_, success|
           raise 'could not execute command' unless success
+
           started = true
         end
       end
@@ -413,6 +421,7 @@ module ForemanRemoteExecutionCore
       if status != 0
         raise "Unable to upload file to #{path} on remote system: exit code: #{status}"
       end
+
       path
     end
 
@@ -446,6 +455,7 @@ module ForemanRemoteExecutionCore
     def check_expecting_disconnect
       last_output = @continuous_output.raw_outputs.find { |d| d['output_type'] == 'stdout' }
       return unless last_output
+
       if EXPECTED_POWER_ACTION_MESSAGES.any? { |message| last_output['output'] =~ /^#{message}/ }
         @expecting_disconnect = true
       end

data/lib/foreman_remote_execution_core/version.rb

@@ -1,3 +1,3 @@
 module ForemanRemoteExecutionCore
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.3.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution_core
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-04 00:00:00.000000000 Z
+date: 2020-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foreman-tasks-core