foreman_remote_execution 5.0.8 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a2e413a1866d81800d45a4e108897ffb2c219abbe3db4273f7276f44ae94267
-  data.tar.gz: da9d69d1268afa1e3a7acd289f3a8c4163364eb06d54eb0a1df780300a889e45
+  metadata.gz: 831ec5aca2ec4aaa3df1229f7addd9a968c7c49b0fc49a15253e33d2ab0f5689
+  data.tar.gz: 07a4f17f0d4d694c4b927291ee8e55dafd839cf902a9ef06c79b227fdf65760f
 SHA512:
-  metadata.gz: 77af52b52f1e49a2f855cec4ad19209248df85915c9411630a842e4446dd0eb9c0f73acccf4d91a42730d32cf6bb6caed916a351e8ed8c6a8211c8836fc281b2
-  data.tar.gz: 917f3540ee35e95fb431ebc37e69e852ffd6fbbcd82e5ddbfb6b5ddfc504f85ce1950ab143e7f40d7c481f56a87b3a91887cb39bafd9bcfd8df73b7fe29c043b
+  metadata.gz: 75fd11cfe45b5223a2427561c0859c39f19923535283fa919927af9b4ffd67ea56a4c9b791632e73c72c4dff1946f2e1046cb10a1f84ed7dae0e686daff2d90d
+  data.tar.gz: 03557c309e25a902c0def6e7575ad74a02e724482da823b4580f4bbe9b80358eadb2e0781b832f45a2e9101196447edf887476fa79782a3d6ee125640dccd92a

data/Gemfile

@@ -2,4 +2,4 @@ source 'http://rubygems.org'
 
 gemspec :name => 'foreman_remote_execution'
 
-gem 'theforeman-rubocop', '~> 0.1.0.pre'
+gem 'theforeman-rubocop', '~> 0.1.1'

data/app/controllers/foreman_remote_execution/concerns/api/v2/registration_controller_extensions.rb

@@ -13,7 +13,6 @@ module ForemanRemoteExecution
 
       def host_setup_extension
         remote_execution_interface
-        reset_host_known_keys! unless @host.new_record?
         super
       end
 
@@ -22,10 +21,6 @@ module ForemanRemoteExecution
 
         @host.set_execution_interface(params['remote_execution_interface'])
       end
-
-      def reset_host_known_keys!
-        @host.host_proxy_invocations.destroy_all
-      end
     end
   end
 end

data/app/helpers/concerns/foreman_remote_execution/hosts_helper_extensions.rb

@@ -8,7 +8,7 @@ module ForemanRemoteExecution
 
     def multiple_actions
       res = super
-      res += [ [_('Schedule Remote Job'), new_job_invocation_path, false] ] if can_schedule_jobs?
+      res += [ [_('Schedule Remote Job'), new_job_invocation_path, false] ] if authorized_for(controller: :job_invocations, action: :new)
       res
     end
 
@@ -22,20 +22,20 @@ module ForemanRemoteExecution
       end
     end
 
-    def rex_host_features(host, *_rest)
-      return [] unless can_execute_on_host?(host)
+    def rex_host_features(*args)
+      return unless authorized_for(controller: :job_invocations, action: :create)
       RemoteExecutionFeature.with_host_action_button.order(:label).map do |feature|
-        link_to(_('%s') % feature.name, job_invocations_path(:host_ids => [host.id], :feature => feature.label), :method => :post)
+        link_to(_('%s') % feature.name, job_invocations_path(:host_ids => [args.first.id], :feature => feature.label), :method => :post)
       end
     end
 
-    def schedule_job_button(host, *_rest)
-      return unless can_execute_on_host?(host)
-      link_to(_('Schedule Remote Job'), new_job_invocation_path(:host_ids => [host.id]), :id => :run_button, :class => 'btn btn-default')
+    def schedule_job_button(*args)
+      return unless authorized_for(controller: :job_invocations, action: :new)
+      link_to(_('Schedule Remote Job'), new_job_invocation_path(:host_ids => [args.first.id]), :id => :run_button, :class => 'btn btn-default')
     end
 
     def web_console_button(host, *args)
