kitchen-vagrant 1.5.0 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 839cf2c46b31628eba46d5e379eb40d42215f68667320df8af7571cd4b544aab
-  data.tar.gz: b1d80326940aeb6e119f8b0f4249db289032c05be5958dce7f26d9c1fa6a59ae
+  metadata.gz: 963b45a50eb74ce9d7bb16de1f63231ab44096c1459652be556bd7bc4c2d38d1
+  data.tar.gz: 5c0297d2879a29dd777b3976d6cda4167463fb9233dca540d848c5d741282145
 SHA512:
-  metadata.gz: ed91d2530854f088194421cb6d79d3062fd93f2b494563150a9ab467b8c1f7b9ff1a20cd00ef1563d1cb10e66dea5ddee8bd08461a3b9d68cf4f310cb7d63a9e
-  data.tar.gz: 562d85090622a80e6ee471ec39b92ae42575fb632bdbed24b25a7262e821f5d9e5253df1eae571d8a618602f08ff99d39d3a3216b14cfd507a51fdf582a9e361
+  metadata.gz: a84c39b2411b0f6f80ad9c5c6dbeeab0eabae479c57c4cec4e7ce9f40f86466a9d7051176da592783dbf94b236bb049831ad61cabd7fc68201dd75baf5b75314
+  data.tar.gz: 4658e11b0222d2fcee0c69739f5e7eae88a7a64f54f43009c142572f7e7e35d922aa4630ab3e26b007d37fdc7bf80fb77b9f0aa77e8cbda0c8cb64f7ffb75d65

data/lib/kitchen/driver/helpers.rb

@@ -76,6 +76,7 @@ module Kitchen
         sh.run_command
         debug("Local Command END #{Util.duration(sh.execution_time)}")
         raise "Failed: #{sh.stderr}" if sh.error?
+
         stdout = sanitize_stdout(sh.stdout)
         JSON.parse(stdout) if stdout.length > 2
       end
@@ -91,12 +92,13 @@ module Kitchen
             default_switch_object["Name"].empty?
           raise "Failed to find a default VM Switch."
         end
+
         default_switch_object["Name"]
       end
 
       def hyperv_default_switch_ps
         <<-VMSWITCH
-          Get-DefaultVMSwitch #{ENV['KITCHEN_HYPERV_SWITCH']} | ConvertTo-Json
+          Get-DefaultVMSwitch #{ENV["KITCHEN_HYPERV_SWITCH"]} | ConvertTo-Json
         VMSWITCH
       end
 
@@ -104,6 +106,7 @@ module Kitchen
 
       def ruby_array_to_ps_array(list)
         return "@()" if list.nil? || list.empty?
+
         list.to_s.tr("[]", "()").prepend("@")
       end
     end

data/lib/kitchen/driver/vagrant.rb

@@ -21,8 +21,8 @@ require "fileutils"
 require "rubygems/version"
 
 require "kitchen"
-require "kitchen/driver/vagrant_version"
-require "kitchen/driver/helpers"
+require_relative "vagrant_version"
+require_relative "helpers"
 
 module Kitchen
 
@@ -40,16 +40,20 @@ module Kitchen
 
       plugin_version Kitchen::Driver::VAGRANT_VERSION
 
-      default_config(:box) { |driver| driver.default_box }
+      default_config(:box, &:default_box)
       required_config :box
 
       default_config :box_check_update, nil
 
+      default_config :box_auto_update, nil
+
+      default_config :box_auto_prune, nil
+
       default_config :box_download_insecure, nil
 
       default_config :box_download_ca_cert, nil
 
-      default_config(:box_url) { |driver| driver.default_box_url }
+      default_config(:box_url, &:default_box_url)
 
       default_config :box_version, nil
 
@@ -109,6 +113,8 @@ module Kitchen
       def create(state)
         create_vagrantfile
         run_pre_create_command
+        run_box_auto_update
+        run_box_auto_prune
         run_vagrant_up
         update_state(state)
         instance.transport.connection(state).wait_until_ready
