foreman_ansible 5.1.1 → 6.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0aa6750e2a4a664a8751b884adee730fbb792221e02596a334e3d6aa3955f84
-  data.tar.gz: f7e9287de112b986314c36603ffccc2e4a776bdfd151e2e2d09ce2232c04e5b4
+  metadata.gz: 3f071261ab8d77b0963a3ee7a82e198139cf095391a9140efb6a7d954778a5fa
+  data.tar.gz: 0355abc8c57df2c8af562546353d76b663d4a4634fb7b16f937a59eb13c3c4e8
 SHA512:
-  metadata.gz: 9559f94d0fc7ab3be558c999ea36305817809c15069104363c934d3e2bbd46e9ca99d8065dd61210de4545e980f8beab5c71bdabcb36fc3e43edf83181f4e654
-  data.tar.gz: 7b426a84ece0047b636498828ff2af087200022a88c295c1e52ee4fd799849fa3a2eaeeee7ac9e79b2d45714b0b0b20f33fcdb18b1f759f22f85d8424b6e3ca4
+  metadata.gz: dfe1e9afb9fb2be9ec55bb2df53c132daa25f8ede8918734bcc6992fc4a75b687bebb32b382abdb5ddb8bd842c890f13e583c932f250e700a2237c1ccf7eda6b
+  data.tar.gz: 2c065cb05f42b4bcf52003a64e8466542ebc6eec5e6fcda5332f8f0f6a0a6f4f9f5c031cda9ae82c306402f6820c15c9a7194d9a2a776aa054276d4dfee2b044

data/app/models/ansible_role.rb

@@ -28,6 +28,12 @@ class AnsibleRole < ApplicationRecord
   scoped_search :relation => :hostgroups,
                 :on => :name, :rename => :hostgroup, :only_explicit => true
 
+  apipie :class, "A class representing #{model_name.human} object" do
+    name 'Ansible role'
+    refs 'AnsibleRole'
+    sections only: %w[all additional]
+    property :name, String, desc: 'Returns name of the ansible role'
+  end
   # Methods to be allowed in any template with safemode enabled
   class Jail < Safemode::Jail
     allow :name

data/app/models/concerns/foreman_ansible/host_managed_extensions.rb

@@ -67,6 +67,11 @@ end
 
 module Host
   class Managed
+    apipie :class do
+      property :all_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns all ansible roles assigned to the host, both its own and inherited'
+      property :ansible_roles, array_of: 'AnsibleRole', desc: 'Returns ansible roles assigned to the host'
+      property :inherited_ansible_roles, array_of: 'AnsibleRole', desc: 'Returns inherited ansible roles assigned to the host'
+    end
     # Methods to be allowed in any template with safemode enabled
     class Jail < Safemode::Jail
       allow :all_ansible_roles, :ansible_roles, :inherited_ansible_roles

data/app/models/foreman_ansible/ansible_provider.rb

@@ -35,8 +35,8 @@ if defined? ForemanRemoteExecution
           {
             'per-host' => {
               host.name => {
-                'ansible_ssh_pass' => rex_ssh_password(host),
-                'ansible_sudo_pass' => rex_sudo_password(host)
+                'ansible_password' => rex_ssh_password(host),
+                'ansible_become_password' => rex_effective_user_password(host)
               }
             }
           }
@@ -46,8 +46,8 @@ if defined? ForemanRemoteExecution
           host_setting(host, 'remote_execution_ssh_password')
         end
 
-        def rex_sudo_password(host)
-          host_setting(host, 'remote_execution_sudo_password')
+        def rex_effective_user_password(host)
+          host_setting(host, 'remote_execution_effective_user_password')
         end
 
         def host_setting(host, setting)
@@ -62,12 +62,8 @@ if defined? ForemanRemoteExecution
           'ansible-runner'
         end
 