-      return if !authorized_for(permission: 'cockpit_hosts', auth_object: host) || !can_execute_on_infrastructure_host?(host)
+      return unless authorized_for(permission: 'cockpit_hosts', auth_object: host)
 
       url = SSHExecutionProvider.cockpit_url_for_host(host.name)
       url ? link_to(_('Web Console'), url, :class => 'btn btn-default', :id => :'web-console-button', :target => '_new') : nil
@@ -46,17 +46,6 @@ module ForemanRemoteExecution
         button_group(web_console_button(*args)))
       super(*args)
     end
-
-    def can_schedule_jobs?
-      authorized_for(controller: :job_invocations, action: :create)
-    end
-
-    def can_execute_on_host?(host)
-      can_schedule_jobs? && can_execute_on_infrastructure_host?(host)
-    end
-
-    def can_execute_on_infrastructure_host?(host)
-      !host.infrastructure_host? || User.current.can?(:execute_jobs_on_infrastructure_hosts)
-    end
   end
+
 end

data/app/helpers/remote_execution_helper.rb

@@ -31,7 +31,7 @@ module RemoteExecutionHelper
 
     if authorized_for(hash_for_host_path(host).merge(auth_object: host, permission: :view_hosts, authorizer: job_hosts_authorizer))
       links << { title: _('Host detail'),
-        action: { href: current_host_details_path(host), 'data-method': 'get', id: "#{host.name}-actions-detail" } }
+        action: { href: host_path(host), 'data-method': 'get', id: "#{host.name}-actions-detail" } }
     end
 
     if authorized_for(hash_for_rerun_job_invocation_path(id: job_invocation, host_ids: [ host.id ], authorizer: job_hosts_authorizer))
@@ -147,10 +147,9 @@ module RemoteExecutionHelper
     end
   end
 
-  def invocation_description(invocation, keep_tooltip: true)
+  def invocation_description(invocation)
     description = invocation.description.try(:capitalize) || invocation.job_category
-    description = trunc_with_tooltip(description, 80) if keep_tooltip
-    description
+    trunc_with_tooltip(description, 80)
   end
 
   def invocation_result(invocation, key)

data/app/lib/actions/remote_execution/run_host_job.rb

@@ -42,6 +42,8 @@ module Actions
 
         provider_type = template_invocation.template.provider_type.to_s
         proxy = determine_proxy!(proxy_selector, provider_type, host)
+        link!(proxy)
+        input[:proxy_id] = proxy.id
 
         renderer = InputTemplateRenderer.new(template_invocation.template, host, template_invocation)
         script = renderer.render

data/app/lib/foreman_remote_execution/renderer/scope/input.rb

@@ -98,7 +98,7 @@ module ForemanRemoteExecution
         def input(name)
           return template_input_values[name.to_s] if template_input_values.key?(name.to_s)
 
-          input = find_by_name(template.template_inputs_with_foreign, name) # rubocop:disable Rails/DynamicFindBy
+          input = find_by_name(template.template_inputs_with_foreign, name)
           if input
             @preview ? input.preview(self) : input.value(self)
           else

data/app/models/concerns/foreman_remote_execution/host_extensions.rb

@@ -46,10 +46,6 @@ module ForemanRemoteExecution
       end
     end
 
-    def cockpit_url
-      SSHExecutionProvider.cockpit_url_for_host(self.name)
-    end
-
     def execution_status(options = {})
       @execution_status ||= get_status(HostStatus::ExecutionStatus).to_status(options)
     end
@@ -125,10 +121,6 @@ module ForemanRemoteExecution
       @cached_rex_host_params_hash = nil
     end
 
-    def infrastructure_host?
-      infrastructure_facet&.foreman_instance || infrastructure_facet&.smart_proxy_id
-    end
-
     private
 
     def extend_host_params_hash(params)

data/app/models/concerns/foreman_remote_execution/nic_extensions.rb

@@ -3,18 +3,16 @@ module ForemanRemoteExecution
     extend ActiveSupport::Concern
 
     included do
+      before_validation :set_execution_flag
       validate :exclusive_execution_interface
-      before_validation :move_execution_flag
     end
 
     private
 