@@ -149,7 +155,8 @@ module Kitchen
         if state[:hostname].nil?
           raise UserError, "Vagrant instance not created!"
         end
-        if not (config[:ssh] && config[:ssh][:insert_key] == false)
+
+        unless config[:ssh] && config[:ssh][:insert_key] == false
           m = "Disable vagrant ssh key replacement to preserve the default key!"
           warn(m)
         end
@@ -171,6 +178,8 @@ module Kitchen
       def finalize_config!(instance)
         super
         finalize_vm_hostname!
+        finalize_box_auto_update!
+        finalize_box_auto_prune!
         finalize_pre_create_command!
         finalize_synced_folders!
         finalize_ca_cert!
@@ -247,7 +256,8 @@ module Kitchen
       # @api private
       def safe_share?(box)
         return false if config[:provider] =~ /(hyperv|libvirt)/
-        box =~ /^bento\/(centos|debian|fedora|opensuse|ubuntu|oracle|amazonlinux)-/
+
+        box =~ %r{^bento/(centos|debian|fedora|opensuse|ubuntu|oracle|amazonlinux)-}
       end
 
       # Return true if we found the criteria to enable the cache_directory
@@ -255,6 +265,7 @@ module Kitchen
       def enable_cache?
         return false unless config[:cache_directory]
         return true if safe_share?(config[:box])
+
         # Otherwise
         false
       end
@@ -291,10 +302,25 @@ module Kitchen
       def finalize_ca_cert!
         unless config[:box_download_ca_cert].nil?
           config[:box_download_ca_cert] = File.expand_path(
-            config[:box_download_ca_cert], config[:kitchen_root])
+            config[:box_download_ca_cert], config[:kitchen_root]
+          )
         end
       end
 
+      # Create vagrant command to update box to the latest version
+      def finalize_box_auto_update!
+        return if config[:box_auto_update].nil?
+
+        config[:box_auto_update] = "vagrant box update #{'--insecure ' if config[:box_download_insecure]}--box #{config[:box]}"
+      end
+
+      # Create vagrant command to remove older versions of the box
+      def finalize_box_auto_prune!
+        return if config[:box_auto_prune].nil?
+
+        config[:box_auto_prune] = "vagrant box prune --name #{config[:box]}"
+      end
+
       # Replaces any `{{vagrant_root}}` tokens in the pre create command.
       #
       # @api private
@@ -344,8 +370,8 @@ module Kitchen
       def finalize_vm_hostname!
         string = config[:vm_hostname]
 
-        if windows_os? && string.is_a?(String) && string.size >= 12
-          config[:vm_hostname] = "#{string[0...10]}-#{string[-1]}"
+        if windows_os? && string.is_a?(String) && string.size > 15
+          config[:vm_hostname] = "#{string[0...12]}-#{string[-1]}"
         end
       end
 
@@ -382,10 +408,11 @@ module Kitchen
       # @api private
       def render_template
         template = File.expand_path(
-          config[:vagrantfile_erb], config[:kitchen_root])
+          config[:vagrantfile_erb], config[:kitchen_root]
+        )
 
         if File.exist?(template)
-          ERB.new(IO.read(template)).result(binding).gsub(%r{^\s*$\n}, "")
+          ERB.new(IO.read(template)).result(binding).gsub(/^\s*$\n/, "")
         else
           raise ActionFailed, "Could not find Vagrantfile template #{template}"
         end
@@ -432,8 +459,8 @@ module Kitchen
         env = merged[:environment]
         %w{BUNDLE_BIN_PATH BUNDLE_GEMFILE GEM_HOME GEM_PATH GEM_ROOT RUBYLIB
            RUBYOPT _ORIGINAL_GEM_PATH}.each do |var|
-          env[var] = nil
-        end
+             env[var] = nil
+           end
 
         # Altering the path seems to break vagrant. When the :environment
         # is passed to a windows process with a PATH, Vagrant's batch installer