-        def required_proxy_selector_for(template)
-          if template.remote_execution_features.where(:label => 'ansible_run_capsule_upgrade').any?
-            ::DefaultProxyProxySelector.new
-          else
-            super
-          end
+        def proxy_action_class
+          'ForemanAnsibleCore::TaskLauncher::Playbook::PlaybookRunnerAction'
         end
 
         private

data/app/services/foreman_ansible/ansible_report_importer.rb

@@ -9,7 +9,7 @@ module ForemanAnsible
     included do
       def host
         hostname = name.downcase
-        if AnsibleReportScanner.ansible_report?(raw['logs']) &&
+        if AnsibleReportScanner.ansible_report?(raw) &&
            IPAddress.valid?(hostname) &&
            Nic::Interface.find_by(:ip => hostname)
           @host = Nic::Interface.find_by(:ip => hostname).host

data/app/services/foreman_ansible/ansible_report_scanner.rb

@@ -5,14 +5,17 @@ module ForemanAnsible
   # sets the origin of the report to 'Ansible'
   class AnsibleReportScanner
     class << self
-      def scan(report, logs)
-        if (is_ansible = ansible_report?(logs))
-          report.origin = 'Ansible'
-        end
-        is_ansible
+      def add_reporter_data(report, raw); end
+
+      def identify_origin(raw)
+        'Ansible' if ansible_report?(raw)
+      end
+
+      def ansible_report?(raw)
+        raw['reporter'] == 'ansible' || ansible_legacy_report?(raw['logs'])
       end
 
-      def ansible_report?(logs)
+      def ansible_legacy_report?(logs)
         return false if logs.blank?
         logs.any? do |log|
           log['log'].fetch('messages', {}).

data/app/services/foreman_ansible/renderer_methods.rb

@@ -4,7 +4,18 @@ module ForemanAnsible
   # Macro to fetch RH Insights plan playbook
   module RendererMethods
     extend ActiveSupport::Concern
+    extend ApipieDSL::Module
 
+    apipie :class, 'Macros related to Ansible playbooks' do
+      name 'Ansible'
+      sections only: %w[all jobs]
+    end
+
+    apipie :method, 'Returns Insights maintenance plan for host' do
+      required :plan_id, String, desc: 'The playbook for the rule coming from insights'
+      optional :organization_id, Integer, desc: 'The Foreman organization associated with the Insights account', default: 'Current organization ID'
+      returns String, desc: 'Insights maintenance plan for host'
+    end
     def insights_remediation(plan_id, organization_id = Organization.current.id)
       return "$INSIGHTS_REMEDIATION[#{plan_id}, #{organization_id}]" if preview?
 

data/app/services/foreman_ansible/variables_importer.rb

@@ -78,7 +78,7 @@ module ForemanAnsible
     def create_new_variables(variables)
       iterate_over_variables(variables) do |role, memo, attrs|
         variable = AnsibleVariable.new(
-          JSON.parse(attrs)['ansible_variable']
+          JSON.parse(attrs)
         )
         variable.ansible_role = ::AnsibleRole.find_by(:name => role)
         variable.save
@@ -88,7 +88,7 @@ module ForemanAnsible
 
     def update_variables(variables)
       iterate_over_variables(variables) do |_role, memo, attrs|
-        attributes = JSON.parse(attrs)['ansible_variable']
+        attributes = JSON.parse(attrs)
         var = AnsibleVariable.find attributes['id']
         var.update(attributes)
         memo << var
@@ -98,7 +98,7 @@ module ForemanAnsible
     def delete_old_variables(variables)
       iterate_over_variables(variables) do |_role, memo, attrs|
         variable = AnsibleVariable.find(
-          JSON.parse(attrs)['ansible_variable']['id']
+          JSON.parse(attrs)['id']
         )
         memo << variable.key
         variable.destroy

data/app/views/ansible_roles/welcome.html.erb

@@ -4,7 +4,7 @@
     <%= icon_text("play", "", :kind => "fa") %>
   </div>
   <h1><%= _('Ansible Roles') %></h1>
