hybrid_platforms_conductor 32.17.0 → 33.0.2

data/lib/hybrid_platforms_conductor/hpc_plugins/log/my_log_plugin.rb.sample

@@ -0,0 +1,100 @@
+require 'hybrid_platforms_conductor/log'
+
+module HybridPlatformsConductor
+
+  module HpcPlugins
+
+    module Log
+
+      # Save logs on the remote node's file system
+      class RemoteFs < HybridPlatformsConductor::Log
+
+        # Get actions to save logs
+        # [API] - This method is mandatory.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): Node for which logs are being saved
+        # * *services* (Array<String>): The list of services that have been deployed on this node
+        # * *deployment_info* (Hash<Symbol,Object>): Additional information to attach to the logs
+        # * *exit_status* (Integer or Symbol): Exit status of the deployment
+        # * *stdout* (String): Deployment's stdout
+        # * *stderr* (String): Deployment's stderr
+        # Result::
+        # * Array< Hash<Symbol,Object> >: List of actions to be done
+        def actions_to_save_logs(node, services, deployment_info, exit_status, stdout, stderr)
+          # Return here all actions that are to be run to save those logs.
+          [
+            {
+              bash: <<~EOS
+                cat <<'EOLOGFILE' >>all_logs.txt
+                Deployment log on #{node}:
+                #{stdout}
+                EOLOGFILE
+              EOS
+            }
+          ]
+        end
+
+        # Get actions to read logs.
+        # If provided, this method can return some actions to be executed that will fetch logs from servers or remote nodes.
+        # By using this method to run actions instead of the synchronous method logs_from, such actions will be run in parallel which can greatly improve time-consuming operations when querying a lot of nodes.
+        # [API] - This method is optional.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): Node for which deployment logs are being read
+        # Result::
+        # * Array< Hash<Symbol,Object> >: List of actions to be done
+        def actions_to_read_logs(node)
+          [
+            { bash: 'cat all_logs.txt' }
+          ]
+        end
+
+        # Get deployment logs from a node.
+        # This method can use the result of actions previously run to read logs, as returned by the actions_to_read_logs method.
+        # [API] - This method is mandatory.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): The node we want deployment logs from
+        # * *exit_status* (Integer, Symbol or nil): Exit status of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # * *stdout* (String or nil): stdout of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # * *stderr* (String or nil): stderr of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # Result::
+        # * Hash<Symbol,Object>: Deployment log information:
+        #   * *error* (String): Error string in case deployment logs could not be retrieved. If set then further properties will be ignored. [optional]
+        #   * *services* (Array<String>): List of services deployed on the node
+        #   * *deployment_info* (Hash<Symbol,Object>): Deployment metadata
+        #   * *exit_status* (Integer or Symbol): Deployment exit status
+        #   * *stdout* (String): Deployment stdout
+        #   * *stderr* (String): Deployment stderr
+        def logs_for(node, exit_status, stdout, stderr)
+          # Here we should retrieve this info from somewhere.
+          # Based on this example, stdout is the value of the execution of the action 'cat all_logs.txt' as returned by actions_to_read_logs.
+          {
+            services: %w[unknown],
+            deployment_info: {},
+            exit_status: 0,
+            stdout: stdout,
+            stderr: ''
+          }
+        end
+
+      end
+
+    end
+
+  end
+
+end

data/lib/hybrid_platforms_conductor/hpc_plugins/log/remote_fs.rb