@@ -456,6 +483,20 @@ module Kitchen
       end
       # rubocop:enable Metrics/CyclomaticComplexity
 
+      # Tell vagrant to update vagrant box to latest version
+      def run_box_auto_update
+        if config[:box_auto_update]
+          run(config[:box_auto_update])
+        end
+      end
+
+      # Tell vagrant to remove older vagrant boxes
+      def run_box_auto_prune
+        if config[:box_auto_prune]
+          run(config[:box_auto_prune])
+        end
+      end
+
       # Runs a local command before `vagrant up` has been called.
       #
       # @api private
@@ -529,9 +570,9 @@ module Kitchen
       def vagrant_config(type)
         lines = run_silently("#{config[:vagrant_binary]} #{type}-config")
           .split("\n").map do |line|
-          tokens = line.strip.partition(" ")
-          [tokens.first, tokens.last.delete('"')]
-        end
+            tokens = line.strip.partition(" ")
+            [tokens.first, tokens.last.delete('"')]
+          end
         Hash[lines]
       end
 
@@ -540,7 +581,8 @@ module Kitchen
       # @api private
       def vagrant_version
         self.class.vagrant_version ||= run_silently(
-          "#{config[:vagrant_binary]} --version", cwd: Dir.pwd)
+          "#{config[:vagrant_binary]} --version", cwd: Dir.pwd
+        )
           .chomp.split(" ").last
       rescue Errno::ENOENT
         raise UserError, "Vagrant #{MIN_VER} or higher is not installed." \
@@ -583,7 +625,8 @@ module Kitchen
         return true if self.class.winrm_plugin_passed
 
         self.class.winrm_plugin_passed = run_silently(
-          "#{config[:vagrant_binary]} plugin list", cwd: Dir.pwd)
+          "#{config[:vagrant_binary]} plugin list", cwd: Dir.pwd
+        )
           .split("\n").find { |line| line =~ /vagrant-winrm\s+/ }
       end
     end

data/lib/kitchen/driver/vagrant_version.rb

@@ -21,6 +21,6 @@ module Kitchen
   module Driver
 
     # Version string for Vagrant Kitchen driver
-    VAGRANT_VERSION = "1.5.0".freeze
+    VAGRANT_VERSION = "1.7.0".freeze
   end
 end

data/templates/Vagrantfile.erb

@@ -92,7 +92,7 @@ Vagrant.configure("2") do |c|
   c.vm.provider :<%= config[:provider] %> do |p|
 <% case config[:provider]
    when "virtualbox" %>
-    p.name = "kitchen-<%= File.basename(config[:kitchen_root]) %>-<%= instance.name %>"
+    p.name = "kitchen-<%= File.basename(config[:kitchen_root]) %>-<%= instance.name %>-<%= SecureRandom.uuid %>"
 <% end %>
 
 <% case config[:provider]
@@ -108,7 +108,7 @@ Vagrant.configure("2") do |c|
     p.linked_clone = <%= config[:linked_clone] %>
 <%   end
    when "hyperv" %>
-    p.vmname = "kitchen-<%= File.basename(config[:kitchen_root]) %>-<%= instance.name %>"
+    p.vmname = "<%="kitchen-#{File.basename(config[:kitchen_root])}-#{instance.name}-#{SecureRandom.uuid}"[0..99].chomp('-') %>"
 <%   if config[:linked_clone] == true || config[:linked_clone] == false %>
     p.linked_clone = <%= config[:linked_clone] %>
 <%   end

metadata

@@ -1,29 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-vagrant
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.7.0
 platform: ruby
 authors:
 - Fletcher Nichol
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-03-14 00:00:00.000000000 Z
+date: 2020-08-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-kitchen
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.4'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.4'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: countloc
   requirement: !ruby/object:Gem::Requirement
@@ -111,7 +117,7 @@ homepage: https://github.com/test-kitchen/kitchen-vagrant/
 licenses:
 - Apache-2.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -119,15 +125,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Kitchen::Driver::Vagrant - A Vagrant Driver for Test Kitchen.
 test_files: []