foreman_ansible 2.3.2 → 2.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3dd4a02656d2b41202e38c91fda9cd2b5b7bfb08
-  data.tar.gz: a7f7f75580a6d389b09dd3a1bb47c4420b1ffab2
+  metadata.gz: 1bc81f85be7efbc4ebb9636fc9bd550d54fb22f3
+  data.tar.gz: d89aa3d74ffb81ed88a678e5f139530bdec076aa
 SHA512:
-  metadata.gz: 52866dabe9b2a7bac15bc23920541c896cc337304a1bafcfbe057645ea73c85939cdbc989cec226c7f32f61b254891b345b7720bb1d771074f25ece5feea4d39
-  data.tar.gz: afdbf297b5fe851adb1a71352b7b821b109d203ad094b51ede3f0d7f3658ea0d358f84022040c056bccf8e12b0810e9b68e2c7db0ae3c15488a44825080bc29c
+  metadata.gz: 853c086e4c7a344844c2da7360221030dba7ea21fd9d65d0f387b3c76825af7b2bae258ca8914eeaecfee5756ca99fbfc0d6bf7580e96dff2f458a3e9d449162
+  data.tar.gz: af2d1693fc63a41a202a2de837a9ae678b1400fc5c5c3eda567d4211e7244ac4036880d061fa74681052b277f356f33840d25f1845f92c5f02cc599f318b3103

data/app/assets/stylesheets/foreman_ansible/foreman-ansible.css

@@ -1,3 +1,7 @@
 .report-json-viewer * {
   background: transparent !important;
 }
+
+.btn.btn-default.no-float {
+  float: none;
+}

data/app/controllers/ansible_variables_controller.rb

@@ -18,6 +18,19 @@ class AnsibleVariablesController < ::LookupKeysController
                                   :per_page => params[:per_page])
   end
 
+  def new
+    @ansible_variable = AnsibleVariable.new
+  end
+
+  def create
+    @ansible_variable = AnsibleVariable.new(ansible_variable_params.merge(:imported => false))
+    if @ansible_variable.save
+      process_success
+    else
+      process_error
+    end
+  end
+
   def import
     import_roles = @importer_roles.import_role_names
     import_roles[:new_roles] = import_roles[:new]
@@ -27,11 +40,12 @@ class AnsibleVariablesController < ::LookupKeysController
   end
 
   def confirm_import
