foreman_remote_execution 17.1.0 → 17.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 248ba991802a19c3ad70d4c93a1a80931065c77f5e075520c01d35d07d3ef4e3
-  data.tar.gz: 83cad360b0893506f1ace4b8c38e60bdab716a74aaa2ff0a741908059f92e56f
+  metadata.gz: e240991349f9cb6bc60f979202f8de2bde03d3d8100c13e701469aa2f8e29a97
+  data.tar.gz: f9936d90d62ce33221969367e5625cdf750c36922855142f460f46b18a8bc925
 SHA512:
-  metadata.gz: d4221e0916237573f00934dac48f0e169cb78ac278ec127b6f57e72cb8982ba12e79624a082986efe878ccbb325dd426ee41dcf0f2a155e08830791e389e3124
-  data.tar.gz: 927fa72b970f31090f729ade6acdbf7b73f33c4ca337af7e3ae4f0dfc998d0f77bc69a6bc32ede1ba3a052a46e98acf38d016277ab0085c75b0bd1edeec93068
+  metadata.gz: eaf96768e4b112d387e660cca80d6b9dd360dd1145cf40b5540ea428bdf4bf3482e47591a05d2e2f0bcc05e36808e8163346258f323cc7f7f60917db3b930686
+  data.tar.gz: 56dbf3ac2a8cb8f90c8ae719fc5213c2fbf40588812bf98552fa8bdd967b6d1150632543dfb6992df09050d74813dd3eeb367cc857c3709a921a404b3772c823

data/app/models/remote_execution_provider.rb

@@ -76,6 +76,17 @@ class RemoteExecutionProvider
       [true, 'true', 'True', 'TRUE', '1'].include?(setting)
     end
 
+    def remote_working_dir(host)
+      dir = host_setting(host, :remote_execution_remote_working_dir).to_s.strip
+      unless Pathname.new(dir).absolute?
+        raise ::Foreman::Exception.new(
+          N_('Remote working directory "%{current_value}" is not an absolute path'),
+          current_value: dir
+        )
+      end
+      dir
+    end
+
     def effective_user_password(host)
       host_setting(host, :remote_execution_effective_user_password)
     end

data/app/models/script_execution_provider.rb

@@ -1,11 +1,14 @@
 class ScriptExecutionProvider < RemoteExecutionProvider
   class << self
     def proxy_command_options(template_invocation, host)
-      super.merge(:ssh_user => ssh_user(host, template_invocation.job_invocation),
-        :effective_user => effective_user(template_invocation),
-        :effective_user_method => effective_user_method(host),
-        :cleanup_working_dirs => cleanup_working_dirs?(host),
-        :ssh_port => ssh_port(host))
+      super.merge(
+        ssh_user: ssh_user(host, template_invocation.job_invocation),
+        effective_user: effective_user(template_invocation),
+        effective_user_method: effective_user_method(host),
+        cleanup_working_dirs: cleanup_working_dirs?(host),
+        remote_working_dir: remote_working_dir(host),
+        ssh_port: ssh_port(host)
+      )
     end
 
     def humanized_name

data/lib/foreman_remote_execution/plugin.rb

@@ -86,6 +86,15 @@ Foreman::Plugin.register :foreman_remote_execution do
         description: N_('When enabled, working directories will be removed after task completion. You may override this per host by setting a parameter called remote_execution_cleanup_working_dirs.'),
         default: true,
         full_name: N_('Cleanup working directories')
+      setting 'remote_execution_remote_working_dir',
+        type: :string,
+        description: N_('Directory on the host where remote execution jobs in push mode place and run their scripts. '\
+                        'You can override this setting per host with the `remote_execution_remote_working_dir` host parameter.'),
+        default: '/var/tmp',
+        full_name: N_('Remote working directory')
+      validates 'remote_execution_remote_working_dir',
+        ->(value) { Pathname.new(value.to_s.strip).absolute? },
+        message: N_('must be an absolute path')
       setting 'remote_execution_cockpit_url',
         type: :string,
         description: N_('Where to find the Cockpit instance for the Web Console button.  By default, no button is shown.'),

data/lib/foreman_remote_execution/version.rb