-  <p><%= _('No ansible roles were found in Foreman. If you want to assign roles to your hosts,
+  <p><%= _('No Ansible Roles were found in Foreman. If you want to assign roles to your hosts,
              you have to import them first.').html_safe %>
   </p>
   <p><%= link_to(_('Learn more about this in the documentation.'), documentation_url('#4.1ImportingRoles', :root_url => ansible_doc_url), target: '_blank') %></p>

data/app/views/foreman_ansible/job_templates/capsule_upgrade_-_ansible_default.erb

@@ -15,7 +15,7 @@ template_inputs:
   value_type: plain
   hidden_value: false
 model: JobTemplate
-job_category: Ansible Playbook
+job_category: Maintenance Operations
 description_format: "%{template_name}"
 provider_type: Ansible
 kind: job_template
@@ -41,8 +41,11 @@ feature: ansible_run_capsule_upgrade
       when: "'rubygem-foreman_maintain' not in ansible_facts.packages"
 
     - block:
+      <%- whitelist_option = if input('whitelist_options').present?
+                               "--whitelist=#{input('whitelist_options')}"
+                             end -%>
       - name: Upgrade Capsule server using satellite-maintain
-        shell: satellite-maintain upgrade run --assumeyes --target-version=<%= input('target_version') %> --whitelist="<%= input('whitelist_options') %>"
+        shell: satellite-maintain upgrade run --assumeyes --target-version=<%= input('target_version') %> <%= whitelist_option %>
         register: result
 
       - name: Re-Gather the rpm package facts after the upgrade

data/app/views/foreman_ansible/job_templates/configure_cloud_connector_-_ansible_default.erb

@@ -22,10 +22,11 @@ template_inputs:
   hidden_value: false
   description: You can specify a HTTP proxy address that should be used for Cloud Connector connection to the cloud.redhat.com. Note that it must be HTTP proxy, not HTTPS. The tunelling of SSL (secured web socket connection) in SSL (HTTPS proxy) is currently unsupported.
 model: JobTemplate
-job_category: Ansible Playbook
+job_category: Maintenance Operations
 description_format: "%{template_name}"
 provider_type: Ansible
 kind: job_template
+feature: ansible_configure_cloud_connector
 %>
 
 ---

data/app/views/foreman_ansible/job_templates/service_action_-_enable_web_console.erb

@@ -0,0 +1,16 @@
+<%#
+name: Service Action - Enable Web Console
+job_category: Ansible Services
+snippet: false
+provider_type: Ansible
+kind: job_template
+model: JobTemplate
+feature: ansible_enable_web_console
+%>
+---
+- hosts: all
+  tasks:
+  - name: ensure cockpit is installed
+    package:
+      name: "cockpit-system"
+      state: present

data/lib/foreman_ansible/register.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 Foreman::Plugin.register :foreman_ansible do
-  requires_foreman '>= 2.0'
+  requires_foreman '>= 2.2'
 
   security_block :foreman_ansible do
     permission :play_roles_on_host,
@@ -109,6 +109,10 @@ Foreman::Plugin.register :foreman_ansible do
   apipie_documented_controllers [
     "#{ForemanAnsible::Engine.root}/app/controllers/api/v2/*.rb"
   ]
+  ApipieDSL.configuration.dsl_classes_matchers += [
+    "#{ForemanAnsible::Engine.root}/app/models/*.rb",
+    "#{ForemanAnsible::Engine.root}/app/services/foreman_ansible/*.rb"
+  ]
 
   register_info_provider ForemanAnsible::AnsibleInfo
 

data/lib/foreman_ansible/remote_execution.rb

@@ -45,7 +45,14 @@ module ForemanAnsible
       RemoteExecutionFeature.register(
         :ansible_run_capsule_upgrade,
         N_('Upgrade Capsules on given hosts'),
-        :description => N_('Upgrade Capsules on given Capsule server hosts')
+        :description => N_('Upgrade Capsules on given Capsule server hosts'),
+        :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
+      )
+      RemoteExecutionFeature.register(
+        :ansible_configure_cloud_connector,
+        N_('Configure Cloud Connector on given hosts'),
+        :description => N_('Configure Cloud Connector on given hosts'),
+        :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
       )
     end
   end

data/lib/foreman_ansible/version.rb

@@ -4,5 +4,5 @@
 # This way other parts of Foreman can just call ForemanAnsible::VERSION
 # and detect what version the plugin is running.
 module ForemanAnsible
-  VERSION = '5.1.1'
+  VERSION = '6.0.2'
 end

data/test/fixtures/report.json

@@ -1,4 +1,5 @@
 {
+  "reporter": "ansible",
   "reported_at":"2018-01-15 17:31:36 521275",
   "metrics": {
     "time":
@@ -17,7 +18,7 @@
           "source": "common : Install Common packages"
         },
         "messages": {
-          "message": "{\"msg\": \"All items completed\", \"changed\": false, \"results\": [{\"_ansible_parsed\": true, \"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"git\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"git\"], \"autoclean\": false, \"name\": \"git\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}, {\"_ansible_parsed\": true, \"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"htop\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"htop\"], \"autoclean\": false, \"name\": \"htop\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}, {\"_ansible_parsed\": true, \"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"zsh\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"zsh\"], \"autoclean\": false, \"name\": \"zsh\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}]}"},"level":"info"}},{"log":{"sources":{"source":"common : Copy default motd"},"messages":{"message":"{\"_ansible_parsed\": true, \"group\": \"root\", \"uid\": 0, \"checksum\": \"0a381ff6a86081af6dc957a77c7e2017a3244c4c\", \"changed\": false, \"owner\": \"root\", \"state\": \"file\", \"gid\": 0, \"mode\": \"0644\", \"diff\": {\"after\": {\"path\": \"/etc/motd\"}, \"before\": {\"path\": \"/etc/motd\"}}, \"invocation\": {\"module_args\": {\"directory_mode\": null, \"force\": false, \"remote_src\": null, \"path\": \"/etc/motd\", \"owner\": \"root\", \"follow\": false, \"group\": \"root\", \"unsafe_writes\": null, \"state\": \"file\", \"content\": null, \"serole\": null, \"diff_peek\": null, \"setype\": null, \"dest\": \"/etc/motd\", \"selevel\": null, \"original_basename\": \"motd.txt\", \"regexp\": null, \"validate\": null, \"src\": \"motd.txt\", \"seuser\": null, \"recurse\": false, \"delimiter\": null, \"mode\": null, \"attributes\": null, \"backup\": null}}, \"path\": \"/etc/motd\", \"size\": 1090, \"_ansible_no_log\": false}"
+          "message": "{\"msg\": \"All items completed\", \"changed\": false, \"results\": [{\"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"git\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"git\"], \"autoclean\": false, \"name\": \"git\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}, {\"_ansible_parsed\": true, \"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"htop\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"htop\"], \"autoclean\": false, \"name\": \"htop\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}, {\"_ansible_parsed\": true, \"changed\": false, \"_ansible_no_log\": false, \"cache_updated\": false, \"_ansible_item_result\": true, \"failed\": false, \"item\": \"zsh\", \"invocation\": {\"module_args\": {\"dpkg_options\": \"force-confdef,force-confold\", \"upgrade\": null, \"force\": false, \"force_apt_get\": false, \"package\": [\"zsh\"], \"autoclean\": false, \"name\": \"zsh\", \"purge\": false, \"allow_unauthenticated\": false, \"state\": \"present\", \"autoremove\": false, \"update_cache\": null, \"default_release\": null, \"only_upgrade\": false, \"cache_valid_time\": 0, \"deb\": null, \"install_recommends\": null}}, \"_ansible_ignore_errors\": null, \"cache_update_time\": 1515797094}]}"},"level":"info"}},{"log":{"sources":{"source":"common : Copy default motd"},"messages":{"message":"{\"_ansible_parsed\": true, \"group\": \"root\", \"uid\": 0, \"checksum\": \"0a381ff6a86081af6dc957a77c7e2017a3244c4c\", \"changed\": false, \"owner\": \"root\", \"state\": \"file\", \"gid\": 0, \"mode\": \"0644\", \"diff\": {\"after\": {\"path\": \"/etc/motd\"}, \"before\": {\"path\": \"/etc/motd\"}}, \"invocation\": {\"module_args\": {\"directory_mode\": null, \"force\": false, \"remote_src\": null, \"path\": \"/etc/motd\", \"owner\": \"root\", \"follow\": false, \"group\": \"root\", \"unsafe_writes\": null, \"state\": \"file\", \"content\": null, \"serole\": null, \"diff_peek\": null, \"setype\": null, \"dest\": \"/etc/motd\", \"selevel\": null, \"original_basename\": \"motd.txt\", \"regexp\": null, \"validate\": null, \"src\": \"motd.txt\", \"seuser\": null, \"recurse\": false, \"delimiter\": null, \"mode\": null, \"attributes\": null, \"backup\": null}}, \"path\": \"/etc/motd\", \"size\": 1090, \"_ansible_no_log\": false}"
         },
         "level": "info"
       }

data/test/unit/ansible_provider_test.rb

@@ -35,13 +35,13 @@ class AnsibleProviderTest < ActiveSupport::TestCase
       it 'generates secrets properly' do
         params = {
           'remote_execution_ssh_password' => 'password',
-          'remote_execution_sudo_password' => 'letmein'
+          'remote_execution_effective_user_password' => 'letmein'
         }
         host.expects(:params).twice.returns(params)
         secrets = ForemanAnsible::AnsibleProvider.secrets(host)
         host_secrets = secrets['per-host'][host.name]
-        assert_equal host_secrets['ansible_ssh_pass'], 'password'
-        assert_equal host_secrets['ansible_sudo_pass'], 'letmein'
+        assert_equal host_secrets['ansible_password'], 'password'
+        assert_equal host_secrets['ansible_become_password'], 'letmein'
       end
     end
 

data/test/unit/lib/foreman_ansible_core/ansible_runner_test.rb

@@ -24,7 +24,7 @@ module ForemanAnsibleCore
             '_meta' => { 'hostvars' => { 'foreman.example.com' => {} } } }
         end
         let(:input) do
-          host_secrets = { 'ansible_ssh_pass' => 'letmein', 'ansible_sudo_pass' => 'iamroot' }
+          host_secrets = { 'ansible_password' => 'letmein', 'ansible_become_password' => 'iamroot' }
           secrets = { 'per-host' => { 'foreman.example.com' => host_secrets } }
           host_input = { 'input' => { 'action_input' => { 'secrets' => secrets } } }
           { 'foreman.example.com' => host_input }
@@ -32,18 +32,18 @@ module ForemanAnsibleCore
         let(:runner) { ForemanAnsibleCore::Runner::AnsibleRunner.allocate }
 
         test 'uses secrets from inventory' do
-          test_inventory = inventory.merge('ssh_password' => 'sshpass', 'sudo_password' => 'sudopass')
+          test_inventory = inventory.merge('ssh_password' => 'sshpass', 'effective_user_password' => 'mypass')
           rebuilt = runner.send(:rebuild_secrets, test_inventory, input)
           host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-          assert_equal 'sshpass', host_vars['ansible_ssh_pass']
-          assert_equal 'sudopass', host_vars['ansible_sudo_pass']
+          assert_equal 'sshpass', host_vars['ansible_password']
+          assert_equal 'mypass', host_vars['ansible_become_password']
         end
 
         test 'host secrets are used when not overriden by inventory secrest' do
           rebuilt = runner.send(:rebuild_secrets, inventory, input)
           host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-          assert_equal 'letmein', host_vars['ansible_ssh_pass']
-          assert_equal 'iamroot', host_vars['ansible_sudo_pass']
+          assert_equal 'letmein', host_vars['ansible_password']
+          assert_equal 'iamroot', host_vars['ansible_become_password']
         end
       end
     end

data/test/unit/lib/foreman_ansible_core/playbook_runner_test.rb

@@ -87,24 +87,24 @@ class PlaybookRunnerTest < ActiveSupport::TestCase
         '_meta' => { 'hostvars' => { 'foreman.example.com' => {} } } }
     end
     let(:secrets) do
