foreman_ansible_core 2.1.1 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c69f61506d641e18d27a22a32514cdb6221174dd
-  data.tar.gz: 9afe8696e6723ad9ab66adf416b457e8fa9dfec8
+SHA256:
+  metadata.gz: 3a2a8033b0b49e4265b56fa56a0376ca02cde2f293a57c618039c60087014381
+  data.tar.gz: 98166355302fab2ab9d5624699cd6cfe6d880d6161b304b689f6410a1006c0f1
 SHA512:
-  metadata.gz: c40433c92f4c4f3d617562cad983a0b116b8240830bb14d6d641aa17187845dc1649a3e4da75f81af6485b66efffc69f8fc2a431212e938989c10fa54604ef3b
-  data.tar.gz: 7080dd69b55873db5305ba237dc7d5e7331cecd1e23c2969125eaf553e115573663205bf0fa46b5331686661330150c44a7dec6fc5278b5f9070498f87e83c5e
+  metadata.gz: 4ec94ef02708b8dd9403e630abf2f3f82a65503124dbcef17ee4f3ca9e78e663d9dd8b2ce89ea43d37641ec13c5b81cd2c96d25dbf7bf3b13b8fbad5d189631e
+  data.tar.gz: 199d21b8a88c256ece9fccc356f580f46d2334fbd87b7310f39043ebdce7560e37d10338ef73b34f4c7af24ed8940b556314faeba74c5206e8fa453a0ce7542c

data/lib/foreman_ansible_core.rb

@@ -1,39 +1,34 @@
-begin
-  require 'foreman_tasks_core'
-  require 'foreman_remote_execution_core'
-# rubocop:disable Lint/HandleExceptions
-rescue LoadError
-  # These gems are not available in a proxy SCLed context
-  # puts 'Running Foreman Ansible Core in non-SCL context'
-end
-# rubocop:enable Lint/HandleExceptions
+# frozen_string_literal: true
+
+require 'foreman_tasks_core'
+require 'foreman_remote_execution_core'
 
 # Core actions for Foreman Ansible, used by both Foreman and Foreman proxy
 # This comprises running playbooks for the moment
 module ForemanAnsibleCore
   require 'foreman_ansible_core/exception'
-  require 'foreman_ansible_core/roles_reader'
   require 'foreman_ansible_core/version'
 
-  if defined? ForemanTasksCore
-    extend ForemanTasksCore::SettingsLoader
-    register_settings(:ansible, :ansible_dir => Dir.home,
-                                :working_dir => nil)
+  extend ForemanTasksCore::SettingsLoader
+  register_settings(:ansible, :ansible_dir => Dir.home,
+                              :working_dir => nil)
 
-    if ForemanTasksCore.dynflow_present?
-      require 'foreman_tasks_core/runner'
-      require 'foreman_ansible_core/playbook_runner'
-      require 'foreman_ansible_core/actions'
-    end
+  if ForemanTasksCore.dynflow_present?
+    require 'foreman_tasks_core/runner'
+    require 'foreman_ansible_core/runner/playbook'
+    require 'foreman_ansible_core/runner/ansible_runner'
+    require 'foreman_ansible_core/actions'
   end
 
-  if defined? ForemanTasksCore
-    require 'foreman_remote_execution_core/actions'
-    require 'foreman_ansible_core/remote_execution_core/ansible_runner'
-    require 'foreman_ansible_core/remote_execution_core/settings_override'
-    ForemanRemoteExecutionCore::Actions::RunScript.send(
-      :prepend,
-      ForemanAnsibleCore::RemoteExecutionCore::SettingsOverride
-    )
+  require 'foreman_remote_execution_core/actions'
+  require 'foreman_ansible_core/remote_execution_core/ansible_runner'
+  require 'foreman_ansible_core/task_launcher/playbook'
+  require 'foreman_ansible_core/task_launcher/ansible_runner'
+
+  if defined?(SmartProxyDynflowCore)
+    SmartProxyDynflowCore::TaskLauncherRegistry.register('ansible-runner',
+                                                         ForemanAnsibleCore::TaskLauncher::AnsibleRunner)
+    SmartProxyDynflowCore::TaskLauncherRegistry.register('ansible-playbook',
+                                                         ForemanAnsibleCore::TaskLauncher::Playbook)
   end
 end