@@ -0,0 +1,180 @@
+require 'hybrid_platforms_conductor/log'
+
+module HybridPlatformsConductor
+
+  module HpcPlugins
+
+    module Log
+
+      # Save logs on the remote node's file system
+      class RemoteFs < HybridPlatformsConductor::Log
+
+        MARKER_STDOUT = '===== STDOUT ====='
+        MARKER_STDERR = '===== STDERR ====='
+
+        # Get actions to save logs
+        # [API] - This method is mandatory.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): Node for which logs are being saved
+        # * *services* (Array<String>): The list of services that have been deployed on this node
+        # * *deployment_info* (Hash<Symbol,Object>): Additional information to attach to the logs
+        # * *exit_status* (Integer or Symbol): Exit status of the deployment
+        # * *stdout* (String): Deployment's stdout
+        # * *stderr* (String): Deployment's stderr
+        # Result::
+        # * Array< Hash<Symbol,Object> >: List of actions to be done
+        def actions_to_save_logs(node, services, deployment_info, exit_status, stdout, stderr)
+          # Create a log file to be scp with all relevant info
+          ssh_user = @actions_executor.connector(:ssh).ssh_user
+          sudo_prefix = ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(node)} "
+          log_file = "#{Dir.tmpdir}/hpc_deploy_logs/#{node}_#{Time.now.utc.strftime('%F_%H%M%S')}_#{ssh_user}"
+          [
+            {
+              ruby: proc do
+                FileUtils.mkdir_p File.dirname(log_file)
+                File.write(log_file, <<~EO_DEPLOYMENT_LOG)
+                  #{
+                    deployment_info.merge(
+                      debug: log_debug? ? 'Yes' : 'No',
+                      services: services.join(', '),
+                      exit_status: exit_status
+                    ).map { |property, value| "#{property}: #{value}" }.join("\n")
+                  }
+                  #{MARKER_STDOUT}
+                  #{stdout}
+                  #{MARKER_STDERR}
+                  #{stderr}
+                EO_DEPLOYMENT_LOG
+              end,
+              remote_bash: "#{sudo_prefix}mkdir -p /var/log/deployments && #{sudo_prefix}chmod 600 /var/log/deployments"
+            },
+            {
+              scp: {
+                log_file => '/var/log/deployments',
+                :sudo => ssh_user != 'root',
+                :owner => 'root',
+                :group => 'root'
+              }
+            },
+            {
+              remote_bash: "#{sudo_prefix}chmod 600 /var/log/deployments/#{File.basename(log_file)}",
+              # Remove temporary files storing logs for security
+              ruby: proc do
+                File.unlink(log_file)
+              end
+            }
+          ]
+        end
+
+        # Get actions to read logs.
+        # If provided, this method can return some actions to be executed that will fetch logs from servers or remote nodes.
+        # By using this method to run actions instead of the synchronous method logs_from, such actions will be run in parallel which can greatly improve time-consuming operations when querying a lot of nodes.
+        # [API] - This method is optional.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): Node for which deployment logs are being read
+        # Result::
+        # * Array< Hash<Symbol,Object> >: List of actions to be done
+        def actions_to_read_logs(node)
+          sudo_prefix = @actions_executor.connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(node)} "
+          [
+            { remote_bash: "#{sudo_prefix}cat /var/log/deployments/`#{sudo_prefix}ls -t /var/log/deployments/ | head -1`" }
+          ]
+        end
+
+        # Get deployment logs from a node.
+        # This method can use the result of actions previously run to read logs, as returned by the actions_to_read_logs method.
+        # [API] - This method is mandatory.
+        # [API] - The following API components are accessible:
+        # * *@config* (Config): Main configuration API.
+        # * *@nodes_handler* (NodesHandler): Nodes handler API.
+        # * *@actions_executor* (ActionsExecutor): Actions executor API.
+        #
+        # Parameters::
+        # * *node* (String): The node we want deployment logs from
+        # * *exit_status* (Integer, Symbol or nil): Exit status of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # * *stdout* (String or nil): stdout of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # * *stderr* (String or nil): stderr of actions to read logs, or nil if no action was returned by actions_to_read_logs
+        # Result::
+        # * Hash<Symbol,Object>: Deployment log information:
+        #   * *error* (String): Error string in case deployment logs could not be retrieved. If set then further properties will be ignored. [optional]
+        #   * *services* (Array<String>): List of services deployed on the node
+        #   * *deployment_info* (Hash<Symbol,Object>): Deployment metadata
+        #   * *exit_status* (Integer or Symbol): Deployment exit status
+        #   * *stdout* (String): Deployment stdout
+        #   * *stderr* (String): Deployment stderr
+        def logs_for(_node, exit_status, stdout, stderr)
+          # Expected format for stdout:
+          # Property1: Value1
+          # ...
+          # PropertyN: ValueN
+          # ===== STDOUT =====
+          # ...
+          # ===== STDERR =====
+          # ...
+          if exit_status.is_a?(Symbol)
+            { error: "Error: #{exit_status}\n#{stderr}" }
+          else
+            stdout_lines = stdout.split("\n")
+            if stdout_lines.first =~ /No such file or directory/
+              { error: '/var/log/deployments missing' }
+            else
+              stdout_idx = stdout_lines.index(MARKER_STDOUT)
+              stderr_idx = stdout_lines.index(MARKER_STDERR)
+              deploy_info = {}
+              stdout_lines[0..stdout_idx - 1].each do |line|
+                if line =~ /^([^:]+): (.+)$/
+                  key_str = Regexp.last_match(1)
+                  value = Regexp.last_match(2)
+                  key = key_str.to_sym
+                  # Type-cast some values
+                  case key_str
+                  when 'date'
+                    # Date and time values
+                    # Thu Nov 23 18:43:01 UTC 2017
+                    deploy_info[key] = Time.parse("#{value} UTC")
+                  when 'debug'
+                    # Boolean values
+                    # Yes
+                    deploy_info[key] = (value == 'Yes')
+                  when /^diff_files_.+$/, 'services'
+                    # Array of strings
+                    # my_file.txt, other_file.txt
+                    deploy_info[key] = value.split(', ')
+                  else
+                    deploy_info[key] = value
+                  end
+                else
+                  deploy_info[:unknown_lines] = [] unless deploy_info.key?(:unknown_lines)
+                  deploy_info[:unknown_lines] << line
+                end
+              end
+              services = deploy_info.delete(:services)
+              exit_status = deploy_info.delete(:exit_status)
+              {
+                services: services,
+                deployment_info: deploy_info,
+                exit_status: exit_status =~ /^\d+$/ ? Integer(exit_status) : exit_status.to_sym,
+                stdout: stdout_lines[stdout_idx + 1..stderr_idx - 1].join("\n"),
+                stderr: stdout_lines[stderr_idx + 1..].join("\n")
+              }
+            end
+          end
+        end
+
+      end
+
+    end
+
+  end
+
+end

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

@@ -45,7 +45,7 @@ module HybridPlatformsConductor
           end
 
         end
-        self.extend_config_dsl_with MyDSLExtension, :init_serverless_chef
+        extend_config_dsl_with MyDSLExtension, :init_serverless_chef
 
         # Constructor
         #
@@ -59,8 +59,8 @@ module HybridPlatformsConductor
         def initialize(
           platform_type,
           repository_path,
-          logger: Logger.new(STDOUT),
-          logger_stderr: Logger.new(STDERR),
+          logger: Logger.new($stdout),
+          logger_stderr: Logger.new($stderr),
           config: Config.new,
           cmd_runner: CmdRunner.new
         )
@@ -146,11 +146,10 @@ module HybridPlatformsConductor
               if info[:status].nil?
                 {}
               else
-                Hash[
-                  (info[:status][:added_files] + info[:status][:changed_files] + info[:status][:untracked_files]).
-                    sort.
-                    map { |f| [f, File.mtime("#{@repository_path}/#{f}").strftime('%F %T')] }
-                ]
+                (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
               end,
             deleted_files: info[:status].nil? ? [] : info[:status][:deleted_files].sort
           }
@@ -159,40 +158,41 @@ module HybridPlatformsConductor
             package_dir = "dist/#{local_environment ? 'local' : 'prod'}/#{service}"
             package_info_file = "#{@repository_path}/#{package_dir}/hpc_package.info"
             current_package_info = File.exist?(package_info_file) ? JSON.parse(File.read(package_info_file)).transform_keys(&:to_sym) : {}
-            unless current_package_info == package_info
-              Bundler.with_unbundled_env do
-                policy_file = "policyfiles/#{service}.rb"
-                if local_environment
-                  local_policy_file = "policyfiles/#{service}.local.rb"
-                  # In local mode, we always regenerate the lock file as we may modify the run list
-                  run_list = known_cookbook_paths.any? { |cookbook_path| File.exist?("#{@repository_path}/#{cookbook_path}/hpc_test/recipes/before_run.rb") } ? ['hpc_test::before_run'] : []
-                  dsl_parser = DslParser.new
-                  dsl_parser.parse("#{@repository_path}/#{policy_file}")
-                  run_list.concat dsl_parser.calls.find { |call_info| call_info[:method] == :run_list }[:args].flatten
-                  run_list << 'hpc_test::after_run' if known_cookbook_paths.any? { |cookbook_path| File.exist?("#{@repository_path}/#{cookbook_path}/hpc_test/recipes/after_run.rb") }
-                  File.write("#{@repository_path}/#{local_policy_file}", File.read("#{@repository_path}/#{policy_file}") + "\nrun_list #{run_list.map { |recipe| "'#{recipe}'" }.join(', ')}\n")
-                  policy_file = local_policy_file
-                end
-                lock_file = "#{File.dirname(policy_file)}/#{File.basename(policy_file, '.rb')}.lock.json"
-                # If the policy lock file does not exist, generate it
-                @cmd_runner.run_cmd "cd #{@repository_path} && /opt/chef-workstation/bin/chef install #{policy_file} --chef-license accept" unless File.exist?("#{@repository_path}/#{lock_file}")
-                extra_cp_data_bags = File.exist?("#{@repository_path}/data_bags") ? " && cp -ar data_bags/ #{package_dir}/" : ''
-                @cmd_runner.run_cmd "cd #{@repository_path} && \
-                  #{@cmd_runner.root? ? '' : 'sudo '}rm -rf #{package_dir} && \
-                  /opt/chef-workstation/bin/chef export #{policy_file} #{package_dir} --chef-license accept#{extra_cp_data_bags}"
-              end
-              unless @cmd_runner.dry_run
-                # Create secrets file
-                secrets_file = "#{@repository_path}/#{package_dir}/data_bags/hpc_secrets/hpc_secrets.json"
-                FileUtils.mkdir_p(File.dirname(secrets_file))
-                File.write(secrets_file, secrets.merge(id: 'hpc_secrets').to_json)
-                # Remember the package info
-                File.write(package_info_file, package_info.to_json)
+            next if current_package_info == package_info
+
+            Bundler.with_unbundled_env do
+              policy_file = "policyfiles/#{service}.rb"
+              if local_environment
+                local_policy_file = "policyfiles/#{service}.local.rb"
+                # In local mode, we always regenerate the lock file as we may modify the run list
+                run_list = known_cookbook_paths.any? { |cookbook_path| File.exist?("#{@repository_path}/#{cookbook_path}/hpc_test/recipes/before_run.rb") } ? ['hpc_test::before_run'] : []
+                dsl_parser = DslParser.new
+                dsl_parser.parse("#{@repository_path}/#{policy_file}")
+                run_list.concat dsl_parser.calls.find { |call_info| call_info[:method] == :run_list }[:args].flatten
+                run_list << 'hpc_test::after_run' if known_cookbook_paths.any? { |cookbook_path| File.exist?("#{@repository_path}/#{cookbook_path}/hpc_test/recipes/after_run.rb") }
+                File.write("#{@repository_path}/#{local_policy_file}", File.read("#{@repository_path}/#{policy_file}") + "\nrun_list #{run_list.map { |recipe| "'#{recipe}'" }.join(', ')}\n")
+                policy_file = local_policy_file
               end