-      host_secrets = { 'ansible_ssh_pass' => 'letmein', 'ansible_sudo_pass' => 'iamroot' }
+      host_secrets = { 'ansible_password' => 'letmein', 'ansible_become_password' => 'iamroot' }
       { 'per-host' => { 'foreman.example.com' => host_secrets } }
     end
     let(:runner) { ForemanAnsibleCore::Runner::Playbook.allocate }
 
     test 'uses secrets from inventory' do
-      test_inventory = inventory.merge('ssh_password' => 'sshpass', 'sudo_password' => 'sudopass')
+      test_inventory = inventory.merge('ssh_password' => 'sshpass', 'effective_user_password' => 'mypass')
       rebuilt = runner.send(:rebuild_secrets, test_inventory, secrets)
       host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-      assert_equal 'sshpass', host_vars['ansible_ssh_pass']
-      assert_equal 'sudopass', host_vars['ansible_sudo_pass']
+      assert_equal 'sshpass', host_vars['ansible_password']
+      assert_equal 'mypass', host_vars['ansible_become_password']
     end
 
     test 'host secrets are used when not overriden by inventory secrest' do
       rebuilt = runner.send(:rebuild_secrets, inventory, secrets)
       host_vars = rebuilt.dig('_meta', 'hostvars', 'foreman.example.com')
