hybrid_platforms_conductor 33.9.1 → 33.9.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f6fe7f17295eff7ec9450d1ee910807a17904b8fa3c2bc5959449805b3c4aee
-  data.tar.gz: 3e39403d767099165e0765ba631effcb76b4d998ab7e34069d6d63ce9d75979a
+  metadata.gz: 03d8f9c77d6fd058348749c2ba567f905af52558207e281c8d20678775bfc85c
+  data.tar.gz: 56ca17b26a52bee7cff5659718a283368270815e5e61f1bc4cae0201da1d39c8
 SHA512:
-  metadata.gz: 2abe0958607c3980f84acfb55a1d1b4b5dd96e03ab94db21f1069a30e9026e7053fca09125c253c619bd3228772718acf744af6447450d8bb36277ee341c18e5
-  data.tar.gz: a14bfcaaf1bff4ca92206d54c7e2bf0d7367c8ccd1609a311bec0c9fb844e3ae1fd1aaf2836ab7e69446b98713bdfea227653da4c55b2c2d1c797ed9896465bb
+  metadata.gz: 6a97cab5245e2b0bcf27a869b783055560b3f0f95a70b3596fdba74fd64f4ddc41af715c54e66561ea59afdde83f382a5ff9e1fc05c9e85842276450e467cae9
+  data.tar.gz: 521ea609ddc3952fbb924fb333f40901ed28396b575899c5dae2a43256c2e821162de13bb94a2c846817816205d062960590cc3088f3d0f6da74821103a62519

data/CHANGELOG.md