-    results = @importer.finish_import(new_vars, old_vars)
-    info _(
+    results = @importer.finish_import(new_vars, old_vars, updated_vars)
+    success _(
       "Import of variables successfully finished.\n"\
-      "Added: #{results[:added].join(', ')} \n "\
-      "Removed: #{results[:obsolete].join(', ')}"
+      "Added: #{results[:added].count} \n "\
+      "Removed: #{results[:obsolete].count} \n"\
+      "Updated: #{results[:updated].count}"
     )
     redirect_to ansible_variables_path
   end
@@ -49,11 +63,19 @@ class AnsibleVariablesController < ::LookupKeysController
   end
 
   def new_vars
-    params[:changed] ? params[:changed][:new].as_json : {}
+    fetch_vars :new
   end
 
   def old_vars
-    params[:changed] ? params[:changed][:obsolete].as_json : {}
+    fetch_vars :obsolete
+  end
+
+  def updated_vars
+    fetch_vars :update
+  end
+
+  def fetch_vars(key)
+    params.fetch(:changed, {}).fetch(key, {}).try(:as_json) || {}
   end
 
   def import_new_roles

data/app/controllers/api/v2/ansible_variables_controller.rb

@@ -6,6 +6,7 @@ module Api
     class AnsibleVariablesController < ::Api::V2::BaseController
       include ::Api::Version2
       include Foreman::Controller::Parameters::VariableLookupKey
+      include Foreman::Controller::Parameters::AnsibleVariable
 
       resource_description do
         api_version 'v2'
@@ -36,7 +37,7 @@ module Api
       def_param_group :ansible_variable do
         param :ansible_variable, Hash, :required => true, :action_aware => true do
           param :variable, String, :required => true, :desc => N_("Name of variable")
-          param :ansible_role_id, :number, :desc => N_("Role ID")
+          param :ansible_role_id, :number, :required => true, :desc => N_("Role ID")
           param :default_value, :any_type, :of => LookupKey::KEY_TYPES, :desc => N_("Default value of variable")
           param :hidden_value, :bool, :desc => N_("When enabled the parameter is hidden in the UI")
           param :override_value_order, String, :desc => N_("The order in which values are resolved")
@@ -50,6 +51,13 @@ module Api
         end
       end
 
+      api :POST, '/ansible_variables', N_('Create Ansible variable')
+      param_group :ansible_variable, :as => :create
+      def create
+        @ansible_variable = AnsibleVariable.new(ansible_variable_params.merge(:imported => false))
+        process_response @ansible_variable.save
+      end
+
       api :PUT, '/ansible_variables/:id', N_('Updates Ansible variable')
       param :id, :identifier, :required => true
       param_group :ansible_variable, :as => :update

data/app/controllers/concerns/foreman/controller/parameters/ansible_variable.rb

@@ -11,7 +11,8 @@ module Foreman
         class_methods do
           def ansible_variable_params_filter
             Foreman::ParameterFilter.new(::AnsibleVariable).tap do |filter|
-              filter.permit :ansible_roles => [], :ansible_role_ids => [],
+              filter.permit :imported, :ansible_role_id,
+                            :ansible_roles => [], :ansible_role_ids => [],
                             :ansible_role_names => [],
                             :param_classes => [], :param_classes_ids => [],
                             :param_classes_names => []

data/app/controllers/ui_ansible_roles_controller.rb

@@ -0,0 +1,14 @@
+class UiAnsibleRolesController < ::Api::V2::BaseController
+  def resource_name(resource = 'AnsibleRole')
+    super resource
+  end
+
+  def index
+    @ui_ansible_roles = resource_scope_for_index(:permission => :view_ansible_roles)
+  end
+
+  # restore original method from find_common to ignore resource nesting
+  def resource_scope(options = {})
+    @resource_scope ||= scope_for(resource_class, options)
+  end
+end

data/app/helpers/foreman_ansible/ansible_roles_helper.rb

@@ -5,12 +5,26 @@ module ForemanAnsible
   module AnsibleRolesHelper
     def ansible_proxy_links(hash, classes = nil)
       SmartProxy.with_features('Ansible').map do |proxy|
-        display_link_if_authorized(_('From %s') % proxy.name,
+        display_link_if_authorized(_('Import from %s') % proxy.name,
                                    hash.merge(:proxy => proxy),
                                    :class => classes)
       end.flatten
     end
 
+    def ansible_role_select(f, persisted)
+      blank_opt = persisted ? {} : { :include_blank => true }
+      select_items = persisted ? [f.object.ansible_role] : AnsibleRole.order(:name)
+      select_f f,
+               :ansible_role_id,
+               select_items,
+               :id,
+               :to_label,
+               blank_opt,
+               :label => _("Ansible Role"),
+               :disabled => persisted,
+               :required => true
+    end
+
     def ansible_proxy_import(hash)
       select_action_button(_('Import'),
                            { :primary => true, :class => 'roles-import' },
@@ -20,5 +34,9 @@ module ForemanAnsible
     def import_time(role)
       _('%s ago') % time_ago_in_words(role.updated_at)
     end
+
+    def roles_attrs(roles)
+      roles.map { |item| { :id => item.id, :name => item.name } }
+    end
   end
 end

data/app/models/ansible_role.rb

@@ -17,6 +17,7 @@ class AnsibleRole < ApplicationRecord
                                :class_name => 'AnsibleVariable'
 
   scoped_search :on => :name, :complete_value => true
+  scoped_search :on => :id, :complete_value => false
   scoped_search :on => :updated_at
   scoped_search :relation => :hosts,
                 :on => :id, :rename => :host_id, :only_explicit => true

data/app/models/ansible_variable.rb

@@ -3,8 +3,9 @@
 # Represents the variables used in Ansible to parameterize playbooks
 class AnsibleVariable < LookupKey
   belongs_to :ansible_role, :inverse_of => :ansible_variables
-  validates :ansible_role, :presence => true
+  validates :ansible_role_id, :presence => true
   scoped_search :on => :key, :aliases => [:name], :complete_value => true
+  scoped_search :on => :imported, :complete_value => { :true => true, :false => false }
   scoped_search :relation => :ansible_role, :on => :name,
                 :complete_value => true, :rename => :ansible_role
 
@@ -12,8 +13,12 @@ class AnsibleVariable < LookupKey
     true
   end
 
-  def self.humanize_class_name
-    'Ansible variable'
+  def self.humanize_class_name(options = nil)
+    if options.present?
+      super
+    else
+      "Ansible variable"
+    end
   end
 
   def editable_by_user?

data/app/services/foreman_ansible/variables_importer.rb

@@ -6,6 +6,17 @@ module ForemanAnsible
   class VariablesImporter
     include ::ForemanAnsible::ProxyAPI
 
+    VARIABLE_TYPES = {
+      'TrueClass' => 'boolean',
+      'FalseClass' => 'boolean',
+      'Integer' => 'integer',
+      'Fixnum' => 'integer',
+      'Float' => 'real',
+      'Array' => 'array',
+      'Hash' => 'hash',
+      'String' => 'string'
+    }.freeze
+
     def initialize(proxy = nil)
       @ansible_proxy = proxy
     end
@@ -34,11 +45,15 @@ module ForemanAnsible
     end
 
     def initialize_variables(variables, role)
-      variables.map do |variable|
+      variables.map do |variable_name, variable_default|
         variable = AnsibleVariable.find_or_initialize_by(
-          :key => variable
-          # :key_type, :default_value, :required
+          :key => variable_name
         )
+        if variable.new_record?
+          variable.assign_attributes(:default_value => variable_default,
+                                     :key_type => infer_key_type(variable_default),
+                                     :imported => true)
+        end
         variable.ansible_role = role
         variable.valid? ? variable : nil
       end
@@ -46,45 +61,48 @@ module ForemanAnsible
 
     def detect_changes(imported)
       changes = {}.with_indifferent_access
-      old, changes[:new] = imported.partition { |role| role.id.present? }
-      changes[:obsolete] = AnsibleVariable.where.not(:id => old.map(&:id))
+      persisted, changes[:new] = imported.partition { |role| role.id.present? }
+      changes[:update], _old = persisted.partition(&:changed?)
+      changes[:obsolete] = AnsibleVariable.where.not(:id => persisted.pluck(:id), :imported => false)
       changes
     end
 
-    def finish_import(new, obsolete)
-      results = { :added => [], :obsolete => [] }
+    def finish_import(new, obsolete, update)
+      results = { :added => [], :obsolete => [], :updated => [] }
       results[:added] = create_new_variables(new) if new.present?
       results[:obsolete] = delete_old_variables(obsolete) if obsolete.present?
+      results[:updated] = update_variables(update) if update.present?
       results
     end
 
-    def create_new_variables(new)
-      added = []
-      new.each do |role, variables|
-        variables.each_value do |variable_properties|
-          variable = AnsibleVariable.new(
-            JSON.parse(variable_properties)['ansible_variable']
-          )
-          variable.ansible_role = ::AnsibleRole.find_by(:name => role)
-          variable.save
-          added << variable.key
-        end
+    def create_new_variables(variables)
+      iterate_over_variables(variables) do |role, memo, attrs|
+        variable = AnsibleVariable.new(
+          JSON.parse(attrs)['ansible_variable']
+        )
+        variable.ansible_role = ::AnsibleRole.find_by(:name => role)
+        variable.save
+        memo << variable
       end
-      added
     end
 
-    def delete_old_variables(old)
-      removed = []
-      old.each_value do |variables|
-        variables.each_value do |variable|
-          variable = AnsibleVariable.find(
-            JSON.parse(variable)['ansible_variable']['id']
-          )
-          removed << variable.key
-          variable.destroy
-        end
+    def update_variables(variables)
+      iterate_over_variables(variables) do |_role, memo, attrs|
+        attributes = JSON.parse(attrs)['ansible_variable']
+        var = AnsibleVariable.find attributes['id']
+        var.update(attributes)
+        memo << var
+      end
+    end
+
+    def delete_old_variables(variables)
+      iterate_over_variables(variables) do |_role, memo, attrs|
+        variable = AnsibleVariable.find(
+          JSON.parse(attrs)['ansible_variable']['id']
+        )
+        memo << variable.key
+        variable.destroy
       end
-      removed
     end
 
     private
@@ -96,5 +114,18 @@ module ForemanAnsible
     def remote_variables
       proxy_api.all_variables
     end
+
+    def infer_key_type(value)
+      VARIABLE_TYPES[value.class.to_s] || 'string'
+    end
+
+    def iterate_over_variables(variables)
+      variables.reduce([]) do |memo, (role, vars)|
+        vars.map do |_key, attrs|
+          yield role, memo, attrs if block_given?
+          memo
+        end
+      end
+    end
   end
 end

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

@@ -7,7 +7,7 @@
   <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)) %></p>
+  <p><%= link_to(_('Learn more about this in the documentation.'), documentation_url('#4.1ImportingRoles', :root_url => ansible_doc_url), target: '_blank') %></p>
   <div class="blank-slate-pf-secondary-action">
     <%= ansible_proxy_import(hash_for_import_ansible_roles_path) %>
   </div>

data/app/views/ansible_variables/_fields.erb

@@ -1,10 +1,11 @@
 <div id='<%= f.object.to_param %>' class='tab-pane fields'>
   <fieldset>
-    <h2><%= _("Ansible Variable Details") %></h2>
-    <%= text_f f, :key, :disabled => true, :size => "col-md-8" %>
-    <%= f.hidden_field :key %>
+    <% is_imported = f.object.new_record? ? false : f.object.imported %>
+    <h2><%= is_imported ? _("Ansible Variable Details (Imported)") : _("Ansible Variable Details")%></h2>
+    <%= text_f f, :key, :disabled => f.object.persisted?, :size => "col-md-8" %>
     <%= textarea_f f, :description, :rows => :auto, :size => "col-md-8", :class => "no-stretch" %>
-    <%= select_f f, :ansible_role_id, [f.object.ansible_role], :id, :to_label, {}, :label => _("Ansible Role"), :disabled => true %>
+    <%= ansible_role_select f, f.object.persisted? %>
+
   </fieldset>
   <fieldset>
     <h2><%= _("Default Behavior") %></h2>
@@ -12,7 +13,13 @@
     <%= checkbox_f(f, :override, :onchange => 'toggleOverrideValue(this)', :size => "col-md-8",
                    :label_help => _('Mark the variable to be managed by Foreman. When the Ansible role of this variable is assigned to a host, the default value will be added to Ansible inventory as a host variable. Specify matchers to set a different value for such variable.')
         ) %>
-    <%= param_type_selector(f, false, :onchange => 'keyTypeChange(this)', :disabled => !f.object.override) %>
+
+    <% version = Foreman::Version.new %>
+    <% if version.major.to_i >= 1 && version.minor.to_i >= 22 %>
+      <%= param_type_selector(f, false, :onchange => 'keyTypeChange(this)', :disabled => !f.object.override) %>
+    <% else %>
+      <%= param_type_selector(f, :onchange => 'keyTypeChange(this)', :disabled => !f.object.override) %>
+    <% end %>
     <%= textarea_f f, :default_value, :value => f.object.default_value_before_type_cast, :size => "col-md-8",
                    :disabled => (!f.object.override || f.object.omit),
                    :input_group_btn => fullscreen_input,

data/app/views/ansible_variables/import.html.erb

@@ -9,6 +9,9 @@
   <%= link_to_function(icon_text("check", _("Obsolete")),
                        "toggleCheckboxesBySelector('.variable_select_boxes_obsolete')",
                        :title => _("Check/Uncheck obsolete")) %>
+  <%= link_to_function(icon_text("check", _("Update")),
+                       "toggleCheckboxesBySelector('.variable_select_boxes_update')",
+                       :title => _("Check/Uncheck update")) %>
   </h6>
   <table class="<%= table_css_classes %>">
     <thead>
@@ -39,7 +42,7 @@
             <td><%= variable.ansible_role.hosts.count %></td>
             <td><%= variable.ansible_role.hostgroups.count %></td>
             <td>
-              <%= { "new" => _("Add"), "obsolete" => _("Remove") }[kind] %>
+              <%= { "new" => _("Add"), "obsolete" => _("Remove"), "update" => _("Update") }[kind] %>
             </td>
           </tr>
         <% end %>

data/app/views/ansible_variables/index.html.erb

@@ -1,14 +1,18 @@
 <% title _("Ansible Variables") %>
+<%= stylesheet 'foreman_ansible/foreman-ansible' %>
 
-<% title_actions ansible_proxy_import(hash_for_import_ansible_variables_path),
-  documentation_button('#4.3Variables', :root_url => ansible_doc_url) %>
+<%= title_actions ansible_proxy_import(hash_for_import_ansible_variables_path),
+      display_link_if_authorized(_('New Ansible Variable'), hash_for_new_ansible_variable_path, :class => "btn btn-default no-float"),
+      documentation_button('#4.3Variables', :root_url => ansible_doc_url)
+       %>
 
 <table class="<%= table_css_classes 'table-fixed' %>">
   <thead>
     <tr>
       <th class='col-md-6'><%= sort :name, :as => s_('Variable|Name') %></th>
       <th class='col-md-2'><%= sort :ansible_role, :as => s_('Variable|Role') %></th>
-      <th class='col-md-2'><%= _('Type') %></th>
+      <th class='col-md-1'><%= _('Type') %></th>
+      <th class='col-md-1'><%= sort :imported, :as => _('Imported?') %></th>
       <th class='col-md-2'><%= _('Actions') %></th>
     </tr>
   </thead>
@@ -27,6 +31,7 @@
           hash_for_ansible_variables_path(:search => "ansible_role = #{variable.ansible_role}")
         ) %></td>
         <td class="ellipsis"><%= variable.key_type || 'string' %></td>