-      assert_equal 'letmein', host_vars['ansible_ssh_pass']
-      assert_equal 'iamroot', host_vars['ansible_sudo_pass']
+      assert_equal 'letmein', host_vars['ansible_password']
+      assert_equal 'iamroot', host_vars['ansible_become_password']
     end
   end
 end

data/test/unit/services/inventory_creator_test.rb

@@ -10,7 +10,7 @@ module ForemanAnsible
       @host = FactoryBot.build(:host)
       @template_invocation = OpenStruct.new(
         :job_invocation => OpenStruct.new(:password => 'foobar',
-                                          :sudo_password => 'foobar'),
+                                          :effective_user_password => 'foobar'),
         :effective_user => 'foobar'
       )
     end
@@ -69,8 +69,8 @@ module ForemanAnsible
                    connection_params['ansible_winrm_server_cert_validation']
       assert_equal Setting['remote_execution_effective_user_method'],
                    connection_params['ansible_become_method']
-      refute connection_params.key?('ansible_ssh_pass')
-      refute connection_params.key?('ansible_sudo_pass')
+      refute connection_params.key?('ansible_password')
+      refute connection_params.key?('ansible_become_password')
     end
 
     test 'ssh private key is passed when available' do

data/webpack/components/ReportJsonViewer.js

