hybrid_platforms_conductor 32.14.0 → 32.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99c918a62cd924ceaca95073e944df866b9b1ecce771dcb710122ec501cadf95
-  data.tar.gz: 959326a99c32ac8eb120af28c8b3c34304e2efa61df64f43a99545d0d33f88e3
+  metadata.gz: c01fcd515c7641751b265f34747561465570795cb75fb0287a75c61915055a2c
+  data.tar.gz: c249eb8d612bdc9b6a09467f195c71c607619e9aea278df4ff5b2bf061b968ed
 SHA512:
-  metadata.gz: 8e6c3a54a5c04f21aaeeba1a047366a7f427680077712742c0b2a7977cecfe14c4058fdd1e1c6c66842b603a7f7590bfea6eae0d975240e19a4ffa8214043c17
-  data.tar.gz: e8dd3ef50cec1a0dd7e94161159a30f359fca736bd6e7ab14ab916e2d743d48889bf891c3ca2bf626d3f087e1472ff43eb1abc0370e3fe63226cb0a8595950ad
+  metadata.gz: 571f2e0effa2d3489eb8c948f90bcf23cc8890ac71a0e6bc4b2ec805dea5d9bd74a608de3d5f6a8d327015058f6c2adf8a23f07aea4f82f1c6445b4b943cd3bd
+  data.tar.gz: 749df7363532aec2976b1b070462288ee8bed7596fe419d1871390022e337ad0472570fe98a513661f95e47366bd4df3a72d022ec0595cdc73ac051ff000dbb3

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# [v32.15.0](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v32.14.0...v32.15.0) (2021-05-31 14:43:32)
+
+## Global changes
+### Patches
+
+* [[Feature(platform_handler_serverless_chef)] Use user-defined cookbook hpc_test to tune chef-client runs in test environments](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/2cfa010997b562a599308ceaaf62a8740ff51468)
+
+## Changes for platform_handler_serverless_chef
+### Features
+
+* [[Feature(platform_handler_serverless_chef)] Use user-defined cookbook hpc_test to tune chef-client runs in test environments](https://github.com/sweet-delights/hybrid-platforms-conductor/commit/2cfa010997b562a599308ceaaf62a8740ff51468)
+
 # [v32.14.0](https://github.com/sweet-delights/hybrid-platforms-conductor/compare/v32.13.4...v32.14.0) (2021-05-31 09:05:45)
 
 ## Global changes

data/docs/plugins/platform_handler/serverless_chef.md

@@ -64,6 +64,12 @@ Services are being deployed by packaging the policy using [`chef install`](https
 
 Then packaged services are uploaded on the node to configured and deployment is done remotely using [Chef Infra Client in local mode](https://docs.chef.io/ctl_chef_client/#run-in-local-mode) on the remote node.
 
+## Test-wrapping cookbook `hpc_test`
+
+If you have a cookbook named `hpc_test` in your Chef repository, then this plugin will wrap any run-list run in a local environment mode (used in tests) with the recipes `hpc_test::before_run` and `hpc_test::after_run`.
+
+This way you can implement such recipes to adapt your Chef client runs to your test environment (like lack of connectivity, failing resources to be ignored...).
+
 ## Config DSL extension
 
 ### `helpers_including_recipes`

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

@@ -47,6 +47,28 @@ module HybridPlatformsConductor
         end
         self.extend_config_dsl_with MyDSLExtension, :init_serverless_chef
 
+        # Constructor
+        #
+        # Parameters::
+        # * *platform_type* (Symbol): Platform type
+        # * *repository_path* (String): Repository path
+        # * *logger* (Logger): Logger to be used [default: Logger.new(STDOUT)]
+        # * *logger_stderr* (Logger): Logger to be used for stderr [default: Logger.new(STDERR)]
+        # * *config* (Config): Config to be used. [default: Config.new]
+        # * *cmd_runner* (CmdRunner): Command executor to be used. [default: CmdRunner.new]
+        def initialize(
+          platform_type,
+          repository_path,
+          logger: Logger.new(STDOUT),
+          logger_stderr: Logger.new(STDERR),
+          config: Config.new,
+          cmd_runner: CmdRunner.new
+        )
+          super
+          # Mutex for getting the full recipes tree
+          @recipes_tree_mutex = Mutex.new
+        end
+
         # Setup the platform, install dependencies...
         # [API] - This method is optional.
         # [API] - @cmd_runner is accessible.
@@ -139,12 +161,25 @@ module HybridPlatformsConductor
             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 policyfiles/#{service}.rb" unless File.exist?("#{@repository_path}/policyfiles/#{service}.lock.json")
+                @cmd_runner.run_cmd "cd #{@repository_path} && /opt/chef-workstation/bin/chef install #{policy_file}" 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} && \
                   sudo rm -rf #{package_dir} && \
-                  /opt/chef-workstation/bin/chef export policyfiles/#{service}.rb #{package_dir}#{extra_cp_data_bags}"
+                  /opt/chef-workstation/bin/chef export #{policy_file} #{package_dir}#{extra_cp_data_bags}"
               end
               unless @cmd_runner.dry_run
                 # Create secrets file
@@ -193,9 +228,15 @@ module HybridPlatformsConductor
             FileUtils.mkdir_p "#{package_dir}/nodes"
             File.write("#{package_dir}/nodes/#{node}.json", (known_nodes.include?(node) ? metadata_for(node) : {}).merge(@nodes_handler.metadata_of(node)).to_json)
           end
+          client_options = [
+            '--local-mode',
+            '--chef-license', 'accept',
+            '--json-attributes', "nodes/#{node}.json"
+          ]
+          client_options << '--why-run' if use_why_run
           if @nodes_handler.get_use_local_chef_of(node)
             # Just run the chef-client directly from the packaged repository
-            [{ bash: "cd #{package_dir} && sudo SSL_CERT_DIR=/etc/ssl/certs /opt/chef-workstation/bin/chef-client --local-mode --json-attributes nodes/#{node}.json#{use_why_run ? ' --why-run' : ''}" }]
+            [{ bash: "cd #{package_dir} && sudo SSL_CERT_DIR=/etc/ssl/certs /opt/chef-workstation/bin/chef-client #{client_options.join(' ')}" }]
           else
             # Upload the package and run it from the node
             package_name = File.basename(package_dir)
@@ -211,7 +252,11 @@ module HybridPlatformsConductor
                   'set -o pipefail',
                   "if [ -n \"$(command -v apt)\" ]; then #{sudo}apt update && #{sudo}apt install -y curl build-essential ; else #{sudo}yum groupinstall 'Development Tools' && #{sudo}yum install -y curl ; fi",
                   'mkdir -p ./hpc_deploy',
-                  "curl --location https://omnitruck.chef.io/install.sh | tac | tac | #{sudo}bash -s -- -d /opt/artefacts -v #{required_chef_client_version} -s once"
+                  'rm -rf ./hpc_deploy/tmp',
+                  'mkdir -p ./hpc_deploy/tmp',
+                  'curl --location https://omnitruck.chef.io/install.sh --output ./hpc_deploy/install.sh',
+                  'chmod a+x ./hpc_deploy/install.sh',
+                  "#{sudo}TMPDIR=./hpc_deploy/tmp ./hpc_deploy/install.sh -d /opt/artefacts -v #{required_chef_client_version} -s once"
                 ]
               },
               {
@@ -219,10 +264,9 @@ module HybridPlatformsConductor
                 remote_bash: [
                   'set -e',
                   "cd ./hpc_deploy/#{package_name}",
-                  "#{sudo}SSL_CERT_DIR=/etc/ssl/certs /opt/chef/bin/chef-client --local-mode --chef-license=accept --json-attributes nodes/#{node}.json#{use_why_run ? ' --why-run' : ''}",
-                  'cd ..',
-                  "#{sudo}rm -rf #{package_name}"
-                ]
+                  "#{sudo}SSL_CERT_DIR=/etc/ssl/certs /opt/chef/bin/chef-client #{client_options.join(' ')}",
+                  'cd ..'
+                ] + (log_debug? ? [] : ["#{sudo}rm -rf ./hpc_deploy/#{package_name}"])
               }
             ]
           end
@@ -371,7 +415,7 @@ module HybridPlatformsConductor
           impacted_recipes.uniq!
           log_debug "* #{impacted_recipes.size} impacted recipes:\n#{impacted_recipes.map { |(cookbook, recipe)| "#{cookbook}::#{recipe}" }.sort.join("\n")}"
 
-          recipes_tree = RecipesTreeBuilder.new(@config, self).full_recipes_tree
+          recipes_tree = full_recipes_tree
           [
             impacted_nodes,
             (
@@ -419,6 +463,45 @@ module HybridPlatformsConductor
           @cookbook_paths
         end
 
+        # Get the run list of a given policy
+        #
+        # Parameters::
+        # * *policy* (String): Policy to get the run list from
+        # Result::
+        # * Array<[String or nil, Symbol, Symbol]>: Run list of the given policy, as [cookbook_dir, cookbook, recipe]
+        def policy_run_list(policy)
+          # Read the policy file
+          dsl_parser = DslParser.new
+          policy_file = "#{@repository_path}/policyfiles/#{policy}.rb"
+          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
+
+        # Return the cookbook directory, cookbook name and recipe name from which a recipe definition is found.
+        # The following forms are handled:
+        # * cookbook
+        # * cookbook::recipe
+        # * recipe[cookbook]
+        # * recipe[cookbook::recipe]
+        #
+        # Parameters::
+        # * *recipe_def* (String): Recipe definition (cookbook or cookbook::recipe).
+        # Result::
+        # * String: The cookbook directory, or nil if unknown
+        # * Symbol: The cookbook name
+        # * Symbol: The recipe name
+        def decode_recipe(recipe_def)
+          recipe_def = $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
+        end
+
         private
 
         # Return the JSON associated to a node
@@ -431,6 +514,18 @@ module HybridPlatformsConductor
           JSON.parse(File.read("#{@repository_path}/nodes/#{node}.json"))
         end
 
+        # Get the full recipes tree.
+        # Keep it in a cache for performance.
+        #
+        # Result::
+        # * Hash: The recipes tree. See RecipesTreeBuilder#full_recipes_tree for the detailed signature
+        def full_recipes_tree
+          @recipes_tree_mutex.synchronize do
+            @recipes_tree = RecipesTreeBuilder.new(@config, self).full_recipes_tree unless defined?(@recipes_tree)
+          end
+          @recipes_tree
+        end
+
       end
 
     end

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

@@ -36,12 +36,12 @@ module HybridPlatformsConductor
           def full_recipes_tree
             @recipes_tree = {}
             @platform.deployable_services.each do |service|
-              policy_run_list(service).each do |(cookbook_dir, cookbook, recipe)|
+              @platform.policy_run_list(service).each do |(cookbook_dir, cookbook, recipe)|
                 add_recipe_in_tree(cookbook_dir, cookbook, recipe)
               end
             end
             @platform.deployable_services.each do |service|
-              policy_run_list(service).each do |(cookbook_dir, cookbook, recipe)|
+              @platform.policy_run_list(service).each do |(cookbook_dir, cookbook, recipe)|
                 mark_recipe_used_by_policy(cookbook, recipe, service)
               end
             end
@@ -50,45 +50,6 @@ module HybridPlatformsConductor
 
           private
 
-          # Get the run list of a given policy
-          #
-          # Parameters::
-          # * *policy* (String): Policy to get the run list from
-          # Result::
-          # * Array<[String or nil, Symbol,Symbol]>: Run list of the given policy, as [cookbook_dir, cookbook, recipe]
-          def policy_run_list(policy)
-            # Read the policy file
-            dsl_parser = DslParser.new
-            policy_file = "#{@platform.repository_path}/policyfiles/#{policy}.rb"
-            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
-
-          # Return the cookbook directory, cookbook name and recipe name from which a recipe definition is found.
-          # The following forms are handled:
-          # * cookbook
-          # * cookbook::recipe
-          # * recipe[cookbook]
-          # * recipe[cookbook::recipe]
-          #
-          # Parameters::
-          # * *recipe_def* (String): Recipe definition (cookbook or cookbook::recipe).
-          # Result::
-          # * String: The cookbook directory, or nil if unknown
-          # * Symbol: The cookbook name
-          # * Symbol: The recipe name
-          def decode_recipe(recipe_def)
-            recipe_def = $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 = @platform.known_cookbook_paths.find { |cookbook_path| File.exist?("#{@platform.repository_path}/#{cookbook_path}/#{cookbook}") }
-            raise "Unknown recipe #{cookbook}::#{recipe} from cookbook #{@platform.repository_path}/#{cookbook_dir}/#{cookbook}." if !cookbook_dir.nil? && !File.exist?("#{@platform.repository_path}/#{cookbook_dir}/#{cookbook}/recipes/#{recipe}.rb")
-            return cookbook_dir, cookbook, recipe
-          end
-
           # Fill the tree with a recipe and all its dependencies
           #
           # Parameters::
@@ -142,11 +103,11 @@ module HybridPlatformsConductor
             # Check for include_recipe
             used_recipes = recipe_content.
               scan(/include_recipe\s+["'](\w+(::\w+)?)["']/).
-              map { |(recipe_def, _sub_grp)| decode_recipe(recipe_def) }
+              map { |(recipe_def, _sub_grp)| @platform.decode_recipe(recipe_def) }
             # Check for some helpers we know include some recipes
             @config.known_helpers_including_recipes.each do |helper_name, used_recipes_by_helper|
               if recipe_content =~ Regexp.new(/(\W|^)#{Regexp.escape(helper_name)}(\W|$)/)
-                used_recipes.concat(used_recipes_by_helper.map { |recipe_def| decode_recipe(recipe_def) })
+                used_recipes.concat(used_recipes_by_helper.map { |recipe_def| @platform.decode_recipe(recipe_def) })
                 used_recipes.uniq!
               end
             end

data/lib/hybrid_platforms_conductor/version.rb

@@ -1,5 +1,5 @@
 module HybridPlatformsConductor
 
-  VERSION = '32.14.0'
+  VERSION = '32.15.0'
 
 end

data/spec/hybrid_platforms_conductor_test/api/platform_handlers/serverless_chef/packaging_spec.rb

@@ -7,24 +7,34 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
     # Parameters::
     # * *repository* (String): Repository to be packaged
     # * *policy* (String): Expected policy to be packaged [default: 'test_policy']
+    # * *policy_file* (String): Expected policy file used [default: "policyfiles/#{policy}.rb"]
     # * *install* (Boolean): Are we expecting the chef install stage? [default: true]
     # * *export* (Boolean): Are we expecting the chef export stage? [default: true]
     # * *data_bags* (Boolean): Do we expect data bags copy? [default: false]
+    # * *env* (String): Expected environment being packaged [default: 'prod']
     # * Proc: Code called with mock in place
     def with_packaging_mocked(
       repository,
       policy: 'test_policy',
+      policy_file: "policyfiles/#{policy}.rb",
       install: true,
       export: true,
-      data_bags: false
+      data_bags: false,
+      env: 'prod'
     )
       with_cmd_runner_mocked(
         if install
           [
             [
-              "cd #{repository} && /opt/chef-workstation/bin/chef install policyfiles/#{policy}.rb",
+              "cd #{repository} && /opt/chef-workstation/bin/chef install #{policy_file}",
               proc do
-                File.write("#{repository}/policyfiles/#{policy}.lock.json", '{}')
+                # Mock the run_list stored in the lock file
+                File.write(
+                  "#{repository}/#{policy_file.gsub(/.rb$/, '.lock.json')}",
+                  {
+                    run_list: eval("[#{File.read("#{repository}/#{policy_file}").split("\n").select { |line| line =~ /^run_list.+$/ }.last.match(/^run_list(.+)$/)[1]}]").flatten
+                  }.to_json
+                )
                 [0, 'Chef install done', '']
               end
             ]
@@ -35,10 +45,10 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
         if export
           [
             [
-              /^cd #{Regexp.escape(repository)} &&\s+sudo rm -rf dist\/prod\/#{Regexp.escape(policy)} &&\s+\/opt\/chef-workstation\/bin\/chef export policyfiles\/#{Regexp.escape(policy)}.rb dist\/prod\/#{Regexp.escape(policy)}#{data_bags ? " && cp -ar data_bags/ dist/prod/#{Regexp.escape(policy)}/" : ''}$/,
+              /^cd #{Regexp.escape(repository)} &&\s+sudo rm -rf dist\/#{Regexp.escape(env)}\/#{Regexp.escape(policy)} &&\s+\/opt\/chef-workstation\/bin\/chef export #{Regexp.escape(policy_file)} dist\/#{Regexp.escape(env)}\/#{Regexp.escape(policy)}#{data_bags ? " && cp -ar data_bags/ dist/#{Regexp.escape(env)}/#{Regexp.escape(policy)}/" : ''}$/,
               proc do
-                FileUtils.mkdir_p "#{repository}/dist/prod/#{policy}"
-                FileUtils.cp_r("#{repository}/data_bags", "#{repository}/dist/prod/#{policy}/") if data_bags
+                FileUtils.mkdir_p "#{repository}/dist/#{env}/#{policy}"
+                FileUtils.cp_r("#{repository}/data_bags", "#{repository}/dist/#{env}/#{policy}/") if data_bags
                 [0, 'Chef export done', '']
               end
             ]
@@ -96,6 +106,26 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
         end
       end
 
+      it 'packages the repository for a given node and service in local mode' do
+        with_serverless_chef_platforms('1_node') do |platform, repository|
+          with_packaging_mocked(repository, policy_file: 'policyfiles/test_policy.local.rb', env: 'local') do
+            platform.package(services: { 'node' => %w[test_policy] }, secrets: {}, local_environment: true)
+            local_policy_file = "#{repository}/policyfiles/test_policy.local.lock.json"
+            expect(File.exist?(local_policy_file)).to eq true
+            expect(JSON.parse(File.read(local_policy_file))).to eq('run_list' => ['recipe[test_cookbook]'])
+          end
+        end
+      end
+
+      it 'packages the repository without resolving dependencies when the lock file already exists' do
+        with_serverless_chef_platforms('1_node') do |platform, repository|
+          File.write("#{repository}/policyfiles/test_policy.lock.json", '{}')
+          with_packaging_mocked(repository, install: false) do
+            platform.package(services: { 'node' => %w[test_policy] }, secrets: {}, local_environment: false)
+          end
+        end
+      end
+
       it 'does not package the repository twice for the same config' do
         with_serverless_chef_platforms('1_node', as_git: true) do |platform, repository|
           with_packaging_mocked(repository) do
@@ -208,6 +238,55 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
 
     end
 
+    context 'with a platform having hpc_test cookbook' do
+
+      it 'packages the repository with before_run and after_run recipes wrapping the run list' do
+        with_serverless_chef_platforms('hpc_test') do |platform, repository|
+          with_packaging_mocked(repository, policy_file: 'policyfiles/test_policy.local.rb', env: 'local') do
+            platform.package(services: { 'node' => %w[test_policy] }, secrets: {}, local_environment: true)
+            local_policy_file = "#{repository}/policyfiles/test_policy.local.lock.json"
+            expect(File.exist?(local_policy_file)).to eq true
+            expect(JSON.parse(File.read(local_policy_file))).to eq('run_list' => [
+              'hpc_test::before_run',
+              'recipe[test_cookbook]',
+              'hpc_test::after_run'
+            ])
+          end
+        end
+      end
+
+      it 'packages the repository with before_run only recipe' do
+        with_serverless_chef_platforms('hpc_test') do |platform, repository|
+          File.unlink "#{repository}/cookbooks/hpc_test/recipes/after_run.rb"
+          with_packaging_mocked(repository, policy_file: 'policyfiles/test_policy.local.rb', env: 'local') do
+            platform.package(services: { 'node' => %w[test_policy] }, secrets: {}, local_environment: true)
+            local_policy_file = "#{repository}/policyfiles/test_policy.local.lock.json"
+            expect(File.exist?(local_policy_file)).to eq true
+            expect(JSON.parse(File.read(local_policy_file))).to eq('run_list' => [
+              'hpc_test::before_run',
+              'recipe[test_cookbook]'
+            ])
+          end
+        end
+      end
+
+      it 'packages the repository with after_run only recipe' do
+        with_serverless_chef_platforms('hpc_test') do |platform, repository|
+          File.unlink "#{repository}/cookbooks/hpc_test/recipes/before_run.rb"
+          with_packaging_mocked(repository, policy_file: 'policyfiles/test_policy.local.rb', env: 'local') do
+            platform.package(services: { 'node' => %w[test_policy] }, secrets: {}, local_environment: true)
+            local_policy_file = "#{repository}/policyfiles/test_policy.local.lock.json"
+            expect(File.exist?(local_policy_file)).to eq true
+            expect(JSON.parse(File.read(local_policy_file))).to eq('run_list' => [
+              'recipe[test_cookbook]',
+              'hpc_test::after_run'
+            ])
+          end
+        end
+      end
+
+    end
+
   end
 
 end

data/spec/hybrid_platforms_conductor_test/api/platform_handlers/serverless_chef/services_deployment_spec.rb

@@ -37,7 +37,11 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
             'set -o pipefail',
             "if [ -n \"$(command -v apt)\" ]; then #{sudo}apt update && #{sudo}apt install -y curl build-essential ; else #{sudo}yum groupinstall 'Development Tools' && #{sudo}yum install -y curl ; fi",
             'mkdir -p ./hpc_deploy',
-            "curl --location https://omnitruck.chef.io/install.sh | tac | tac | #{sudo}bash -s -- -d /opt/artefacts -v 17.0 -s once"
+            'rm -rf ./hpc_deploy/tmp',
+            'mkdir -p ./hpc_deploy/tmp',
+            'curl --location https://omnitruck.chef.io/install.sh --output ./hpc_deploy/install.sh',
+            'chmod a+x ./hpc_deploy/install.sh',
+            "#{sudo}TMPDIR=./hpc_deploy/tmp ./hpc_deploy/install.sh -d /opt/artefacts -v 17.0 -s once"
           ]
         },
         {
@@ -45,9 +49,9 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
           remote_bash: [
             'set -e',
             "cd ./hpc_deploy/#{policy}",
-            "#{sudo}SSL_CERT_DIR=/etc/ssl/certs /opt/chef/bin/chef-client --local-mode --chef-license=accept --json-attributes nodes/#{node}.json#{check_mode ? ' --why-run' : ''}",
+            "#{sudo}SSL_CERT_DIR=/etc/ssl/certs /opt/chef/bin/chef-client --local-mode --chef-license accept --json-attributes nodes/#{node}.json#{check_mode ? ' --why-run' : ''}",
             'cd ..',
-            "#{sudo}rm -rf #{policy}"
+            "#{sudo}rm -rf ./hpc_deploy/#{policy}"
           ]
         }
       ]
@@ -237,7 +241,7 @@ describe HybridPlatformsConductor::HpcPlugins::PlatformHandler::ServerlessChef d
           )
           expect(platform.actions_to_deploy_on('local', 'test_policy_1', use_why_run: false)).to eq [
             {
-              bash: "cd #{repository}/dist/prod/test_policy_1 && sudo SSL_CERT_DIR=/etc/ssl/certs /opt/chef-workstation/bin/chef-client --local-mode --json-attributes nodes/local.json"
+              bash: "cd #{repository}/dist/prod/test_policy_1 && sudo SSL_CERT_DIR=/etc/ssl/certs /opt/chef-workstation/bin/chef-client --local-mode --chef-license accept --json-attributes nodes/local.json"
             }
           ]
         end

data/spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/chef_versions.yml

@@ -0,0 +1,3 @@
+---
+workstation: '21.5'
+client: '17.0'

data/spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/cookbooks/hpc_test/recipes/after_run.rb

@@ -0,0 +1 @@
+# Recipe

data/spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/cookbooks/hpc_test/recipes/before_run.rb

@@ -0,0 +1 @@
+# Recipe

data/spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/nodes/node.json

@@ -0,0 +1,10 @@
+{
+  "name": "node",
+  "normal": {
+    "description": "Single test node",
+    "image": "debian_9",
+    "private_ips": ["172.16.0.1"]
+  },
+  "policy_name": "test_policy",
+  "policy_group": "test_group"
+}

data/spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/policyfiles/test_policy.rb

@@ -0,0 +1,3 @@
+name File.basename(__FILE__, '.rb')
+default_source :supermarket
+run_list 'recipe[test_cookbook]'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hybrid_platforms_conductor
 version: !ruby/object:Gem::Version
-  version: 32.14.0
+  version: 32.15.0
 platform: ruby
 authors:
 - Muriel Salvan
@@ -873,6 +873,11 @@ files:
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/data_bags/data_bags/my_bag/my_item.json
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/data_bags/nodes/node.json
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/data_bags/policyfiles/test_policy.rb
+- spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/chef_versions.yml
+- spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/cookbooks/hpc_test/recipes/after_run.rb
+- spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/cookbooks/hpc_test/recipes/before_run.rb
+- spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/nodes/node.json
+- spec/hybrid_platforms_conductor_test/serverless_chef_repositories/hpc_test/policyfiles/test_policy.rb
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/recipes/cookbooks/test_cookbook_1/recipes/default.rb
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/recipes/cookbooks/test_cookbook_2/libraries/default.rb
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/recipes/cookbooks/test_cookbook_2/recipes/default.rb