+        <td align='center'><%= checked_icon variable.imported %></td>
         <td class="ellipsis">
           <% links = [
             display_delete_if_authorized(

data/app/views/ansible_variables/new.html.erb

@@ -0,0 +1,17 @@
+<%= breadcrumbs(
+  :items => [
+    {
+      :caption => _('Ansible Variables'),
+      :url => url_for(ansible_variables_path)
+    },
+    {
+      :caption => _('Create Ansible Variable')
+    }
+  ],
+  :switchable => false
+) %>
+<% title(_('Create Ansible Variable')) %>
+<%= form_for(@ansible_variable) do |f| %>
+    <%= render 'fields', :f => f %>
+    <%= submit_or_cancel f %>
+<% end %>

data/app/views/api/v2/ansible_variables/create.json.rabl

@@ -0,0 +1,3 @@
+object @ansible_variable
+
+extends "api/v2/ansible_variables/show"

data/app/views/api/v2/ansible_variables/show.json.rabl

@@ -5,7 +5,7 @@ attributes :id, :variable, :ansible_role, :ansible_role_id, :description, :overr
            :variable_type, :hidden_value?, :validator_type,
            :validator_rule, :merge_overrides, :merge_default,
            :avoid_duplicates, :override_value_order, :created_at, :updated_at,
-           :default_value
+           :default_value, :imported
 
 node do |ansible_variable|
   {

data/app/views/foreman_ansible/ansible_roles/_select_tab_content.html.erb

@@ -1,15 +1,15 @@
+<%= webpacked_plugins_js_for :foreman_ansible %>
+<%= webpacked_plugins_css_for :foreman_ansible %>
+
 <div class='tab-pane' id='ansible_roles'>
-  <%= multiple_selects(
-    f,
-    :ansible_roles,
-    AnsibleRole,
-    f.object.is_a?(Hostgroup) ? (f.object.inherited_and_own_ansible_roles).map(&:id) : f.object.all_ansible_roles.map(&:id),
-    {
-      :disabled => f.object.inherited_ansible_roles.map(&:id),
-      :label => _('Available roles'),
-      :label_help => _('This list of roles will be applied when the host finishes<br/> '\
-            'provisioning. Users can also play these roles through the API<br/>'\
-            'or by clicking on the Play Roles button on the Host page ').html_safe
-    },
-    { 'data-inheriteds' => f.object.inherited_ansible_roles.map(&:id).to_json }) %>
+  <div id='ansible_roles_switcher'></div>
+  <% roles = f.object.is_a?(Hostgroup) ? roles_attrs(f.object.inherited_and_own_ansible_roles) : roles_attrs(f.object.all_ansible_roles) %>
+  <% class_name = f.object.is_a?(Hostgroup) ? 'Hostgroup' : 'Host' %>
+  <%= mount_react_component('AnsibleRolesSwitcher', '#ansible_roles_switcher', { :initialAssignedRoles => roles,
+                                                                                 :inheritedRoleIds => f.object.inherited_ansible_roles.map(&:id),
+                                                                                 :availableRolesUrl => ui_ansible_roles_path,
+                                                                                 :resourceId => f.object.id,
+                                                                                 :resourceName => class_name,
+                                                                                 :canView => User.current.can?(:view_ansible_roles)
+                                                                                }.to_json) %>
 </div>

data/app/views/ui_ansible_roles/index.json.rabl

@@ -0,0 +1,3 @@
+collection @ui_ansible_roles
+
+extends "ui_ansible_roles/main"

data/app/views/ui_ansible_roles/main.json.rabl

@@ -0,0 +1,3 @@
+object @ansible_role
+
+extends "ui_ansible_roles/show"

data/app/views/ui_ansible_roles/show.json.rabl

@@ -0,0 +1,3 @@
+object @ansible_role
+
+extends "api/v2/ansible_roles/show"

data/config/routes.rb

@@ -58,7 +58,9 @@ Rails.application.routes.draw do
       end
     end
 
-    resources :ansible_variables, :except => [:show, :new, :create] do
+    resources :ui_ansible_roles, :only => [:index]
+
+    resources :ansible_variables, :except => [:show] do
       resources :lookup_values, :only => [:index, :create, :update, :destroy]
       collection do
         get :import
@@ -82,7 +84,7 @@ Rails.application.routes.draw do
           end
         end
 
-        resources :ansible_variables, :only => [:show, :index, :destroy, :update] do
+        resources :ansible_variables, :only => [:show, :index, :destroy, :update, :create] do
           collection do
             put :import
             put :obsolete

data/db/migrate/20190318094437_add_imported_attr_to_ansible_variables.rb

@@ -0,0 +1,15 @@
+class AddImportedAttrToAnsibleVariables < ActiveRecord::Migration[5.2]
+  def up
+    add_column :lookup_keys, :imported, :boolean
+
+    AnsibleVariable.find_in_batches do |batch|
+      batch.map do |variable|
+        variable.update_attribute :imported, true
+      end
+    end
+  end
+
+  def down
+    remove_column :lookup_keys, :imported
+  end
+end

data/lib/foreman_ansible/register.rb

@@ -2,7 +2,7 @@
 
 # rubocop:disable BlockLength
 Foreman::Plugin.register :foreman_ansible do
-  requires_foreman '>= 1.16'
+  requires_foreman '>= 1.21'
 
   security_block :foreman_ansible do
     permission :play_roles_on_host,
@@ -17,7 +17,8 @@ Foreman::Plugin.register :foreman_ansible do
                :resource_type => 'Hostgroup'
     permission :view_ansible_roles,
                { :ansible_roles => [:index, :auto_complete_search],
-                 :'api/v2/ansible_roles' => [:index, :show, :fetch] },
+                 :'api/v2/ansible_roles' => [:index, :show, :fetch],
+                 :ui_ansible_roles => [:index] },
                :resource_type => 'AnsibleRole'
     permission :destroy_ansible_roles,
                { :ansible_roles => [:destroy],
@@ -44,6 +45,12 @@ Foreman::Plugin.register :foreman_ansible do
                  :'api/v2/ansible_variables' => [:destroy, :obsolete]
                },
                :resource_type => 'AnsibleVariable'
+    permission :create_ansible_variables,
+               {
+                 :ansible_variables => [:new, :create],
+                 :'api/v2/ansible_variables' => [:create]
+               },
+               :resource_type => 'AnsibleVariable'
     permission :import_ansible_variables,
                {
                  :ansible_variables => [:import, :confirm_import],
@@ -69,6 +76,10 @@ Foreman::Plugin.register :foreman_ansible do
         :view_ansible_roles, :destroy_ansible_roles,
         :import_ansible_roles]
 
+  role 'Ansible Tower Inventory Reader',
+       [:view_hosts, :view_hostgroups, :view_facts],
+       'Permissions required for the user which is used by Ansible Tower Dynamic Inventory Item'
+
   add_all_permissions_to_default_roles
   extend_template_helpers ForemanAnsible::RendererMethods
   allowed_template_helpers :insights_remediation

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 = '2.3.2'
+  VERSION = '2.3.3'
 end