@@ -1,3 +1,3 @@
 module ForemanRemoteExecution
-  VERSION = '17.1.0'.freeze
+  VERSION = '17.2.0'.freeze
 end

data/test/unit/remote_execution_provider_test.rb

@@ -202,6 +202,62 @@ class RemoteExecutionProviderTest < ActiveSupport::TestCase
       end
     end
 
+    describe 'remote working directory' do
+      it 'defaults to the value of the global setting' do
+        assert_equal '/var/tmp', proxy_options[:remote_working_dir]
+      end
+
+      it 'updates the value via settings' do
+        Setting[:remote_execution_remote_working_dir] = '/opt/rex'
+        assert_equal '/opt/rex', proxy_options[:remote_working_dir]
+      end
+
+      it 'takes the value from host parameters over the global setting' do
+        Setting[:remote_execution_remote_working_dir] = '/opt/rex'
+        host.host_parameters << FactoryBot.build(:host_parameter, name: 'remote_execution_remote_working_dir', value: '/opt/host')
+        host.clear_host_parameters_cache!
+        assert_equal '/opt/host', proxy_options[:remote_working_dir]
+      end
+
+      it 'takes the value inherited from the host group over the global setting' do
+        Setting[:remote_execution_remote_working_dir] = '/opt/rex'
+        hostgroup = FactoryBot.create(:hostgroup)
+        hostgroup.group_parameters << FactoryBot.build(:hostgroup_parameter, name: 'remote_execution_remote_working_dir', value: '/opt/hostgroup')
+        host.update!(hostgroup: hostgroup)
+        host.clear_host_parameters_cache!
+        assert_equal '/opt/hostgroup', proxy_options[:remote_working_dir]
+      end
+
+      it 'strips surrounding whitespace' do
+        Setting[:remote_execution_remote_working_dir] = ' /opt/rex '
+        assert_equal '/opt/rex', proxy_options[:remote_working_dir]
+      end
+
+      it 'rejects a setting value that is not an absolute path' do
+        assert_raises(ActiveRecord::RecordInvalid) do
+          Setting[:remote_execution_remote_working_dir] = 'opt/rex'
+        end
+      end
+
+      it 'rejects an empty setting value' do
+        assert_raises(ActiveRecord::RecordInvalid) do
+          Setting[:remote_execution_remote_working_dir] = ''
+        end
+      end
+
+      it 'raises when the host parameter is not an absolute path' do
+        host.host_parameters << FactoryBot.build(:host_parameter, name: 'remote_execution_remote_working_dir', value: 'opt/host')
+        host.clear_host_parameters_cache!
+        assert_raises(::Foreman::Exception) { proxy_options }
+      end
+
+      it 'raises when the host parameter is empty' do
+        host.host_parameters << FactoryBot.build(:host_parameter, name: 'remote_execution_remote_working_dir', value: '')
+        host.clear_host_parameters_cache!
+        assert_raises(::Foreman::Exception) { proxy_options }
+      end
+    end
+
     describe '#find_ip_or_hostname' do
       let(:host) do
         FactoryBot.create(:host) do |host|

data/webpack/JobInvocationDetail/CheckboxesActions.js

@@ -86,6 +86,7 @@ const ActionsKebab = ({
           id="toggle-kebab"
           isExpanded={isDropdownOpen}
           onClick={() => setIsDropdownOpen(prev => !prev)}
+          ouiaId="actions-kebab-toggle"
           ref={toggleRef}
           variant="plain"
         >

data/webpack/JobInvocationDetail/DropdownFilter.js

@@ -22,6 +22,7 @@ const DropdownFilter = ({ dropdownFilter, setDropdownFilter }) => {
       ref={toggleRef}
       onClick={() => setIsOpen(!isOpen)}
       isExpanded={isOpen}
+      ouiaId="host-status-select-toggle"
       style={{
         width: '200px',
       }}

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 17.1.0
+  version: 17.2.0
 platform: ruby
 authors:
 - Foreman Remote Execution team
 bindir: bin
 cert_chain: []
-date: 2026-08-17 00:00:00.000000000 Z
+date: 2026-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface