cem_acpt 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c774f50c327fab457a1047b30f5155cbe3afb8d55c630ddeb0525dde147304b
-  data.tar.gz: 134d1d6cdb6b9de6b7d77a8e634aad01809b2b9fbeb7ddd72f283b32cbcdc292
+  metadata.gz: 1e2aaf3de96bb10c9091d11e8d5e2e9680cd07bd601fa349be1c75db15e7428e
+  data.tar.gz: 3ac23bcddfd8ebd461d437de6b77454a19bf6e72ebd7e1986b76c5601f8e87c2
 SHA512:
-  metadata.gz: 65d80bccfc97735e653a01d067efac47c292c432ffd8ade7d4257d89b8a6553a7e5a3b8a3d762e9687c0f2695af0a7cbd9337baef6c6afd65ab2baf0d4c221f4
-  data.tar.gz: 3dd0bf49d2d71ca3a18150c1182d0a2b8255981be71653e86e8d1a19bfd4b190348bfce153d244bb6dc35cd0f0b6f2db9edf8d5f8b8f7b40f3231869994ecf25
+  metadata.gz: 8eeb0684612c1df4d7d1087bc411a351906c475b5ad9d32b61221023efc806a4137c74bb926e700cbe21ee36d1d0f5d1dd2915e7f006102ac9b5c872de82d97b
+  data.tar.gz: f7fce1e31b862cabfb19aba1e3ce8c0c32e9e99b80ca892dba64f7cfd711ebf6a64a253470803ce0dac9b65da3af2a7e02d2f8597c552e891e785cc4960fc27e

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    cem_acpt (0.6.1)
+    cem_acpt (0.6.2)
       async-http (>= 0.60, < 0.70)
       bcrypt_pbkdf (>= 1.0, < 2.0)
       deep_merge (>= 1.2, < 2.0)

data/lib/cem_acpt/config.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'digest'
 require 'json'
 require 'yaml'
 require_relative 'core_extensions'
@@ -203,6 +204,14 @@ module CemAcpt
       @terraform_dir ||= File.join(user_config_dir, 'terraform')
     end
 
+    def module_terraform_checksum_file
+      @module_terraform_checksum_file ||= File.join(user_config_dir, 'terraform_checksum.txt')
+    end
+
+    def module_terraform_checksum
+      @module_terraform_checksum ||= new_module_terraform_checksum
+    end
+
     def valid_env_var?(env_var)
       env_var.start_with?('CEM_ACPT_') && ENV[env_var]
     end
@@ -338,8 +347,33 @@ module CemAcpt
     end
 
     def create_config_dirs!
-      FileUtils.mkdir_p(user_config_dir) unless Dir.exist?(user_config_dir)
+      FileUtils.mkdir_p(user_config_dir)
+      create_terraform_dir!
+    end
+
+    def create_terraform_dir!
+      raise 'Cannot create terraform dir without a user config dir' unless Dir.exist? user_config_dir
+
+      if File.exist?(module_terraform_checksum_file)
+        checksum = File.read(module_terraform_checksum_file).strip
+        return if checksum == module_terraform_checksum
+      end
       FileUtils.cp_r(File.expand_path(File.join(__dir__, '..', 'terraform')), user_config_dir)
+      @module_terraform_checksum = new_module_terraform_checksum
+      File.write(module_terraform_checksum_file, module_terraform_checksum)
+    end
+
+    def new_module_terraform_checksum
+      sha256 = Digest::SHA256.new
+      files_and_dirs = Dir.glob(File.join(__dir__, '..', 'terraform', '**', '*'))
+      files_and_dirs.each do |file|
+        sha256 << if File.directory?(file)
+                    File.basename(file)
+                  else
+                    File.read(file)
+                  end
+      end
+      sha256.hexdigest
     end
   end
 end

data/lib/cem_acpt/platform/base.rb

@@ -15,12 +15,14 @@ module CemAcpt::Platform
 
     # @param conf [CemAcpt::Config] the config object.
     # @param single_test_data [Hash] the test data for the current test.
-    def initialize(config, single_test_data)
+    # @param run_data [Hash] the run data for the current run minus the test data key.
+    def initialize(config, single_test_data, **run_data)
       raise ArgumentError, 'single_test_data must be a Hash' unless single_test_data.is_a?(Hash)
 
       @config = config
       @test_data = single_test_data
       @node_name = @test_data[:node_name] || random_node_name
+      @run_data = run_data
     end
 
     def to_h

data/lib/cem_acpt/platform/gcp.rb

@@ -54,11 +54,11 @@ module Platform
   end
 
   def gcp_private_key
-    @gcp_private_key ||= (@test_data[:private_key] || File.join(Dir.home, '.ssh', 'google_compute_engine'))
+    @gcp_private_key ||= (@run_data[:private_key] || File.join(Dir.home, '.ssh', 'google_compute_engine'))
   end
 
   def gcp_public_key
-    @gcp_public_key ||= (@test_data[:public_key] || File.join(Dir.home, '.ssh', 'google_compute_engine.pub'))
+    @gcp_public_key ||= (@run_data[:public_key] || File.join(Dir.home, '.ssh', 'google_compute_engine.pub'))
   end
 
   def gcp_machine_type

data/lib/cem_acpt/platform.rb

@@ -17,16 +17,16 @@ module CemAcpt::Platform
     # item in the test data.
     # @param platform [String] the name of the platform
     # @param config [CemAcpt::Config] the config object
-    # @param test_data [Hash] the test data
-    def use(platform, config, test_data)
+    # @param run_data [Hash] the current run data. Must include a :test_data key
+    def use(platform, config, run_data)
       raise Error, "Platform #{platform} is not supported" unless platforms.include?(platform)
-      raise Error, 'test_data must be an Array' unless test_data.is_a?(Array)
+      raise Error, 'run_data must be an Hash' unless run_data.is_a?(Hash)
+      raise Error, 'run_data must include a :test_data key' unless run_data.key?(:test_data)
+      raise Error, 'run_data[:test_data] must be an Array' unless run_data[:test_data].is_a?(Array)
 
-      logger.info "Using #{platform} for #{test_data.length} tests..."
-      test_data.each_with_object([]) do |single_test_data, ary|
-        #local_port = local_port_allocator.allocate
-        #logger.debug("Allocated local port #{local_port} for test #{single_test_data[:test_name]}")
-        ary << new_platform_object(platform, config, single_test_data)
+      logger.info "Using #{platform} for #{run_data[:test_data].length} tests..."
+      run_data[:test_data].dup.each_with_object([]) do |single_test_data, ary|
+        ary << new_platform_object(platform, config, single_test_data, **run_data.except(:test_data))
       end
     end
 
@@ -62,10 +62,10 @@ module CemAcpt::Platform
     # @param config [CemAcpt::Config] the config object.
     # @param single_test_data [Hash] the test data for a single test.
     # @return [CemAcpt::Platform::Base] an initialized platform class.
-    def new_platform_object(platform, config, single_test_data)
+    def new_platform_object(platform, config, single_test_data, **run_data)
       raise Error, 'single_test_data must be a Hash' unless single_test_data.is_a?(Hash)
 
-      platform_class(platform).new(config, single_test_data)
+      platform_class(platform).new(config, single_test_data, **run_data)
     end
 
     # Creates a new platform-specific Class object for the given platform.

data/lib/cem_acpt/provision/terraform.rb

@@ -134,7 +134,7 @@ module CemAcpt
       end
 
       def new_working_dir
-        logger.debug('Terraform') { "Creating new working directory" }
+        logger.debug('Terraform') { 'Creating new working directory' }
         base_dir = File.join(@config.get('terraform.dir'), @config.get('platform.name'))
         logger.verbose('Terraform') { "Base directory defined as #{base_dir}" }
         @backend.base_provision_directory = base_dir
@@ -157,6 +157,7 @@ module CemAcpt
         logger.verbose('Terraform') { "Content of #{working_dir}:\n#{Dir.glob(File.join(working_dir, '*')).join("\n")}" }
         raise "Terraform working directory #{working_dir} does not exist" unless File.directory?(working_dir)
         raise "Terraform working directory #{working_dir} does not contain a Terraform file" unless Dir.glob(File.join(working_dir, '*.tf')).any?
+
         logger.info('Terraform') { "Using working directory: #{working_dir}" }
       rescue StandardError => e
         logger.error('Terraform') { 'Error validating working directory' }
@@ -165,13 +166,15 @@ module CemAcpt
 
       def provision_node_data
         node_data = @provision_data[:nodes].each_with_object({}) do |node, h|
-          h[node.node_name] = node.node_data.merge({
-            goss_file: node.test_data[:goss_file],
-            puppet_manifest: node.test_data[:puppet_manifest],
-            provision_dir_source: @backend.provision_directory,
-            provision_dir_dest: @backend.destination_provision_directory,
-            provision_commands: @backend.provision_commands,
-          })
+          h[node.node_name] = node.node_data.merge(
+            {
+              goss_file: node.test_data[:goss_file],
+              puppet_manifest: node.test_data[:puppet_manifest],
+              provision_dir_source: @backend.provision_directory,
+              provision_dir_dest: @backend.destination_provision_directory,
+              provision_commands: @backend.provision_commands,
+            }
+          )
         end
         node_data.to_json
       rescue StandardError => e
@@ -180,10 +183,14 @@ module CemAcpt
       end
 
       def formatted_vars
-        @provision_data[:nodes].first.platform_data.merge({
-          puppet_module_package: @provision_data[:module_package_path],
-          node_data: provision_node_data,
-        })
+        @provision_data[:nodes].first.platform_data.merge(
+          {
+            puppet_module_package: @provision_data[:module_package_path],
+            private_key: @provision_data[:private_key],
+            public_key: @provision_data[:public_key],
+            node_data: provision_node_data,
+          }
+        )
       rescue StandardError => e
         logger.error('Terraform') { 'Error creating formatted vars' }
         raise e

data/lib/cem_acpt/test_runner.rb

@@ -44,7 +44,7 @@ module CemAcpt
         logger.info('CemAcpt') { "Using module directory: #{config.get('module_dir')}..." }
         Dir.chdir(config.get('module_dir')) do
           keep_terminal_alive
-          @run_data[:priv_key], @run_data[:pub_key], @run_data[:known_hosts] = new_ephemeral_ssh_keys
+          @run_data[:private_key], @run_data[:public_key], @run_data[:known_hosts] = new_ephemeral_ssh_keys
           logger.info('CemAcpt') { 'Created ephemeral SSH key pair...' }
           @run_data[:module_package_path] = build_module_package
           logger.info('CemAcpt') { "Created module package: #{@run_data[:module_package_path]}..." }
@@ -67,9 +67,8 @@ module CemAcpt
       end
 
       def clean_up(trap_context = false)
-        unless trap_context
-          kill_keep_terminal_alive
-        end
+        kill_keep_terminal_alive unless trap_context
+
         clean_ephemeral_ssh_keys
         destroy_test_nodes
       end
@@ -114,7 +113,7 @@ module CemAcpt
       end
 
       def new_node_data
-        CemAcpt::Platform.use(config.get('platform.name'), config, @run_data[:test_data])
+        CemAcpt::Platform.use(config.get('platform.name'), config, @run_data)
       end
 
       def provision_test_nodes

data/lib/cem_acpt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CemAcpt
-  VERSION = '0.6.1'
+  VERSION = '0.6.2'
 end

data/lib/terraform/gcp/linux/main.tf

@@ -173,9 +173,9 @@ resource "google_compute_instance" "acpt-test-node" {
     }
 
     metadata = {
-        oslogin = "TRUE"
-        ssh-keys = "${var.username}:${file(var.public_key)}"
-        cem-acpt-test = each.value.test_name
+        "enable-oslogin" = "FALSE"
+        "ssh-keys" = "${var.username}:${file(var.public_key)}"
+        "cem-acpt-test" = each.value.test_name
     }
 
     tags = [ "cem-acpt-test-node" ]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cem_acpt
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - puppetlabs
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-04-20 00:00:00.000000000 Z
+date: 2023-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async-http