-    def move_execution_flag
-      return unless host && self.execution?
+    def set_execution_flag
+      return unless primary? && host.present?
 
-      host.interfaces
-          .select { |i| i.execution? && i != self }
-          .each { |i| i.execution = false }
+      self.execution = true if host.interfaces.detect(&:execution).nil?
     end
 
     def exclusive_execution_interface

data/app/models/job_invocation.rb

@@ -15,8 +15,10 @@ class JobInvocation < ApplicationRecord
 
   belongs_to :targeting, :dependent => :destroy
   has_many :all_template_invocations, :inverse_of => :job_invocation, :dependent => :destroy, :class_name => 'TemplateInvocation'
+  # rubocop:disable Rails/HasManyOrHasOneDependent
   has_many :template_invocations, -> { where('template_invocations.host_id IS NOT NULL') }, :inverse_of => :job_invocation
   has_many :pattern_template_invocations, -> { where('template_invocations.host_id IS NULL') }, :inverse_of => :job_invocation, :class_name => 'TemplateInvocation'
+  # rubocop:enable Rails/HasManyOrHasOneDependent
   has_many :pattern_templates, :through => :pattern_template_invocations, :source => :template
 
   validates :targeting, :presence => true

data/app/models/job_invocation_composer.rb

@@ -217,7 +217,7 @@ class JobInvocationComposer
     def format_datetime(datetime)
       return datetime if datetime.blank?
 
-      Time.zone.parse(datetime).strftime('%Y-%m-%d %H:%M')
+      Time.parse(datetime).utc.strftime('%Y-%m-%d %H:%M')
     end
   end
 

data/app/models/job_template.rb

@@ -17,8 +17,10 @@ class JobTemplate < ::Template
 
   has_many :audits, :as => :auditable, :class_name => Audited.audit_class.name, :dependent => :nullify
   has_many :all_template_invocations, :dependent => :destroy, :foreign_key => 'template_id', :class_name => 'TemplateInvocation'
+  # rubocop:disable Rails/HasManyOrHasOneDependent
   has_many :template_invocations, -> { where('host_id IS NOT NULL') }, :foreign_key => 'template_id'
   has_many :pattern_template_invocations, -> { where('host_id IS NULL') }, :foreign_key => 'template_id', :class_name => 'TemplateInvocation'
+  # rubocop:enable Rails/HasManyOrHasOneDependent
   has_many :remote_execution_features, :dependent => :nullify
 
   # these can't be shared in parent class, scoped search can't handle STI properly
@@ -192,7 +194,7 @@ class JobTemplate < ::Template
   end
 
   def default_input_values(ignore_keys)
-    result = self.template_inputs_with_foreign.select { |ti| !ti.required? && ti.user_template_input? }.map { |ti| ti.name.to_s }
+    result = self.template_inputs_with_foreign.select { |ti| !ti.required? && ti.input_type == 'user' }.map { |ti| ti.name.to_s }
     result -= ignore_keys.map(&:to_s)
     Hash[result.map { |k| [ k, nil ] }]
   end

data/app/models/targeting.rb

@@ -67,7 +67,7 @@ class Targeting < ApplicationRecord
     return '' if ids.empty?
 
     hosts = Host.execution_scope.where(:id => ids).distinct.pluck(:name)
-    hosts.any? ? "name ^ (#{hosts.join(', ')})" : ''
+    "name ^ (#{hosts.join(', ')})"
   end
 
   def resolved?

data/app/views/dashboard/_latest-jobs.html.erb

@@ -10,7 +10,7 @@
     </tr>
     <% JobInvocation.latest_jobs.each do |invocation| %>
       <tr>
-        <td class="ellipsis"><%= link_to_if_authorized invocation_description(invocation, keep_tooltip: false), hash_for_job_invocation_path(invocation).merge(:auth_object => invocation, :permission => :view_job_invocations, :authorizer => authorizer) %></td>
+        <td class="ellipsis"><%= link_to_if_authorized invocation_description(invocation), hash_for_job_invocation_path(invocation).merge(:auth_object => invocation, :permission => :view_job_invocations, :authorizer => authorizer) %></td>
         <td><%= link_to_invocation_task_if_authorized(invocation) %></td>
         <td><%= time_in_words_span(invocation.start_at) %></td>
       </tr>