@@ -1,3 +1,36 @@
+# [v33.9.5](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.9.4...v33.9.5) (2022-03-20 17:46:29)
+
+### Patches
+
+* [[#118] Fix linear strategy checks when merge commits are found](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/01464ae5bb5c39e95be6216fceed0e9ac00ae83e)
+
+# [v33.9.4](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.9.3...v33.9.4) (2022-03-20 16:20:04)
+
+### Patches
+
+* [Added missing tests and corrections from PR #119](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/e86be2f99e273e65bf9073c1b389a10ad53d7353)
+* [Merge branch 'rebase_pull_116'](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/4030ba58e0e9cf4ef96328a32cf4daa6226270c0)
+
+# [v33.9.3](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.9.2...v33.9.3) (2022-03-20 00:06:14)
+
+### Patches
+
+* [Fix semantic-release dependencies](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/68ec49fbec772c1de6971173830a85b76398de42)
+* [Work-around IceCube dependency issue and correct rubocop errors](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/68858e4d5fbb9d91fcee0525f56dfc40850af7df)
+* [fixing issue 118](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/bb917101c336119104889e5d6cedd34aef538f54)
+
+# [v33.9.2](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.9.1...v33.9.2) (2021-09-01 17:46:34)
+
+## Global changes
+### Patches
+
+* [[Fix(nodes_to_deploy)] [#112] Support UTF-8 encoding from log files](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/26a7bd2fd53d387bd972b5e1b60cac9be6541ce5)
+
+## Changes for nodes_to_deploy
+### Patches
+
+* [[Fix(nodes_to_deploy)] [#112] Support UTF-8 encoding from log files](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/26a7bd2fd53d387bd972b5e1b60cac9be6541ce5)
+
 # [v33.9.1](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v33.9.0...v33.9.1) (2021-09-01 16:59:59)
 
 ## Global changes

data/bin/last_deploys

@@ -8,12 +8,12 @@ possible_string_sorts = %i[
   node
   user
 ]
-possible_sorts = possible_string_sorts.map do |property_name|
+possible_sorts = possible_string_sorts.to_h do |property_name|
   [
     property_name,
     proc { |node, deploy_info| [deploy_info.key?(:error) || !deploy_info.key?(property_name) ? '' : deploy_info[property_name], node] }
   ]
-end.to_h
+end
 
 sort_by = :node
 sort_desc = false
@@ -34,7 +34,7 @@ raise "Unknown sort name: #{sort_by}. Should be one of #{possible_sorts.keys.joi
 
 sorted_deploy_info = deployer.
   deployment_info_from(nodes_handler.select_nodes(executable.selected_nodes.empty? ? [{ all: true }] : executable.selected_nodes)).
-  map do |node, deploy_info|
+  to_h do |node, deploy_info|
     decorated_deploy_info = deploy_info.merge(node: node)
     if deploy_info.key?(:deployment_info)
       decorated_deploy_info.merge!(deploy_info[:deployment_info])
@@ -43,7 +43,6 @@ sorted_deploy_info = deployer.
     decorated_deploy_info[:services] = deploy_info[:services].join(', ') if deploy_info.key?(:services)
     [node, decorated_deploy_info]
   end.
-  to_h.
   sort_by(&possible_sorts[sort_by])
 sorted_deploy_info.reverse! if sort_desc
 info_displayed = {

data/lib/hybrid_platforms_conductor/actions_executor.rb

@@ -146,7 +146,7 @@ module HybridPlatformsConductor
           actions_per_node[node].concat(resolved_nodes_actions)
         end
       end
-      result = actions_per_node.keys.map { |node| [node, nil] }.to_h
+      result = actions_per_node.keys.to_h { |node| [node, nil] }
       with_connections_prepared_to(nodes_needing_connectors, no_exception: true) do |connected_nodes|
         missing_nodes = []
         connected_nodes.each do |node, connector|
@@ -194,7 +194,7 @@ module HybridPlatformsConductor
     #     * *connected_nodes* (Hash<String, Connector or Symbol>): Prepared connectors (or Symbol in case of failure with no_exception), per node name
     def with_connections_prepared_to(nodes, no_exception: false)
       # Make sure every node needing connectors finds a connector
-      nodes_needing_connectors = nodes.map { |node| [node, nil] }.to_h
+      nodes_needing_connectors = nodes.to_h { |node| [node, nil] }
       @connector_plugins.each_value do |connector|
         nodes_without_connectors = nodes_needing_connectors.select { |_node, selected_connector| selected_connector.nil? }.keys
         break if nodes_without_connectors.empty?

data/lib/hybrid_platforms_conductor/common_config_dsl/file_system_tests.rb

@@ -84,7 +84,7 @@ module HybridPlatformsConductor
           select_confs_for_node(node, fs_paths_rules).
           inject({}) do |merged_paths, paths_info|
             if paths_info[:context][:file_system_type] == file_system_type
-              merged_paths.merge(paths_info[:paths].map do |path|
+              merged_paths.merge(paths_info[:paths].to_h do |path|
                 [
                   path,
                   {
@@ -92,7 +92,7 @@ module HybridPlatformsConductor
                     context: paths_info[:context]
                   }
                 ]
-              end.to_h) do |path, rule_info_1, rule_info_2|
+              end) do |path, rule_info_1, rule_info_2|
                 # Just check that configuration is not inconsistent
                 raise "Inconsistent rule for #{file_system_type} file system checks in configuration for #{node}: #{path} is marked as being both #{rule_info_1[:state]} and #{rule_info_2[:state]}" if rule_info_1[:state] != rule_info_2[:state]
 

data/lib/hybrid_platforms_conductor/config.rb

@@ -3,6 +3,7 @@ require 'hybrid_platforms_conductor/core_extensions/cleanroom/fix_kwargs'
 require 'git'
 require 'ice_cube'
 require 'hybrid_platforms_conductor/plugins'
+require 'active_support/core_ext/module'
 
 module HybridPlatformsConductor
 

data/lib/hybrid_platforms_conductor/deployer.rb

@@ -263,9 +263,9 @@ module HybridPlatformsConductor
     def deploy_on(*nodes_selectors)
       # Get the sorted list of services to be deployed, per node
       # Hash<String, Array<String> >
-      services_to_deploy = @nodes_handler.select_nodes(nodes_selectors.flatten).map do |node|
+      services_to_deploy = @nodes_handler.select_nodes(nodes_selectors.flatten).to_h do |node|
         [node, @nodes_handler.get_services_of(node)]
-      end.to_h
+      end
 
       # Get the secrets to be deployed
       secrets = {}
@@ -474,12 +474,18 @@ module HybridPlatformsConductor
         timeout: 10,
         progress_name: 'Read deployment logs'
       )
-      nodes.map do |node|
+      nodes.to_h do |node|
+        exit_code, stdout, stderr = read_actions_results[node] || [nil, nil, nil]
         [
           node,
-          @log_plugins[log_plugins_for(node).first].logs_for(node, *(read_actions_results[node] || [nil, nil, nil]))
+          @log_plugins[log_plugins_for(node).first].logs_for(
+            node,
+            exit_code,
+            stdout&.force_encoding(Encoding::UTF_8),
+            stderr&.force_encoding(Encoding::UTF_8)
+          )
         ]
-      end.to_h
+      end
     end
 
     # Parse stdout and stderr of a given deploy run and get the list of tasks with their status
@@ -547,7 +553,7 @@ module HybridPlatformsConductor
       # Deploy for real
       @nodes_handler.prefetch_metadata_of services.keys, :image
       outputs = @actions_executor.execute_actions(
-        services.map do |node, node_services|
+        services.to_h do |node, node_services|
           image_id = @nodes_handler.get_image_of(node)
           need_sudo = !@actions_executor.privileged_access?(node)
           sudo = @actions_executor.sudo_prefix(node)
@@ -577,12 +583,12 @@ module HybridPlatformsConductor
                     remote_bash: "#{sudo}yum install -y ca-certificates"
                   },
                   {
-                    scp: Dir.glob("#{ENV['hpc_certificates']}/*.crt").map do |cert_file|
+                    scp: Dir.glob("#{ENV['hpc_certificates']}/*.crt").to_h do |cert_file|
                       [
                         cert_file,
                         '/etc/pki/ca-trust/source/anchors'
                       ]
-                    end.to_h.merge(sudo: need_sudo),
+                    end.merge(sudo: need_sudo),
                     remote_bash: [
                       "#{sudo}update-ca-trust enable",
                       "#{sudo}update-ca-trust extract"
@@ -607,19 +613,19 @@ module HybridPlatformsConductor
               certificate_actions +
               @services_handler.actions_to_deploy_on(node, node_services, @use_why_run)
           ]
-        end.to_h,
+        end,
         timeout: @timeout,
         concurrent: @concurrent_execution,
         log_to_stdout: !@concurrent_execution
       )
       # Free eventual locks
       @actions_executor.execute_actions(
-        services.keys.map do |node|
+        services.keys.to_h do |node|
           [
             node,
             { remote_bash: "#{@actions_executor.sudo_prefix(node)}./mutex_dir unlock /tmp/hybrid_platforms_conductor_deploy_lock" }
           ]
-        end.to_h,
+        end,
         timeout: 10,
         concurrent: true,
         log_to_dir: nil
@@ -641,7 +647,7 @@ module HybridPlatformsConductor
       section "Saving deployment logs for #{logs.size} nodes" do
         ssh_user = @actions_executor.connector(:ssh).ssh_user
         @actions_executor.execute_actions(
-          logs.map do |node, (exit_status, stdout, stderr)|
+          logs.to_h do |node, (exit_status, stdout, stderr)|
             [
               node,
               log_plugins_for(node).
@@ -660,7 +666,7 @@ module HybridPlatformsConductor
                 end.
                 flatten(1)
             ]
-          end.to_h,
+          end,
           timeout: 10,
           concurrent: true,
           log_to_dir: nil,

data/lib/hybrid_platforms_conductor/executable.rb

@@ -99,7 +99,7 @@ module HybridPlatformsConductor
           @instantiated_components[component] = HybridPlatformsConductor.const_get(component.to_s.split('_').collect(&:capitalize).join.to_sym).new(
             logger: @logger,
             logger_stderr: @logger_stderr,
-            **dependencies.map { |dependency| [dependency, send(dependency)] }.to_h
+            **dependencies.to_h { |dependency| [dependency, send(dependency)] }
           )
         end
         @instantiated_components[component]

data/lib/hybrid_platforms_conductor/hpc_plugins/cmdb/host_ip.rb

@@ -73,19 +73,33 @@ module HybridPlatformsConductor
         def ip_for(*hosts)
           results = {}
           log_debug "Get IPs of #{hosts.size} hosts..."
+          exit_status, _stdout, _stderr = @cmd_runner.run_cmd('command -v getent', no_exception: true)
+          getent_present = exit_status.zero?
           for_each_element_in(
             hosts,
             parallel: true,
             nbr_threads_max: MAX_THREADS_GETENT,
             progress: log_debug? ? 'Gather IPs' : nil
           ) do |host|
-            _exit_status, stdout, _stderr = @cmd_runner.run_cmd(
-              "getent hosts #{host}",
-              timeout: TIMEOUT_GETENT,
-              log_to_stdout: log_debug?,
-              no_exception: true
-            )
-            ip = stdout.strip.split(/\s+/).first
+            ip =
+              if getent_present
+                _exit_status, stdout, _stderr = @cmd_runner.run_cmd(
+                  "getent hosts #{host}",
+                  timeout: TIMEOUT_GETENT,
+                  log_to_stdout: log_debug?,
+                  no_exception: true
+                )
+                stdout.strip.split(/\s+/).first
+              else
+                _exit_status, stdout, _stderr = @cmd_runner.run_cmd(
+                  "host #{host} | grep 'has address'",
+                  timeout: TIMEOUT_GETENT,
+                  log_to_stdout: log_debug?,
+                  no_exception: true
+                )
+                stdout.strip.split(/\s+/).last
+              end
+
             if ip.nil?
               log_warn "Host #{host} has no IP."
             else

data/lib/hybrid_platforms_conductor/hpc_plugins/cmdb/platform_handlers.rb

@@ -23,7 +23,7 @@ module HybridPlatformsConductor
         # * Hash<String, Object>: The corresponding property, per required node.
         #     Nodes for which the property can't be fetched can be ommitted.
         def get_services(nodes, _metadata)
-          nodes.map { |node| [node, platform_for(node).services_for(node)] }.to_h
+          nodes.to_h { |node| [node, platform_for(node).services_for(node)] }
         end
 
         # Get other properties for a given set of nodes.
@@ -42,7 +42,7 @@ module HybridPlatformsConductor
         # * Hash<String, Hash<Symbol,Object> >: The corresponding properties, per required node.
         #     Nodes for which the property can't be fetched can be ommitted.
         def get_others(nodes, _metadata)
-          nodes.map { |node| [node, platform_for(node).metadata_for(node)] }.to_h
+          nodes.to_h { |node| [node, platform_for(node).metadata_for(node)] }
         end
 
         private

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

@@ -605,7 +605,7 @@ module HybridPlatformsConductor
                 end
               else
                 # We have not created any ControlMaster, but still consider the nodes to be ready to connect
-                user_locks = nodes.map { |node| [node, nil] }.to_h
+                user_locks = nodes.to_h { |node| [node, nil] }
               end
               yield user_locks.keys
             ensure

data/lib/hybrid_platforms_conductor/hpc_plugins/platform_handler/serverless_chef.rb

@@ -132,8 +132,7 @@ module HybridPlatformsConductor
               else
                 (info[:status][:added_files] + info[:status][:changed_files] + info[:status][:untracked_files]).
                   sort.
-                  map { |f| [f, File.mtime("#{@repository_path}/#{f}").strftime('%F %T')] }.
-                  to_h
+                  to_h { |f| [f, File.mtime("#{@repository_path}/#{f}").strftime('%F %T')] }
               end,
             deleted_files: info[:status].nil? ? [] : info[:status][:deleted_files].sort
           }

data/lib/hybrid_platforms_conductor/hpc_plugins/provisioner/proxmox/proxmox_waiter.rb

@@ -371,7 +371,7 @@ class ProxmoxWaiter
   # Result::
   # * Hash<String, [Float or nil, Float or nil]>: The set of 2 scores, per PVE node name
   def pve_scores_for(_nbr_cpus, ram_mb, disk_gb)
-    @config['pve_nodes'].map do |pve_node|
+    @config['pve_nodes'].to_h do |pve_node|
       # Get some resource usages stats from the node directly
       status_info = api_get("nodes/#{pve_node}/status")
       load_average = status_info['loadavg'].map { |load_str| Float(load_str) }
@@ -418,11 +418,11 @@ class ProxmoxWaiter
           [
             if expected_ram_percent_used <= @config['limits']['ram_percent_used_max'] &&
                 expected_disk_percent_used <= @config['limits']['disk_percent_used_max']
-              expected_ram_percent_used * @config['coeff_ram_consumption'] + expected_disk_percent_used * @config['coeff_disk_consumption']
+              (expected_ram_percent_used * @config['coeff_ram_consumption']) + (expected_disk_percent_used * @config['coeff_disk_consumption'])
             end,
             if expected_ram_percent_used_without_expired <= @config['limits']['ram_percent_used_max'] &&
                 expected_disk_percent_used_without_expired <= @config['limits']['disk_percent_used_max']
-              expected_ram_percent_used_without_expired * @config['coeff_ram_consumption'] + expected_disk_percent_used_without_expired * @config['coeff_disk_consumption']
+              (expected_ram_percent_used_without_expired * @config['coeff_ram_consumption']) + (expected_disk_percent_used_without_expired * @config['coeff_disk_consumption'])
             end
           ]
         else
@@ -431,7 +431,7 @@ class ProxmoxWaiter
           [nil, nil]
         end
       ]
-    end.to_h
+    end
   end
 
   # Is a given VM expired?
@@ -499,10 +499,10 @@ class ProxmoxWaiter
     if hpc_marker_idx.nil?
       {}
     else
-      vm_description_lines[hpc_marker_idx + 1..].map do |line|
+      vm_description_lines[hpc_marker_idx + 1..].to_h do |line|
         property, value = line.split(': ')
         [property.to_sym, value]
-      end.to_h
+      end
     end
   end
 

data/lib/hybrid_platforms_conductor/hpc_plugins/provisioner/proxmox.rb

@@ -101,10 +101,10 @@ module HybridPlatformsConductor
 
                 # Get the HPC info associated to this VM
                 # Hash<Symbol,String>
-                vm_hpc_info = vm_description_lines[hpc_marker_idx + 1..].map do |line|
+                vm_hpc_info = vm_description_lines[hpc_marker_idx + 1..].to_h do |line|
                   property, value = line.split(': ')
                   [property.to_sym, value]
-                end.to_h
+                end
                 next unless vm_hpc_info[:node] == @node && vm_hpc_info[:environment] == @environment
 
                 # Found it

data/lib/hybrid_platforms_conductor/hpc_plugins/report/mediawiki.rb

@@ -87,7 +87,7 @@ module HybridPlatformsConductor
           @nodes_handler.prefetch_metadata_of nodes, locale.keys
           nodes.
             map do |node|
-              { node: node }.merge(all_properties.map { |property| [property, @nodes_handler.metadata_of(node, property)] }.to_h)
+              { node: node }.merge(all_properties.to_h { |property| [property, @nodes_handler.metadata_of(node, property)] })
             end.
             # Group them by physical / VMs
             group_by do |node_info|

data/lib/hybrid_platforms_conductor/hpc_plugins/secrets_reader/keepass.rb

@@ -125,7 +125,7 @@ module HybridPlatformsConductor
         # * Hash: The JSON secrets parsed from this group
         def parse_xml_secrets(group)
           # Parse all entries
-          group.xpath('Entry').map do |entry|
+          group.xpath('Entry').to_h do |entry|
             [
               entry.at_xpath('String/Key[contains(.,"Title")]/../Value').text,
               FIELDS.map do |property, field|
@@ -139,7 +139,7 @@ module HybridPlatformsConductor
                   ]
                 end
               end.compact.to_h.merge(
-                entry.xpath('Binary').map do |binary|
+                entry.xpath('Binary').to_h do |binary|
                   binary_meta = group.document.at_xpath("KeePassFile/Meta/Binaries/Binary[@ID=#{Integer(binary.xpath('Value').attr('Ref').value)}]")
                   binary_content = Base64.decode64(binary_meta.text)
                   if binary_meta.attr('Compressed') == 'True'
@@ -151,17 +151,17 @@ module HybridPlatformsConductor
                     binary.xpath('Key').text,
                     binary_content
                   ]
-                end.to_h
+                end
               )
             ]
-          end.to_h.merge(
+          end.merge(
             # Add children groups
-            group.xpath('Group').map do |sub_group|
+            group.xpath('Group').to_h do |sub_group|
               [
                 sub_group.at_xpath('Name').text,
                 parse_xml_secrets(sub_group)
               ]
-            end.to_h
+            end
           )
         end
 

data/lib/hybrid_platforms_conductor/hpc_plugins/test/file_system.rb

@@ -15,7 +15,7 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           # Flatten the paths rules so that we can spot inconsistencies in configuration
-          @config.aggregate_files_rules(@nodes_handler, @node).map do |path, rule_info|
+          @config.aggregate_files_rules(@nodes_handler, @node).to_h do |path, rule_info|
             [
               "if #{@actions_executor.sudo_prefix(@node)}/bin/bash -c '[[ -d \"#{path}\" ]]' ; then echo 1 ; else echo 0 ; fi",
               {
@@ -32,7 +32,7 @@ module HybridPlatformsConductor
                 timeout: 2
               }
             ]
-          end.to_h
+          end
         end
 
       end

data/lib/hybrid_platforms_conductor/hpc_plugins/test/file_system_hdfs.rb

@@ -14,7 +14,7 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           # Flatten the paths rules so that we can spot inconsistencies in configuration
-          @config.aggregate_files_rules(@nodes_handler, @node, file_system_type: :hdfs).map do |path, rule_info|
+          @config.aggregate_files_rules(@nodes_handler, @node, file_system_type: :hdfs).to_h do |path, rule_info|
             [
               "if sudo#{rule_info[:context][:sudo_user] ? " -u #{rule_info[:context][:sudo_user]}" : ''} hdfs dfs -ls \"#{path}\" ; then echo 1 ; else echo 0 ; fi",
               {
@@ -31,7 +31,7 @@ module HybridPlatformsConductor
                 timeout: 5
               }
             ]
-          end.to_h
+          end
         end
 
       end

data/lib/hybrid_platforms_conductor/hpc_plugins/test/ip.rb

@@ -18,7 +18,7 @@ module HybridPlatformsConductor
               else
                 private_ips = @nodes_handler.get_private_ips_of @node
                 if private_ips
-                  host_ips = stdout.first.split.select { |ip| ip =~ /^172\.16\.\d+\.\d+$/ }.sort
+                  host_ips = stdout.first.split.grep(/^172\.16\.\d+\.\d+$/).sort
                   ref_ips = private_ips.sort
                   assert_equal(
                     host_ips,

data/lib/hybrid_platforms_conductor/hpc_plugins/test/linear_strategy.rb

@@ -20,18 +20,18 @@ module HybridPlatformsConductor
             _exit_status, stdout, _stderr = @cmd_runner.run_cmd(<<~EO_BASH, log_to_stdout: log_debug?, no_exception: true, expected_code: [0, 1])
               cd #{@platform.repository_path} && \
               git --no-pager log \
+                --pretty=format:\"%H\" \
+                --graph \
                 $(git merge-base \
                   --octopus \
                   $(git --no-pager log #{merge_commit_id} --max-count 1 --pretty=format:\"%P\") \
-                )..#{merge_commit_id} \
-                --pretty=format:\"%H\" \
-                --graph \
+                )..#{merge_commit_id}
               | grep '|'
             EO_BASH
             next if stdout.empty?
 
             _exit_status, stdout, _stderr = @cmd_runner.run_cmd(
-              "cd #{@platform.repository_path} && git --no-pager log #{merge_commit_id} --pretty=format:%aI",
+              "cd #{@platform.repository_path} && git show --no-patch --format=%ci #{merge_commit_id}",
               log_to_stdout: log_debug?
             )
             error "Git history is not linear because of Merge commit #{merge_commit_id}" if Time.now - Time.parse(stdout.strip) < LOOKING_PERIOD

data/lib/hybrid_platforms_conductor/hpc_plugins/test/private_ips.rb

@@ -13,8 +13,7 @@ module HybridPlatformsConductor
           @nodes_handler.prefetch_metadata_of @nodes_handler.known_nodes, :private_ips
           private_ips = @nodes_handler.
             known_nodes.
-            map { |node| [node, @nodes_handler.get_private_ips_of(node) || []] }.
-            to_h
+            to_h { |node| [node, @nodes_handler.get_private_ips_of(node) || []] }
 
           # Check there are no duplicates
           nodes_per_private_ip = {}

data/lib/hybrid_platforms_conductor/hpc_plugins/test/public_ips.rb

@@ -13,8 +13,7 @@ module HybridPlatformsConductor
           @nodes_handler.prefetch_metadata_of @nodes_handler.known_nodes, :public_ips
           public_ips = @nodes_handler.
             known_nodes.
-            map { |node| [node, @nodes_handler.get_public_ips_of(node) || []] }.
-            to_h
+            to_h { |node| [node, @nodes_handler.get_public_ips_of(node) || []] }
 
           # Check there are no duplicates
           nodes_per_public_ip = {}

data/lib/hybrid_platforms_conductor/hpc_plugins/test/veids.rb

@@ -13,8 +13,7 @@ module HybridPlatformsConductor
           @nodes_handler.prefetch_metadata_of @nodes_handler.known_nodes, :veid
           veids = @nodes_handler.
             known_nodes.
-            map { |node| [node, @nodes_handler.get_veid_of(node) ? @nodes_handler.get_veid_of(node).to_i : nil] }.
-            to_h
+            to_h { |node| [node, @nodes_handler.get_veid_of(node) ? @nodes_handler.get_veid_of(node).to_i : nil] }
 
           # Check there are no duplicates
           veids.group_by { |_node, veid| veid }.each do |veid, nodes|

data/lib/hybrid_platforms_conductor/hpc_plugins/test/vulnerabilities.rb

@@ -57,7 +57,7 @@ module HybridPlatformsConductor
                 end
               )
               sudo = @actions_executor.sudo_prefix(@node)
-              urls.map do |url|
+              urls.to_h do |url|
                 # 1. Get the OVAL file on the node to be tested (uncompress it if needed)
                 # 2. Make sure oscap is installed
                 # 3. Generate the report for this OVAL file using oscap
@@ -143,7 +143,7 @@ module HybridPlatformsConductor
                     timeout: 240
                   }
                 ]
-              end.to_h
+              end
             else
               error "No OVAL file defined for image #{image} at #{oval_file}"
               {}

data/lib/hybrid_platforms_conductor/plugins.rb

@@ -75,10 +75,11 @@ module HybridPlatformsConductor
     # Register plugins by parsing gems
     def register_plugins_from_gems
       # Require all possible files that could define such a plugin, from all gems
-      files_regexp = %r{lib/(.*hpc_plugins/#{Regexp.escape(@plugins_type.to_s)}/[^/]+)\.rb$}
-      Gem.loaded_specs.each do |gem_name, gem_specs|
+      Gem.loaded_specs.clone.each do |gem_name, gem_specs|
+        gem_path = gem_specs.full_gem_path
+        files_regexp = %r{#{Regexp.escape(gem_path)}/lib/(.*hpc_plugins/#{Regexp.escape(@plugins_type.to_s)}/[^/]+)\.rb$}
         # Careful to not use gem_specs.files here as if your gem name contains "-" or other weird characters, files won't appear in the gemspec list.
-        Dir.glob("#{gem_specs.full_gem_path}/lib/**/*.rb").each do |file|
+        Dir.glob("#{gem_path}/lib/**/*.rb").each do |file|
           next unless file =~ files_regexp
 
           require_name = Regexp.last_match(1)

data/lib/hybrid_platforms_conductor/tests_runner.rb

@@ -524,14 +524,14 @@ module HybridPlatformsConductor
           nodes_to_test = selected_tests.map(&:node).uniq.sort
           @outputs =
             if @skip_run
-              nodes_to_test.map do |node|
+              nodes_to_test.to_h do |node|
                 run_log_file_name = "#{@config.hybrid_platforms_dir}/run_logs/#{node}.stdout"
                 [
                   node,
                   # TODO: Find a way to also save stderr and the status code
                   [0, File.exist?(run_log_file_name) ? File.read(run_log_file_name) : nil, '']
                 ]
-              end.to_h
+              end
             else
               # Why-run deploy on all nodes
               @deployer.concurrent_execution = !log_debug?