data/lib/foreman_ansible_core/actions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'foreman_tasks_core/shareable_action'
 
 module ForemanAnsibleCore
@@ -6,7 +8,7 @@ module ForemanAnsibleCore
     # to execute the playbook run
     class RunPlaybook < ForemanTasksCore::Runner::Action
       def initiate_runner
-        ForemanAnsibleCore::PlaybookRunner.new(
+        ForemanAnsibleCore::Runner::Playbook.new(
           input[:inventory],
           input[:playbook],
           input[:options]

data/lib/foreman_ansible_core/exception.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ForemanAnsibleCore
   # Taken from Foreman core, this class creates an error code for any exception
   class Exception < ::StandardError

data/lib/foreman_ansible_core/remote_execution_core/ansible_runner.rb

@@ -1,19 +1,24 @@
+# frozen_string_literal: true
+
 module ForemanAnsibleCore
   module RemoteExecutionCore
     # Takes an inventory and runs it through REXCore CommandRunner
     class AnsibleRunner < ::ForemanTasksCore::Runner::CommandRunner
       DEFAULT_REFRESH_INTERVAL = 1
+      CONNECTION_PROMPT = 'Are you sure you want to continue connecting (yes/no)? '
 
-      def initialize(options)
-        super(options)
-        @playbook_runner = ForemanAnsibleCore::PlaybookRunner.new(
+      def initialize(options, suspended_action:)
+        super(options, :suspended_action => suspended_action)
+        @playbook_runner = ForemanAnsibleCore::Runner::Playbook.new(
           options['ansible_inventory'],
           options['script'],
-          options
+          options,
+          :suspended_action => suspended_action
         )
       end
 
       def start
+        @playbook_runner.logger = logger
         @playbook_runner.start
       rescue StandardError => e
         logger.error(
@@ -36,6 +41,21 @@ module ForemanAnsibleCore
         @command_in = @playbook_runner.command_in
         @command_pid = @playbook_runner.command_pid
         super
+        kill if unknown_host_key_fingerprint?
+      end
+
+      def kill
+        publish_exit_status(1)
+        ::Process.kill('SIGTERM', @command_pid)
+        close
+      end
+
+      private
+
+      def unknown_host_key_fingerprint?
+        last_output = @continuous_output.raw_outputs.last
+        return if last_output.nil?
+        last_output['output']&.lines&.last == CONNECTION_PROMPT
       end
     end
   end

data/lib/foreman_ansible_core/runner/ansible_runner.rb

@@ -0,0 +1,143 @@
+module ForemanAnsibleCore
+  module Runner
+    class AnsibleRunner < ForemanTasksCore::Runner::Parent
+      include ForemanTasksCore::Runner::Command
+
+      def initialize(input, suspended_action:)
+        super input, :suspended_action => suspended_action
+        @inventory = rebuild_inventory(input)
+        @playbook = input.values.first[:input][:action_input][:script]
+        @root = working_dir
+      end
+
+      def start
+        prepare_directory_structure
+        write_inventory
+        write_playbook
+        start_ansible_runner
+      end
+
+      def refresh
+        return unless super
+        @counter ||= 1
+        @uuid ||= File.basename(Dir["#{@root}/artifacts/*"].first)
+        job_event_dir = File.join(@root, 'artifacts', @uuid, 'job_events')
+        loop do
+          files = Dir["#{job_event_dir}/*.json"].map do |file|
+            num = File.basename(file)[/\A\d+/].to_i unless file.include?('partial')
+            [file, num]
+          end
+          files_with_nums = files.select { |(_, num)| num && num >= @counter }.sort_by(&:last)
+          break if files_with_nums.empty?
+          logger.debug("[foreman_ansible] - processing event files: #{files_with_nums.map(&:first).inspect}}")
+          files_with_nums.map(&:first).each { |event_file| handle_event_file(event_file) }
+          @counter = files_with_nums.last.last + 1
+        end
+      end
+
+      private
+
+      def handle_event_file(event_file)
+        logger.debug("[foreman_ansible] - parsing event file #{event_file}")
+        begin
+          event = JSON.parse(File.read(event_file))
+          if (hostname = event['event_data']['host'])
+            handle_host_event(hostname, event)
+          else
+            handle_broadcast_data(event)
+          end
+          true
+        rescue JSON::ParserError => e
+          logger.error("[foreman_ansible] - Error parsing runner event at #{event_file}: #{e.class}: #{e.message}")
+          logger.debug(e.backtrace.join("\n"))
+        end
+      end
+
+      def handle_host_event(hostname, event)
+        log_event("for host: #{hostname.inspect}", event)
+        publish_data_for(hostname, event['stdout'] + "\n", 'stdout') if event['stdout']
+        case event['event']
+        when 'runner_on_ok'
+          publish_exit_status_for(hostname, 0) if @exit_statuses[hostname].nil?
+        when 'runner_on_unreachable'
+          publish_exit_status_for(hostname, 1)
+        when 'runner_on_failed'
+          publish_exit_status_for(hostname, 2) if event['ignore_errors'].nil?
+        end
+      end
+
+      def handle_broadcast_data(event)
+        log_event("broadcast", event)
+        if event['event'] == 'playbook_on_stats'
+          header, *rows = event['stdout'].strip.lines.map(&:chomp)
+          @outputs.keys.select { |key| key.is_a? String }.each do |host|
+            line = rows.find { |row| row =~ /#{host}/ }
+            publish_data_for(host, [header, line].join("\n"), 'stdout')
+          end
+        else
+          broadcast_data(event['stdout'] + "\n", 'stdout')
+        end
+      end
+
+      def write_inventory
+        inventory_script = <<~INVENTORY_SCRIPT
+          #!/bin/sh
+          cat <<-EOS
+          #{JSON.dump(@inventory)}
+          EOS
+        INVENTORY_SCRIPT
+        path = File.join(@root, 'inventory', 'hosts')
+        File.write(path, inventory_script)
+        File.chmod(0o0755, path)
+      end
+
+      def write_playbook
+        File.write(File.join(@root, 'project', 'playbook.yml'), @playbook)
+      end
+
+      def start_ansible_runner
+        command = ['ansible-runner', 'run', @root, '-p', 'playbook.yml']
+        initialize_command(*command)
+        logger.debug("[foreman_ansible] - Running command '#{command.join(' ')}'")
+      end
+
+      def prepare_directory_structure
+        inner = %w[inventory project].map { |part| File.join(@root, part) }
+        ([@root] + inner).each do |path|
+          FileUtils.mkdir_p path
+        end
+      end
+
+      def log_event(description, event)
+        # TODO: replace this ugly code with block variant once https://github.com/Dynflow/dynflow/pull/323
+        # arrives in production
+        logger.debug("[foreman_ansible] - handling event #{description}: #{JSON.pretty_generate(event)}") if logger.level <= ::Logger::DEBUG
+      end
+
+      # Each per-host task has inventory only for itself, we must
+      # collect all the partial inventories into one large inventory
+      # containing all the hosts.
+      def rebuild_inventory(input)
+        action_inputs = input.values.map { |hash| hash[:input][:action_input] }
+        hostnames = action_inputs.map { |hash| hash[:name] }
+        inventories = action_inputs.map { |hash| JSON.parse(hash[:ansible_inventory]) }
+        host_vars = inventories.map { |i| i['_meta']['hostvars'] }.reduce(&:merge)
+
+        { '_meta' => { 'hostvars' => host_vars },
+          'all' => { 'hosts' => hostnames,
+                     'vars' => inventories.first['all']['vars'] } }
+      end
+
+      def working_dir
+        return @root if @root
+        dir = ForemanAnsibleCore.settings[:working_dir]
+        @tmp_working_dir = true
+        if dir.nil?
+          Dir.mktmpdir
+        else
+          Dir.mktmpdir(nil, File.expand_path(dir))
+        end
+      end
+    end
+  end
+end

data/lib/foreman_ansible_core/{command_creator.rb → runner/command_creator.rb}

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ForemanAnsibleCore
   # Creates the actual command to be passed to foreman_tasks_core to run
   class CommandCreator
@@ -32,7 +34,7 @@ module ForemanAnsibleCore
     end
 
     def json_inventory_script
-      File.expand_path('../../bin/json_inventory.sh', File.dirname(__FILE__))
+      File.expand_path('../../../bin/json_inventory.sh', File.dirname(__FILE__))
     end
 
     def setup_verbosity
@@ -48,7 +50,7 @@ module ForemanAnsibleCore
       verbosity_level = @options[:verbosity_level]
       # rubocop:disable Rails/Present
       !verbosity_level.nil? && !verbosity_level.empty? &&
-        verbosity_level.to_i > 0
+        verbosity_level.to_i.positive?
       # rubocop:enable Rails/Present
     end
 

data/lib/foreman_ansible_core/runner/playbook.rb

@@ -0,0 +1,123 @@
+# frozen_string_literal: true
+
+require 'foreman_tasks_core/runner/command_runner'
+require_relative 'command_creator'
+require 'tmpdir'
+require 'net/ssh'
+
+module ForemanAnsibleCore
+  module Runner
+    # Implements ForemanTasksCore::Runner::Base interface for running
+    # Ansible playbooks, used by the Foreman Ansible plugin and Ansible proxy
+    class Playbook < ForemanTasksCore::Runner::CommandRunner
+      attr_reader :command_out, :command_in, :command_pid
+
+      def initialize(inventory, playbook, options = {}, suspended_action:)
+        super :suspended_action => suspended_action
+        @inventory = inventory
+        unknown_hosts.each do |host|
+          add_to_known_hosts(host)
+        end
+        @playbook  = playbook
+        @options   = options
+        initialize_dirs
+      end
+
+      def start
+        write_inventory
+        write_playbook
+        command = CommandCreator.new(inventory_file,
+                                     playbook_file,
+                                     @options).command
+        logger.debug('[foreman_ansible] - Initializing Ansible Runner')
+        Dir.chdir(@ansible_dir) do
+          initialize_command(*command)
+          logger.debug("[foreman_ansible] - Running command '#{command.join(' ')}'")
+        end
+      end
+
+      def kill
+        publish_data('== TASK ABORTED BY USER ==', 'stdout')
+        publish_exit_status(1)
+        ::Process.kill('SIGTERM', @command_pid)
+        close
+      end
+
+      def close
+        super
+        FileUtils.remove_entry(@working_dir) if @tmp_working_dir
+      end
+
+      private
+
+      def write_inventory
+        ensure_directory(File.dirname(inventory_file))
+        File.write(inventory_file, @inventory)
+      end
+
+      def write_playbook
+        ensure_directory(File.dirname(playbook_file))
+        File.write(playbook_file, @playbook)
+      end
+
+      def inventory_file
+        File.join(@working_dir, 'foreman-inventories', id)
+      end
+
+      def playbook_file
+        File.join(@working_dir, "foreman-playbook-#{id}.yml")
+      end
+
+      def events_dir
+        File.join(@working_dir, 'events', id.to_s)
+      end
+
+      def ensure_directory(path)
+        if File.exist?(path)
+          raise "#{path} expected to be a directory" unless File.directory?(path)
+        else
+          FileUtils.mkdir_p(path)
+        end
+        path
+      end
+
+      def initialize_dirs
+        settings = ForemanAnsibleCore.settings
+        initialize_working_dir(settings[:working_dir])
+        initialize_ansible_dir(settings[:ansible_dir])
+      end
+
+      def initialize_working_dir(working_dir)
+        if working_dir.nil?
+          @working_dir = Dir.mktmpdir
+          @tmp_working_dir = true
+        else
+          @working_dir = File.expand_path(working_dir)
+        end
+      end
+
+      def initialize_ansible_dir(ansible_dir)
+        raise "Ansible dir #{ansible_dir} does not exist" unless
+          !ansible_dir.nil? && File.exist?(ansible_dir)
+        @ansible_dir = ansible_dir
+      end
+
+      def unknown_hosts
+        JSON.parse(@inventory)['all']['hosts'].select do |host|
+          Net::SSH::KnownHosts.search_for(host).empty?
+        end
+      end
+
+      def add_to_known_hosts(host)
+        logger.warn("[foreman_ansible] - Host #{host} not found in known_hosts")
+        Net::SSH::Transport::Session.new(host).host_keys.each do |host_key|
+          Net::SSH::KnownHosts.add(host, host_key)
+        end
+        logger.warn("[foreman_ansible] - Added host key #{host} to known_hosts")
+      rescue StandardError => e
+        logger.error('[foreman_ansible] - Failed to save host key for '\
+          "#{host}: #{e}")
+      end
+    end
+  end
+end

data/lib/foreman_ansible_core/task_launcher/ansible_runner.rb

@@ -0,0 +1,38 @@
+require 'fileutils'
+
+module ForemanAnsibleCore
+  module TaskLauncher
+    class AnsibleRunner < ForemanTasksCore::TaskLauncher::AbstractGroup
+      def runner_input(input)
+        super(input).reduce({}) do |acc, (_id, data)|
+          acc.merge(data[:input]['action_input']['name'] => data)
+        end
+      end
+
+      def operation
+        'ansible-runner'
+      end
+
+      def self.runner_class
+        Runner::AnsibleRunner
+      end
+
+      # def self.input_format
+      #   {
+      #     $UUID => {
+      #       :execution_plan_id => $EXECUTION_PLAN_UUID,
+      #       :run_step_id => Integer,
+      #       :input => {
+      #         :action_class => Class,
+      #         :action_input => {
+      #           "ansible_inventory"=> String,
+      #           "hostname"=>"127.0.0.1",
+      #           "script"=>"---\n- hosts: all\n  tasks:\n    - shell: |\n        true\n      register: out\n    - debug: var=out"
+      #         }
+      #       }
+      #     }
+      #   }
+      # end
+    end
+  end
+end

data/lib/foreman_ansible_core/task_launcher/playbook.rb

@@ -0,0 +1,23 @@
+module ForemanAnsibleCore
+  module TaskLauncher
+    class Playbook < ForemanTasksCore::TaskLauncher::Batch
+      class PlaybookRunnerAction < ForemanTasksCore::Runner::Action
+        def initiate_runner
+          additional_options = {
+            :step_id => run_step_id,
+            :uuid => execution_plan_id
+          }
+          ::ForemanAnsibleCore::RemoteExecutionCore::AnsibleRunner.new(
+            input.merge(additional_options),
+            :suspended_action => suspended_action
+          )
+        end
+      end
+
+      def child_launcher(parent)
+        ForemanTasksCore::TaskLauncher::Single.new(world, callback, :parent => parent,
+                                                                    :action_class_override => PlaybookRunnerAction)
+      end
+    end
+  end
+end

data/lib/foreman_ansible_core/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ForemanAnsibleCore
-  VERSION = '2.1.1'.freeze
+  VERSION = '3.0.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible_core
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 3.0.1
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-19 00:00:00.000000000 Z
+date: 2019-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: 0.3.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: 0.3.2
 - !ruby/object:Gem::Dependency
   name: foreman_remote_execution_core
   requirement: !ruby/object:Gem::Requirement
@@ -56,16 +56,16 @@ dependencies:
   name: net-ssh
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
 description: |2
       Ansible integration with Foreman - core parts for dealing with Ansible concepts,
       usable by foreman_ansible or smart_proxy_ansible to delegate the execution.
@@ -79,12 +79,13 @@ files:
 - bin/json_inventory.sh
 - lib/foreman_ansible_core.rb
 - lib/foreman_ansible_core/actions.rb
-- lib/foreman_ansible_core/command_creator.rb
 - lib/foreman_ansible_core/exception.rb
-- lib/foreman_ansible_core/playbook_runner.rb
 - lib/foreman_ansible_core/remote_execution_core/ansible_runner.rb
-- lib/foreman_ansible_core/remote_execution_core/settings_override.rb
-- lib/foreman_ansible_core/roles_reader.rb
+- lib/foreman_ansible_core/runner/ansible_runner.rb
+- lib/foreman_ansible_core/runner/command_creator.rb
+- lib/foreman_ansible_core/runner/playbook.rb
+- lib/foreman_ansible_core/task_launcher/ansible_runner.rb
+- lib/foreman_ansible_core/task_launcher/playbook.rb
 - lib/foreman_ansible_core/version.rb
 homepage: https://github.com/theforeman/foreman_ansible
 licenses:
@@ -105,8 +106,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: 'Ansible integration with Foreman (theforeman.org): core bits'

data/lib/foreman_ansible_core/playbook_runner.rb

@@ -1,119 +0,0 @@
-require 'foreman_tasks_core/runner/command_runner'
-require_relative 'command_creator'
-require 'tmpdir'
-require 'net/ssh'
-
-module ForemanAnsibleCore
-  # Implements ForemanTasksCore::Runner::Base interface for running
-  # Ansible playbooks, used by the Foreman Ansible plugin and Ansible proxy
-  class PlaybookRunner < ForemanTasksCore::Runner::CommandRunner
-    attr_reader :command_out, :command_in, :command_pid
-
-    def initialize(inventory, playbook, options = {})
-      super
-      @inventory = inventory
-      unknown_hosts.each do |host|
-        add_to_known_hosts(host)
-      end
-      @playbook  = playbook
-      @options   = options
-      initialize_dirs
-    end
-
-    def start
-      write_inventory
-      write_playbook
-      command = CommandCreator.new(inventory_file,
-                                   playbook_file,
-                                   @options).command
-      logger.debug('[foreman_ansible] - Initializing Ansible Runner')
-      Dir.chdir(@ansible_dir) do
-        initialize_command(*command)
-        logger.debug("[foreman_ansible] - Running command #{command}")
-      end
-    end
-
-    def kill
-      publish_data('== TASK ABORTED BY USER ==', 'stdout')
-      publish_exit_status(1)
-      ::Process.kill('SIGTERM', @command_pid)
-      close
-    end
-
-    def close
-      super
-      FileUtils.remove_entry(@working_dir) if @tmp_working_dir
-    end
-
-    private
-
-    def write_inventory
-      ensure_directory(File.dirname(inventory_file))
-      File.write(inventory_file, @inventory)
-    end
-
-    def write_playbook
-      ensure_directory(File.dirname(playbook_file))
-      File.write(playbook_file, @playbook)
-    end
-
-    def inventory_file
-      File.join(@working_dir, 'foreman-inventories', id)
-    end
-
-    def playbook_file
-      File.join(@working_dir, "foreman-playbook-#{id}.yml")
-    end
-
-    def events_dir
-      File.join(@working_dir, 'events', id.to_s)
-    end
-
-    def ensure_directory(path)
-      if File.exist?(path)
-        raise "#{path} expected to be a directory" unless File.directory?(path)
-      else
-        FileUtils.mkdir_p(path)
-      end
-      path
-    end
-
-    def initialize_dirs
-      settings = ForemanAnsibleCore.settings
-      initialize_working_dir(settings[:working_dir])
-      initialize_ansible_dir(settings[:ansible_dir])
-    end
-
-    def initialize_working_dir(working_dir)
-      if working_dir.nil?
-        @working_dir = Dir.mktmpdir
-        @tmp_working_dir = true
-      else
-        @working_dir = File.expand_path(working_dir)
-      end
-    end
-
-    def initialize_ansible_dir(ansible_dir)
-      raise "Ansible dir #{ansible_dir} does not exist" unless
-        !ansible_dir.nil? && File.exist?(ansible_dir)
-      @ansible_dir = ansible_dir
-    end
-
-    def unknown_hosts
-      JSON.parse(@inventory)['all']['hosts'].select do |host|
-        Net::SSH::KnownHosts.search_for(host).empty?
-      end
-    end
-
-    def add_to_known_hosts(host)
-      logger.warn("[foreman_ansible] - Host #{host} not found in known_hosts")
-      Net::SSH::Transport::Session.new(host).host_keys.each do |host_key|
-        Net::SSH::KnownHosts.add(host, host_key)
-      end
-      logger.warn("[foreman_ansible] - Added host key #{host} to known_hosts")
-    rescue StandardError => e
-      logger.error('[foreman_ansible] - Failed to save host key for '\
-                   "#{host}: #{e}")
-    end
-  end
-end

data/lib/foreman_ansible_core/remote_execution_core/settings_override.rb

@@ -1,18 +0,0 @@
-module ForemanAnsibleCore
-  module RemoteExecutionCore
-    # Ensure the Ansible provider is used whenever a JobTemplate using this
-    # provider is called.
-    module SettingsOverride
-      def initiate_runner
-        return super unless input['ansible_inventory']
-        additional_options = {
-          :step_id => run_step_id,
-          :uuid => execution_plan_id
-        }
-        ::ForemanAnsibleCore::RemoteExecutionCore::AnsibleRunner.new(
-          input.merge(additional_options)
-        )
-      end
-    end
-  end
-end

data/lib/foreman_ansible_core/roles_reader.rb

@@ -1,61 +0,0 @@
-module ForemanAnsibleCore
-  # Implements the logic needed to read the roles and associated information
-  class RolesReader
-    class << self
-      DEFAULT_CONFIG_FILE = '/etc/ansible/ansible.cfg'.freeze
-      DEFAULT_ROLES_PATH = '/etc/ansible/roles'.freeze
-
-      def list_roles
-        roles_path.split(':').map { |path| read_roles(path) }.flatten
-      end
-
-      def roles_path(roles_line = roles_path_from_config)
-        # Default to /etc/ansible/roles if none found
-        return DEFAULT_ROLES_PATH if roles_line.empty?
-        roles_path_key = roles_line.first.split('=').first.strip
-        # In case of commented roles_path key "#roles_path", return default
-        return DEFAULT_ROLES_PATH unless roles_path_key == 'roles_path'
-        roles_line.first.split('=').last.strip
-      end
-
-      def logger
-        # Return a different logger depending on where ForemanAnsibleCore is
-        # running from
-        if defined?(::Foreman::Logging)
-          ::Foreman::Logging.logger('foreman_ansible')
-        else
-          ::Proxy::LogBuffer::Decorator.instance
-        end
-      end
-
-      private
-
-      def read_roles(roles_path)
-        rescue_and_raise_file_exception ReadRolesException,
-                                        roles_path, 'roles' do
-          Dir.glob("#{roles_path}/*").map do |path|
-            path.split('/').last
-          end
-        end
-      end
-
-      def roles_path_from_config
-        rescue_and_raise_file_exception ReadConfigFileException,
-                                        DEFAULT_CONFIG_FILE, 'config file' do
-          File.readlines(DEFAULT_CONFIG_FILE).select do |line|
-            line =~ /roles_path/
-          end
-        end
-      end
-
-      def rescue_and_raise_file_exception(exception, path, type)
-        yield
-      rescue Errno::ENOENT, Errno::EACCES => e
-        logger.debug(e.backtrace)
-        exception_message = "Could not read Ansible #{type} "\
-                            "#{path} - #{e.message}"
-        raise exception.new(exception_message)
-      end
-    end
-  end
-end