+              lock_file = "#{File.dirname(policy_file)}/#{File.basename(policy_file, '.rb')}.lock.json"
+              # If the policy lock file does not exist, generate it
+              @cmd_runner.run_cmd "cd #{@repository_path} && /opt/chef-workstation/bin/chef install #{policy_file} --chef-license accept" unless File.exist?("#{@repository_path}/#{lock_file}")
+              extra_cp_data_bags = File.exist?("#{@repository_path}/data_bags") ? " && cp -ar data_bags/ #{package_dir}/" : ''
+              @cmd_runner.run_cmd "cd #{@repository_path} && \
+                #{@cmd_runner.root? ? '' : 'sudo '}rm -rf #{package_dir} && \
+                /opt/chef-workstation/bin/chef export #{policy_file} #{package_dir} --chef-license accept#{extra_cp_data_bags}"
             end
+            next if @cmd_runner.dry_run
+
+            # Create secrets file
+            secrets_file = "#{@repository_path}/#{package_dir}/data_bags/hpc_secrets/hpc_secrets.json"
+            FileUtils.mkdir_p(File.dirname(secrets_file))
+            File.write(secrets_file, secrets.merge(id: 'hpc_secrets').to_json)
+            # Remember the package info
+            File.write(package_info_file, package_info.to_json)
           end
         end
 
+        # rubocop:disable Lint/UnusedMethodArgument
         # Prepare deployments.
         # This method is called just before getting and executing the actions to be deployed.
         # It is called once per platform.
@@ -208,6 +208,7 @@ module HybridPlatformsConductor
         def prepare_for_deploy(services:, secrets:, local_environment:, why_run:)
           @local_env = local_environment
         end
+        # rubocop:enable Lint/UnusedMethodArgument
 
         # Get the list of actions to perform to deploy on a given node.
         # Those actions can be executed in parallel with other deployments on other nodes. They must be thread safe.
@@ -242,6 +243,7 @@ module HybridPlatformsConductor
             package_name = File.basename(package_dir)
             chef_versions_file = "#{@repository_path}/chef_versions.yml"
             raise "Missing file #{chef_versions_file} specifying the Chef Infra Client version to be deployed" unless File.exist?(chef_versions_file)
+
             required_chef_client_version = YAML.load_file(chef_versions_file)['client']
             sudo = (@actions_executor.connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(node)} ")
             [
@@ -285,7 +287,7 @@ module HybridPlatformsConductor
         #     * *:changed*: The task has been changed
         #     * *:identical*: The task has not been changed
         #   * *diffs* (String): Differences, if any
-        def parse_deploy_output(stdout, stderr)
+        def parse_deploy_output(stdout, _stderr)
           tasks = []
           current_task = nil
           stdout.split("\n").each do |line|
@@ -293,8 +295,8 @@ module HybridPlatformsConductor
             case line.gsub(/\e\[[^\x40-\x7E]*[\x40-\x7E]/, '').strip
             when /^\* (\w+\[[^\]]+\]) action (.+)$/
               # New task
-              task_name = $1
-              task_action = $2
+              task_name = Regexp.last_match(1)
+              task_action = Regexp.last_match(2)
               current_task = {
                 name: task_name,
                 action: task_action,
@@ -303,7 +305,7 @@ module HybridPlatformsConductor
               tasks << current_task
             when /^- (.+)$/
               # Diff on the current task
-              diff_description = $1
+              diff_description = Regexp.last_match(1)
               unless current_task.nil?
                 current_task[:diffs] = '' unless current_task.key?(:diffs)
                 current_task[:diffs] << "#{diff_description}\n"
@@ -333,25 +335,23 @@ module HybridPlatformsConductor
           impacted_recipes = []
           impacted_global = false
           files_diffs.keys.sort.each do |impacted_file|
-            if impacted_file =~ /^policyfiles\/([^\/]+)\.rb$/
-              log_debug "[#{impacted_file}] - Impacted service: #{$1}"
-              impacted_services << $1
-            elsif impacted_file =~ /^policyfiles\/([^\/]+)\.lock.json$/
-              log_debug "[#{impacted_file}] - Impacted service: #{$1}"
-              impacted_services << $1
-            elsif impacted_file =~ /^nodes\/([^\/]+)\.json/
-              log_debug "[#{impacted_file}] - Impacted node: #{$1}"
-              impacted_nodes << $1
+            case impacted_file
+            when %r{^policyfiles/([^/]+)\.rb$}, %r{^policyfiles/([^/]+)\.lock.json$}
+              log_debug "[#{impacted_file}] - Impacted service: #{Regexp.last_match(1)}"
+              impacted_services << Regexp.last_match(1)
+            when %r{^nodes/([^/]+)\.json}
+              log_debug "[#{impacted_file}] - Impacted node: #{Regexp.last_match(1)}"
+              impacted_nodes << Regexp.last_match(1)
             else
-              cookbook_path = known_cookbook_paths.find { |cookbooks_path| impacted_file =~ /^#{Regexp.escape(cookbooks_path)}\/.+$/ }
+              cookbook_path = known_cookbook_paths.find { |cookbooks_path| impacted_file =~ %r{^#{Regexp.escape(cookbooks_path)}/.+$} }
               if cookbook_path.nil?
                 # Global file
                 log_debug "[#{impacted_file}] - Global file impacted"
                 impacted_global = true
               else
                 # File belonging to a cookbook
-                cookbook_name, file_path = impacted_file.match(/^#{cookbook_path}\/(\w+)\/(.+)$/)[1..2]
-                cookbook = cookbook_name.to_sym
+                file_cookbook_name, file_path = impacted_file.match(%r{^#{cookbook_path}/(\w+)/(.+)$})[1..2]
+                cookbook = file_cookbook_name.to_sym
                 # Small helper to register a recipe
                 register = proc do |source, recipe_name, cookbook_name: cookbook|
                   cookbook_name = cookbook_name.to_sym if cookbook_name.is_a?(String)
@@ -359,41 +359,41 @@ module HybridPlatformsConductor
                   impacted_recipes << [cookbook_name, recipe_name.to_sym]
                 end
                 case file_path
-                when /recipes\/(.+)\.rb/
-                  register.call('direct', $1)
-                when /attributes\/.+\.rb/, 'metadata.rb'
+                when %r{recipes/(.+)\.rb}
+                  register.call('direct', Regexp.last_match(1))
+                when %r{attributes/.+\.rb}, 'metadata.rb'
                   # Consider all recipes are impacted
                   Dir.glob("#{@repository_path}/#{cookbook_path}/#{cookbook}/recipes/*.rb") do |recipe_path|
                     register.call('attributes', File.basename(recipe_path, '.rb'))
                   end
-                when /(templates|files)\/(.+)/
+                when %r{(templates|files)/(.+)}
                   # Find recipes using this file name
-                  included_file = File.basename($2)
+                  included_file = File.basename(Regexp.last_match(2))
                   template_regexp = /["']#{Regexp.escape(included_file)}["']/
                   Dir.glob("#{@repository_path}/#{cookbook_path}/#{cookbook}/recipes/*.rb") do |recipe_path|
                     register.call("included file #{included_file}", File.basename(recipe_path, '.rb')) if File.read(recipe_path) =~ template_regexp
                   end
-                when /resources\/(.+)/
+                when %r{resources/(.+)}
                   # Find any recipe using this resource
-                  included_resource = "#{cookbook}_#{File.basename($1, '.rb')}"
+                  included_resource = "#{cookbook}_#{File.basename(Regexp.last_match(1), '.rb')}"
                   resource_regexp = /(\W|^)#{Regexp.escape(included_resource)}(\W|$)/
                   known_cookbook_paths.each do |cookbooks_path|
                     Dir.glob("#{@repository_path}/#{cookbooks_path}/**/recipes/*.rb") do |recipe_path|
                       if File.read(recipe_path) =~ resource_regexp
-                        cookbook_name, recipe_name = recipe_path.match(/#{cookbooks_path}\/(\w+)\/recipes\/(\w+)\.rb/)[1..2]
+                        cookbook_name, recipe_name = recipe_path.match(%r{#{cookbooks_path}/(\w+)/recipes/(\w+)\.rb})[1..2]
                         register.call("included resource #{included_resource}", recipe_name, cookbook_name: cookbook_name)
                       end
                     end
                   end
-                when /libraries\/(.+)/
+                when %r{libraries/(.+)}
                   # Find any recipe using methods from this library
-                  lib_methods_regexps = File.read("#{@repository_path}/#{impacted_file}").scan(/(\W|^)def\s+(\w+)(\W|$)/).map { |_grp1, method_name, _grp2| /(\W|^)#{Regexp.escape(method_name)}(\W|$)/ }
+                  lib_methods_regexps = File.read("#{@repository_path}/#{impacted_file}").scan(/(\W|^)def\s+(\w+)(\W|$)/).map { |_grp_1, method_name, _grp_2| /(\W|^)#{Regexp.escape(method_name)}(\W|$)/ }
                   known_cookbook_paths.each do |cookbooks_path|
                     Dir.glob("#{@repository_path}/#{cookbooks_path}/**/recipes/*.rb") do |recipe_path|
                       file_content = File.read(recipe_path)
                       found_lib_regexp = lib_methods_regexps.find { |regexp| file_content =~ regexp }
                       unless found_lib_regexp.nil?
-                        cookbook_name, recipe_name = recipe_path.match(/#{cookbooks_path}\/(\w+)\/recipes\/(\w+)\.rb/)[1..2]
+                        cookbook_name, recipe_name = recipe_path.match(%r{#{cookbooks_path}/(\w+)/recipes/(\w+)\.rb})[1..2]
                         register.call("included library helper #{found_lib_regexp.source[6..-7]}", recipe_name, cookbook_name: cookbook_name)
                       end
                     end
@@ -476,6 +476,7 @@ module HybridPlatformsConductor
           dsl_parser.parse(policy_file)
           run_list_call = dsl_parser.calls.find { |call_info| call_info[:method] == :run_list }
           raise "Policy #{policy} has no run list defined in #{policy_file}" if run_list_call.nil?
+
           run_list_call[:args].map { |recipe_def| decode_recipe(recipe_def) }
         end
 
@@ -493,13 +494,14 @@ module HybridPlatformsConductor
         # * Symbol: The cookbook name
         # * Symbol: The recipe name
         def decode_recipe(recipe_def)
-          recipe_def = $1 if recipe_def =~ /^recipe\[(.+)\]$/
+          recipe_def = Regexp.last_match(1) if recipe_def =~ /^recipe\[(.+)\]$/
           cookbook, recipe = recipe_def.split('::').map(&:to_sym)
           recipe = :default if recipe.nil?
           # Find the cookbook it belongs to
           cookbook_dir = known_cookbook_paths.find { |cookbook_path| File.exist?("#{@repository_path}/#{cookbook_path}/#{cookbook}") }
           raise "Unknown recipe #{cookbook}::#{recipe} from cookbook #{@repository_path}/#{cookbook_dir}/#{cookbook}." if !cookbook_dir.nil? && !File.exist?("#{@repository_path}/#{cookbook_dir}/#{cookbook}/recipes/#{recipe}.rb")
-          return cookbook_dir, cookbook, recipe
+
+          [cookbook_dir, cookbook, recipe]
         end
 
         private