hybrid_platforms_conductor 33.0.4 → 33.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bedd336d4e713f15ac516cd61d50cf964c2a25051fbb624e1ca75196ccd9dd39
-  data.tar.gz: 2d75b29868194072733dd273f6d2f17ca001cc167f446105b72381e1620c57a1
+  metadata.gz: b6645a2b01eb498947d9d5e39803a664a18b263f5074d75b2f0191989df050c8
+  data.tar.gz: 4191d477ce05a66619be72968cbbc67d2e906e33f2a9cd797fb18ca3122712de
 SHA512:
-  metadata.gz: f4333f83879592e3089ce6cf0e40609efab2435fe9bbecae699462871bab7c3df6b5e44da476b823dfde6f0f1868d1cbd5ca621a076ca147355950b1610b6e38
-  data.tar.gz: e0f21e5ccec863d19c93fa1f36bcf51ce1be1a7b1e7bcdbcf83329914df3b3c852eb43b8af2e178b68f94305584c406007de0d47acc713f8170e7b7bba3da621
+  metadata.gz: 332a36210a05466e95172a3b2eececb081deea896fc19252286c1220fc18ceed70e29b29674127cb7ff51a094ff9358afc1c151f55d0d2135fa987a5d4534002
+  data.tar.gz: 2f7b69283769da47dd742f10b3546cf6625422c16fcdeed9882ade4c6f6a5eef2e1e795b10bc5dd2fa93a8d0bc6fcd0e33fab57fa5c4707f6a2a5afb1ee4df36

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# [v33.1.0](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.0.4...v33.1.0) (2021-06-18 11:37:28)
+
+## Global changes
+### Patches
+
+* [[Feature(connector_local)] [#68] Add sudo support when copying files using the local connector](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/7725beca4429de0e81bcac0c0ac4fe149e625da2)
+
+## Changes for connector_local
+### Features
+
+* [[Feature(connector_local)] [#68] Add sudo support when copying files using the local connector](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/7725beca4429de0e81bcac0c0ac4fe149e625da2)
+
 # [v33.0.4](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.0.3...v33.0.4) (2021-06-18 10:09:57)
 
 ## Global changes

data/lib/hybrid_platforms_conductor/hpc_plugins/connector/local.rb

@@ -75,7 +75,11 @@ module HybridPlatformsConductor
         def remote_copy(from, to, sudo: false, owner: nil, group: nil)
           # If the destination is a relative path, prepend the workspace dir to it.
           to = "#{workspace_for(@node)}/#{to}" unless to.start_with?('/')
-          FileUtils.cp_r from, to
+          if sudo
+            run_cmd "#{@nodes_handler.sudo_on(@node)} cp -r \"#{from}\" \"#{to}\""
+          else
+            FileUtils.cp_r from, to unless @cmd_runner.dry_run
+          end
         end
         # rubocop:enable Lint/UnusedMethodArgument
 

data/lib/hybrid_platforms_conductor/version.rb

@@ -1,5 +1,5 @@
 module HybridPlatformsConductor
 
-  VERSION = '33.0.4'
+  VERSION = '33.1.0'
 
 end

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/local/remote_actions_spec.rb

@@ -87,6 +87,43 @@ describe HybridPlatformsConductor::ActionsExecutor do
         end
       end
 
+      it 'does not copy files remotely in dry-run mode' do
+        with_test_platform_for_remote_testing do
+          test_cmd_runner.dry_run = true
+          expect(FileUtils).not_to receive(:cp_r)
+          test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir')
+        end
+      end
+
+      it 'copies files remotely with sudo' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [
+            [
+              'sudo -u root cp -r "/path/to/src.file" "/remote_path/to/dst.dir"',
+              proc { [0, '', ''] }
+            ]
+          ]
+        ) do
+          test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir', sudo: true)
+        end
+      end
+
+      it 'copies files remotely with a different sudo' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [
+            [
+              'other_sudo --user root cp -r "/path/to/src.file" "/remote_path/to/dst.dir"',
+              proc { [0, '', ''] }
+            ]
+          ],
+          additional_config: <<~'EO_CONFIG'
+            sudo_for { |user| "other_sudo --user #{user}" }
+          EO_CONFIG
+        ) do
+          test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir', sudo: true)
+        end
+      end
+
       it 'copies files remotely with timeout' do
         with_test_platform_for_remote_testing(
           timeout: 5
@@ -103,6 +140,35 @@ describe HybridPlatformsConductor::ActionsExecutor do
         end
       end
 
+      it 'copies relative files remotely with sudo' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [
+            [
+              'sudo -u root cp -r "/path/to/src.file" "/tmp/hpc_local_workspaces/node/to/dst.dir"',
+              proc { [0, '', ''] }
+            ]
+          ]
+        ) do
+          test_connector.remote_copy('/path/to/src.file', 'to/dst.dir', sudo: true)
+        end
+      end
+
+      it 'copies relative files remotely with a different sudo' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [
+            [
+              'other_sudo --user root cp -r "/path/to/src.file" "/tmp/hpc_local_workspaces/node/to/dst.dir"',
+              proc { [0, '', ''] }
+            ]
+          ],
+          additional_config: <<~'EO_CONFIG'
+            sudo_for { |user| "other_sudo --user #{user}" }
+          EO_CONFIG
+        ) do
+          test_connector.remote_copy('/path/to/src.file', 'to/dst.dir', sudo: true)
+        end
+      end
+
     end
 
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hybrid_platforms_conductor
 version: !ruby/object:Gem::Version
-  version: 33.0.4
+  version: 33.1.0
 platform: ruby
 authors:
 - Muriel Salvan