data/app/views/job_invocations/_preview_hosts_list.html.erb

@@ -7,7 +7,7 @@
 <% if @hosts.any? -%>
   <ul>
     <% @hosts.each do |host| -%>
-        <li><%= link_to h(host.name), current_host_details_path(host), :target => '_blank' %></li>
+        <li><%= link_to h(host.name), host_path(host), :target => '_blank' %></li>
     <% end -%>
 
     <% if @additional > 0 -%>

data/app/views/job_templates/_custom_tabs.html.erb

@@ -1,14 +1,9 @@
 <div class="tab-pane" id="template_job">
 
-  <%= field(f, :job_category, :label => _('Job category')) do %>
-    <%= auto_complete_search(:job_category,
-                             f.object.job_category,
-                             :placeholder => _("Job category") + ' ...',
-                             :name => 'job_template[job_category]',
-                             :id => 'search',
-                             :disabled => @template.locked?) %>
-  <% end %>
-
+  <%= autocomplete_f(f, :job_category,
+        :search_query => '',
+        :placeholder => _("Job category") + ' ...',
+        :disabled => @template.locked?) %>
   <%= text_f f, :description_format,
              :disabled => @template.locked?,
              :label_help => description_format_help %>

data/app/views/template_invocations/show.html.erb

@@ -28,7 +28,13 @@ end
   </div>
 </div>
 <% if @host %>
-  <h3><%= _('Target: ') %><%= link_to(@host.name, current_host_details_path(@host)) %></h3>
+  <% proxy_id = @template_invocation_task.input[:proxy_id] %>
+  <h3>
+    <%= _('Target: ') %><%= link_to(@host.name, host_path(@host)) %>
+    <% if proxy_id && proxy = SmartProxy.find_by(id: proxy_id) %>
+      <%= _('using Smart Proxy') %> <%= link_to(proxy.name, smart_proxy_path(proxy)) %>
+    <% end %>
+  </h3>
 
   <div class="preview hidden">
     <%= preview_box(@template_invocation, @host) %>

data/app/views/templates/ssh/package_action.erb

@@ -69,11 +69,8 @@ handle_zypp_res_codes () {
 
   if [ "${ZYPP_RES_CODES[$RETVAL]}" != "" ]; then
     echo ${ZYPP_RES_CODES[$RETVAL]}
-  fi
-  if [[ $RETVAL -ge 100 && $RETVAL -le 103 ]]; then
     RETVAL=0
   fi
-
   return $RETVAL
 }
 <% end -%>

data/lib/foreman_remote_execution/engine.rb

@@ -152,7 +152,7 @@ module ForemanRemoteExecution
             setting 'remote_execution_job_invocation_report_template',
               type: :string,
               description: N_('Select a report template used for generating a report for a particular remote execution job'),
-              default: 'Jobs - Invocation report template',
+              default: 'Job invocation - report template',
               full_name: N_('Job Invocation Report Template'),
               collection: proc { ForemanRemoteExecution.job_invocation_report_templates_select }
           end
@@ -214,11 +214,7 @@ module ForemanRemoteExecution
         role 'Remote Execution User', USER_PERMISSIONS, 'Role with permissions to run remote execution jobs against hosts'
         role 'Remote Execution Manager', MANAGER_PERMISSIONS, 'Role with permissions to manage job templates, remote execution features, cancel jobs and view audit logs'
 
-        add_all_permissions_to_default_roles(except: [:execute_jobs_on_infrastructure_hosts])
-        add_permissions_to_default_roles({
-          Role::MANAGER => [:execute_jobs_on_infrastructure_hosts],
-          Role::SITE_MANAGER => USER_PERMISSIONS + [:execute_jobs_on_infrastructure_hosts],
-        })
+        add_all_permissions_to_default_roles
 
         # add menu entry
         menu :top_menu, :job_templates,
@@ -265,7 +261,6 @@ module ForemanRemoteExecution
         extend_rabl_template 'api/v2/smart_proxies/main', 'api/v2/smart_proxies/pubkey'
         extend_rabl_template 'api/v2/interfaces/main', 'api/v2/interfaces/execution_flag'
         extend_rabl_template 'api/v2/subnets/show', 'api/v2/subnets/remote_execution_proxies'
-        extend_rabl_template 'api/v2/hosts/main', 'api/v2/host/main'
         parameter_filter ::Subnet, :remote_execution_proxy_ids
         describe_host { overview_buttons_provider :host_overview_buttons }
 

data/lib/foreman_remote_execution/version.rb

@@ -1,3 +1,3 @@
 module ForemanRemoteExecution
-  VERSION = '5.0.8'.freeze
+  VERSION = '5.1.0'.freeze
 end

data/package.json

@@ -21,19 +21,20 @@
   },
   "devDependencies": {
     "@babel/core": "^7.7.0",
-    "@theforeman/builder": "^10.1.0",
-    "@theforeman/eslint-plugin-foreman": "^10.1.0",
-    "@theforeman/stories": "^10.1.0",
-    "@theforeman/test": "^10.1.0",
-    "@theforeman/vendor-dev": "^10.1.0",
+    "@theforeman/builder": "^8.16.0",
+    "@theforeman/eslint-plugin-foreman": "^8.16.0",
+    "@theforeman/stories": "^8.16.0",
+    "@theforeman/test": "^8.16.0",
+    "@theforeman/vendor-dev": "^8.16.0",
     "babel-eslint": "^10.0.0",
     "eslint": "^6.8.0",
     "prettier": "^1.19.1",
+    "@patternfly/react-catalog-view-extension": "^4.8.126",
     "redux-mock-store": "^1.2.2",
     "graphql-tag": "^2.11.0",
     "graphql": "^15.5.0"
   },
   "peerDependencies": {
-    "@theforeman/vendor": "^10.1.0"
+    "@theforeman/vendor": "^8.16.0"
   }
 }

data/test/unit/concerns/host_extensions_test.rb

@@ -67,8 +67,7 @@ class ForemanRemoteExecutionHostExtensionsTest < ActiveSupport::TestCase
     it 'should only have one execution interface' do
       host.interfaces << FactoryBot.build(:nic_managed)
       host.interfaces.each { |interface| interface.execution = true }
-      _(host).must_be :valid?
-      _(host.interfaces.count(&:execution?)).must_equal 1
+      _(host).wont_be :valid?
     end
 
     it 'returns the execution interface' do

data/test/unit/job_invocation_report_template_test.rb

@@ -9,7 +9,7 @@ class JobReportTemplateTest < ActiveSupport::TestCase
 
   context 'with valid job invocation report template' do
     let(:job_invocation_template) do
-      file_path = File.read(File.expand_path(Rails.root + "app/views/unattended/report_templates/jobs_-_invocation_report_template.erb"))
+      file_path = File.read(File.expand_path(Rails.root + "app/views/unattended/report_templates/job_invocation_-_report_template.erb"))
       template = ReportTemplate.import_without_save("Job Invocation Report Template", file_path)
       template.save!
       template

data/test/unit/targeting_test.rb

@@ -61,7 +61,7 @@ class TargetingTest < ActiveSupport::TestCase
       users(:one).destroy
     end
 
-    it { assert targeting.reload.user.nil? }
+    it { assert_nil targeting.reload.user }
     it do
       -> { targeting.resolve_hosts! }.must_raise(Foreman::Exception)
     end

data/webpack/JobWizard/steps/form/GroupedSelectField.js

@@ -69,8 +69,6 @@ export const GroupedSelectField = ({
         className="without_select2"
         onClear={onClear}
         menuAppendTo={() => document.body}
-        aria-labelledby={fieldId}
-        toggleAriaLabel={`${label} toggle`}
         {...props}
       >
         {options}

data/webpack/global_index.js

@@ -2,13 +2,9 @@ import { registerRoutes } from 'foremanReact/routes/RoutingService';
 import routes from './Routes/routes';
 import fillregistrationAdvanced from './react_app/extend/fillregistrationAdvanced';
 import fillRecentJobsCard from './react_app/extend/fillRecentJobsCard';
-import fillFeaturesDropdown from './react_app/extend/fillRexFeaturesDropdown';
-import fillKebabItems from './react_app/extend/fillKebabItems';
 import registerReducers from './react_app/extend/reducers';
 
 registerReducers();
 registerRoutes('foreman_remote_execution', routes);
-fillFeaturesDropdown();
 fillRecentJobsCard();
 fillregistrationAdvanced();
-fillKebabItems();

data/webpack/react_app/components/RecentJobsCard/RecentJobsCard.js

@@ -52,12 +52,7 @@ const RecentJobsCard = ({ hostDetails: { name, id } }) => {
         </DropdownItem>,
       ]}
     >
-      <Tabs
-        mountOnEnter
-        unmountOnExit
-        activeKey={activeTab}
-        onSelect={handleTabClick}
-      >
+      <Tabs mountOnEnter activeKey={activeTab} onSelect={handleTabClick}>
         <Tab
           eventKey={FINISHED_TAB}
           title={<TabTitleText>{__('Finished')}</TabTitleText>}

data/webpack/react_app/components/RecentJobsCard/RecentJobsTable.js

@@ -19,7 +19,7 @@ import { translate as __ } from 'foremanReact/common/I18n';
 import { foremanUrl } from 'foremanReact/common/helpers';
 
 import JobStatusIcon from './JobStatusIcon';
-import { JOB_API_URL, JOBS_IN_CARD, RECENT_JOBS_KEY } from './constants';
+import { JOB_API_URL, JOBS_IN_CARD } from './constants';
 
 const RecentJobsTable = ({ status, hostId }) => {
   const jobsUrl =
@@ -30,7 +30,7 @@ const RecentJobsTable = ({ status, hostId }) => {
   const {
     response: { results: jobs },
     status: responseStatus,
-  } = useAPI('get', jobsUrl, RECENT_JOBS_KEY);
+  } = useAPI('get', jobsUrl);
 
   return (
     <DataList aria-label="recent-jobs-table" isCompact>

data/webpack/react_app/components/RecentJobsCard/constants.js

@@ -10,4 +10,3 @@ export const JOB_BASE_URL = '/job_invocations?search=host+%3D+';
 export const JOB_API_URL =
   '/api/job_invocations?order=start_at+DESC&search=targeted_host_id%3D';
 export const JOBS_IN_CARD = 3;
-export const RECENT_JOBS_KEY = { key: 'RECENT_JOBS_KEY' };

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 5.0.8
+  version: 5.1.0
 platform: ruby
 authors:
 - Foreman Remote Execution team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-04 00:00:00.000000000 Z
+date: 2021-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -201,7 +201,6 @@ files:
 - app/views/api/v2/foreign_input_sets/index.json.rabl
 - app/views/api/v2/foreign_input_sets/main.json.rabl
 - app/views/api/v2/foreign_input_sets/show.json.rabl
-- app/views/api/v2/host/main.rabl
 - app/views/api/v2/interfaces/execution_flag.json.rabl
 - app/views/api/v2/job_invocations/base.json.rabl
 - app/views/api/v2/job_invocations/create.json.rabl
@@ -335,7 +334,6 @@ files:
 - extra/cockpit/foreman-cockpit.service
 - extra/cockpit/settings.yml.example
 - foreman_remote_execution.gemspec
-- jsconfig.json
 - lib/foreman_remote_execution.rb
 - lib/foreman_remote_execution/engine.rb
 - lib/foreman_remote_execution/version.rb
@@ -388,7 +386,6 @@ files:
 - test/test_plugin_helper.rb
 - test/unit/actions/run_host_job_test.rb
 - test/unit/actions/run_hosts_job_test.rb
-- test/unit/api_params_test.rb
 - test/unit/concerns/foreman_tasks_cleaner_extensions_test.rb
 - test/unit/concerns/host_extensions_test.rb
 - test/unit/concerns/nic_extensions_test.rb
@@ -479,10 +476,6 @@ files:
 - webpack/global_index.js
 - webpack/helpers.js
 - webpack/index.js
-- webpack/react_app/components/FeaturesDropdown/actions.js
-- webpack/react_app/components/FeaturesDropdown/constant.js
-- webpack/react_app/components/FeaturesDropdown/index.js
-- webpack/react_app/components/HostKebab/KebabItems.js
 - webpack/react_app/components/RecentJobsCard/JobStatusIcon.js
 - webpack/react_app/components/RecentJobsCard/RecentJobsCard.js
 - webpack/react_app/components/RecentJobsCard/RecentJobsTable.js
@@ -515,9 +508,7 @@ files:
 - webpack/react_app/components/jobInvocations/AggregateStatus/index.js
 - webpack/react_app/components/jobInvocations/AggregateStatus/index.test.js
 - webpack/react_app/components/jobInvocations/index.js
-- webpack/react_app/extend/fillKebabItems.js
 - webpack/react_app/extend/fillRecentJobsCard.js
-- webpack/react_app/extend/fillRexFeaturesDropdown.js
 - webpack/react_app/extend/fillregistrationAdvanced.js
 - webpack/react_app/extend/reducers.js
 - webpack/react_app/redux/actions/jobInvocations/index.js
@@ -546,7 +537,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.26
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: A plugin bringing remote execution to the Foreman, completing the config
@@ -573,7 +564,6 @@ test_files:
 - test/test_plugin_helper.rb
 - test/unit/actions/run_host_job_test.rb
 - test/unit/actions/run_hosts_job_test.rb
-- test/unit/api_params_test.rb
 - test/unit/concerns/foreman_tasks_cleaner_extensions_test.rb
 - test/unit/concerns/host_extensions_test.rb
 - test/unit/concerns/nic_extensions_test.rb

data/app/views/api/v2/host/main.rabl

@@ -1 +0,0 @@
-attributes :cockpit_url

data/jsconfig.json

@@ -1,8 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "es6",
-    "paths": {
-      "foremanReact/*": ["../foreman/webpack/assets/javascripts/react_app/*"]
-    }
-  }
-}

data/test/unit/api_params_test.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_plugin_helper'
-
-class ApiParamsTest < ActiveSupport::TestCase
-  describe '#format_datetime' do
-    let(:params) { JobInvocationComposer::ApiParams.allocate }
-
-    it 'leaves empty string as is' do
-      assert_equal params.send(:format_datetime, ''), ''
-    end
-
-    it 'honors explicitly supplied time zone' do
-      in_time_zone(ActiveSupport::TimeZone['America/New_York']) do
-        assert_equal '2022-07-08 08:53', params.send(:format_datetime, '2022-07-08 12:53:20 UTC')
-      end
-    end
-
-    it 'implicitly honors current user\'s time zone' do
-      in_time_zone(ActiveSupport::TimeZone['America/New_York']) do
-        assert_equal '2022-07-08 12:53', params.send(:format_datetime, '2022-07-08 12:53:20')
-      end
-    end
-  end
-
-  def in_time_zone(zone)
-    old_tz = Time.zone
-    Time.zone = zone
-    yield
-  ensure
-    Time.zone = old_tz
-  end
-end

data/webpack/react_app/components/FeaturesDropdown/actions.js

@@ -1,13 +0,0 @@
-import { foremanUrl } from 'foremanReact/common/helpers';
-import { sprintf, translate as __ } from 'foremanReact/common/I18n';
-import { post } from 'foremanReact/redux/API';
-
-export const runFeature = (hostId, feature, label) => dispatch => {
-  const url = foremanUrl(
-    `/job_invocations?feature=${feature}&host_ids%5B%5D=${hostId}`
-  );
-
-  const successToast = () => sprintf(__('%s job has been invoked'), label);
-  const errorToast = ({ message }) => message;
-  dispatch(post({ key: feature.toUpperCase(), url, successToast, errorToast }));
-};

data/webpack/react_app/components/FeaturesDropdown/constant.js

@@ -1,2 +0,0 @@
-export const REX_FEATURES_API = '/api/remote_execution_features';
-export const NEW_JOB_PAGE = '/job_invocations/new?host_ids%5B%5D';

data/webpack/react_app/components/FeaturesDropdown/index.js

@@ -1,74 +0,0 @@
-import PropTypes from 'prop-types';
-import React, { useState } from 'react';
-import { useDispatch } from 'react-redux';
-import {
-  DropdownItem,
-  Dropdown,
-  DropdownToggle,
-  DropdownToggleAction,
-} from '@patternfly/react-core';
-import { push } from 'connected-react-router';
-
-import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
-import { translate as __ } from 'foremanReact/common/I18n';
-import { foremanUrl } from 'foremanReact/common/helpers';
-import { STATUS } from 'foremanReact/constants';
-
-import { REX_FEATURES_API, NEW_JOB_PAGE } from './constant';
-import { runFeature } from './actions';
-
-const FeaturesDropdown = ({ hostId }) => {
-  const [isOpen, setIsOpen] = useState(false);
-  const {
-    response: { results: features },
-    status,
-  } = useAPI('get', foremanUrl(REX_FEATURES_API));
-
-  const dispatch = useDispatch();
-  const dropdownItems = features
-    ?.filter(feature => feature.host_action_button)
-    ?.map(({ name, label, id, description }) => (
-      <DropdownItem
-        onClick={() => dispatch(runFeature(hostId, label, name))}
-        key={id}
-        description={description}
-      >
-        {name}
-      </DropdownItem>
-    ));
-  const scheduleJob = [
-    <DropdownToggleAction
-      onClick={() => dispatch(push(`${NEW_JOB_PAGE}=${hostId}`))}
-      key="schedule-job-action"
-    >
-      {__('Schedule a job')}
-    </DropdownToggleAction>,
-  ];
-
-  return (
-    <Dropdown
-      alignments={{ default: 'right' }}
-      onSelect={() => setIsOpen(false)}
-      toggle={
-        <DropdownToggle
-          splitButtonItems={scheduleJob}
-          toggleVariant="primary"
-          onToggle={() => setIsOpen(prev => !prev)}
-          isDisabled={status === STATUS.PENDING}
-          splitButtonVariant="action"
-        />
-      }
-      isOpen={isOpen}
-      dropdownItems={dropdownItems}
-    />
-  );
-};
-
-FeaturesDropdown.propTypes = {
-  hostId: PropTypes.number,
-};
-FeaturesDropdown.defaultProps = {
-  hostId: undefined,
-};
-
-export default FeaturesDropdown;

data/webpack/react_app/components/HostKebab/KebabItems.js

@@ -1,27 +0,0 @@
-import React from 'react';
-import { useSelector } from 'react-redux';
-import { DropdownItem } from '@patternfly/react-core';
-import { CodeIcon } from '@patternfly/react-icons';
-import { selectAPIResponse } from 'foremanReact/redux/API/APISelectors';
-import { translate as __ } from 'foremanReact/common/I18n';
-import { HOST_DETAILS_KEY } from 'foremanReact/components/HostDetails/consts';
-
-const HostKebabItems = () => {
-  const { cockpit_url: consoleUrl } = useSelector(state =>
-    selectAPIResponse(state, HOST_DETAILS_KEY)
-  );
-
-  if (!consoleUrl) return null;
-  return (
-    <DropdownItem
-      icon={<CodeIcon />}
-      href={consoleUrl}
-      target="_blank"
-      rel="noreferrer"
-    >
-      {__('Web Console')}
-    </DropdownItem>
-  );
-};
-
-export default HostKebabItems;

data/webpack/react_app/extend/fillKebabItems.js

@@ -1,11 +0,0 @@
-import React from 'react';
-import { addGlobalFill } from 'foremanReact/components/common/Fill/GlobalFill';
-import KebabItems from '../components/HostKebab/KebabItems';
-
-export default () =>
-  addGlobalFill(
-    'host-details-kebab',
-    'rex-host-details-kebab-job',
-    <KebabItems key="rex-host-details-kebab-job" />,
-    100
-  );

data/webpack/react_app/extend/fillRexFeaturesDropdown.js

@@ -1,11 +0,0 @@
-import React from 'react';
-import { addGlobalFill } from 'foremanReact/components/common/Fill/GlobalFill';
-import FeaturesDropdown from '../components/FeaturesDropdown';
-
-export default () =>
-  addGlobalFill(
-    '_rex-host-features',
-    '_rex-host-features',
-    <FeaturesDropdown key="_rex-host-features" />,
-    1000
-  );