@@ -10,7 +10,7 @@ const theme = {
 
 const ReportJsonViewer = ({ data }) => (
   <div className="report-json-viewer">
-    <JSONTree data={this.props.data} hideRoot theme={theme} />
+    <JSONTree data={data} hideRoot theme={theme} />
   </div>
 );
 

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 5.1.1
+  version: 6.0.2
 platform: ruby
 authors:
 - Daniel Lobato Garcia
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-06-05 00:00:00.000000000 Z
+date: 2021-01-22 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.80.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.80.0
 - !ruby/object:Gem::Dependency
   name: foreman_ansible_core
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 4.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 4.2.0
 - !ruby/object:Gem::Dependency
   name: ipaddress
   requirement: !ruby/object:Gem::Requirement
@@ -196,6 +182,7 @@ files:
 - app/views/foreman_ansible/job_templates/run_command_-_ansible_default.erb
 - app/views/foreman_ansible/job_templates/run_playbook-ansible_default.erb
 - app/views/foreman_ansible/job_templates/service_action_-_ansible_default.erb
+- app/views/foreman_ansible/job_templates/service_action_-_enable_web_console.erb
 - app/views/ui_ansible_roles/index.json.rabl
 - app/views/ui_ansible_roles/main.json.rabl
 - app/views/ui_ansible_roles/show.json.rabl
@@ -346,7 +333,7 @@ homepage: https://github.com/theforeman/foreman_ansible
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -361,8 +348,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Ansible integration with Foreman (theforeman.org)
 test_files:
@@ -372,39 +359,39 @@ test_files:
 - test/fixtures/insights_playbook.yaml
 - test/fixtures/report.json
 - test/fixtures/sample_facts.json
+- test/functional/ansible_roles_controller_test.rb
+- test/functional/ansible_variables_controller_test.rb
+- test/functional/api/v2/ansible_inventories_controller_test.rb
 - test/functional/api/v2/ansible_roles_controller_test.rb
+- test/functional/api/v2/ansible_variables_controller_test.rb
 - test/functional/api/v2/hostgroups_controller_test.rb
 - test/functional/api/v2/hosts_controller_test.rb
-- test/functional/api/v2/ansible_inventories_controller_test.rb
-- test/functional/api/v2/ansible_variables_controller_test.rb
-- test/functional/ansible_roles_controller_test.rb
-- test/functional/ansible_variables_controller_test.rb
-- test/functional/ui_ansible_roles_controller_test.rb
 - test/functional/hosts_controller_test.rb
+- test/functional/ui_ansible_roles_controller_test.rb
+- test/test_plugin_helper.rb
 - test/unit/actions/run_ansible_job_test.rb
 - test/unit/actions/run_proxy_ansible_command_test.rb
+- test/unit/ansible_role_test.rb
+- test/unit/ansible_variable_test.rb
 - test/unit/concerns/config_reports_extensions_test.rb
 - test/unit/concerns/host_managed_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
 - test/unit/helpers/ansible_reports_helper_test.rb
+- test/unit/host_ansible_role_test.rb
+- test/unit/hostgroup_ansible_role_test.rb
 - test/unit/lib/foreman_ansible_core/command_creator_test.rb
 - test/unit/lib/foreman_ansible_core/ansible_runner_test.rb
 - test/unit/lib/foreman_ansible_core/playbook_runner_test.rb
 - test/unit/lib/proxy_api/ansible_test.rb
+- test/unit/services/ansible_report_importer_test.rb
+- test/unit/services/ansible_variables_importer_test.rb
 - test/unit/services/api_roles_importer_test.rb
+- test/unit/services/fact_importer_test.rb
+- test/unit/services/fact_parser_test.rb
 - test/unit/services/fact_sparser_test.rb
 - test/unit/services/insights_plan_runner_test.rb
 - test/unit/services/roles_importer_test.rb
 - test/unit/services/structured_fact_importer_test.rb
 - test/unit/services/ui_roles_importer_test.rb
-- test/unit/services/ansible_report_importer_test.rb
-- test/unit/services/fact_importer_test.rb
-- test/unit/services/fact_parser_test.rb
 - test/unit/services/inventory_creator_test.rb
-- test/unit/services/ansible_variables_importer_test.rb
-- test/unit/ansible_role_test.rb
-- test/unit/host_ansible_role_test.rb
-- test/unit/hostgroup_ansible_role_test.rb
 - test/unit/ansible_provider_test.rb
-- test/unit/ansible_variable_test.rb
-- test/test_plugin_helper.rb