kitchen-vagrant 2.2.0 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f38360fb2e76468a3b8fec2cc9605529a3627fb6887fc0054db601ef2b14c1b
-  data.tar.gz: 8697570d8be74b4e5f674707be02a1aa45b21acd2836b033075e0bea0667bf76
+  metadata.gz: cf0fbd431e7e5a52a144f6ce2e79edea1d2783c74c0ef212c3674a12e147683e
+  data.tar.gz: fdde8621c74520cf97a5367bfa3098aedc814d8037ad564ef8cada2056237807
 SHA512:
-  metadata.gz: 2742d9a6cf27c11dd986ff5054c9c389c94637b7cfc50bb677ca43411e8b3d336d3adce1bf50be6a20ab524f41c8cfa24237b2591d265a0b46843adaffb429e0
-  data.tar.gz: 02cab0d3a12310cf6da7efdfc62011a621b9daa991430fc1efce6041ebbc40d18d4c5c661562eac4d055db020c0bc06f308b98bb089daf17fd579fe222d454ad
+  metadata.gz: 1877e0240f40b4d4880ba3032f9801a984252051a88ba2a8fc0bd5af9d314ac5aa701b1116e9df747614b6ca3adacbbf898be6fdfe1d3d104f41f2321d2aceaa
+  data.tar.gz: eef20bc7471c539f0b11cbb31d80337bf7037905e7a1fecc9f7104d9b9335f57bd5d5437e376bb19942d5e80dcffb5193ba5ddab7836d8c3cb315aced9149361

data/lib/kitchen/driver/helpers.rb

@@ -15,30 +15,90 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+require "base64" unless defined?(Base64)
 require "mixlib/shellout" unless defined?(Mixlib::ShellOut)
 require "fileutils" unless defined?(FileUtils)
 require "json" unless defined?(JSON)
 
 module Kitchen
+  # Test Kitchen driver plugins.
   module Driver
+    # Helpers for talking to Hyper-V from the Vagrant driver.
+    #
+    # Hyper-V has no command line interface of its own, so everything here
+    # funnels through PowerShell: a script is wrapped so that it dot-sources
+    # the bundled `support/hyperv.ps1`, base64-encoded for `-EncodedCommand`
+    # (which sidesteps all shell quoting problems), executed, and its JSON
+    # output parsed back into Ruby.
+    #
+    # The module expects to be mixed into a {Kitchen::Configurable} that also
+    # provides {Kitchen::Logging}, which the Vagrant driver does.
+    #
+    # @author Steven Murawski <smurawski@chef.io>
     module HypervHelpers
+      # Encodes a PowerShell script for `powershell.exe -EncodedCommand`.
+      #
+      # PowerShell requires UTF-16LE, not UTF-8, and strict (unwrapped) base64.
+      #
+      # @param script [String] a PowerShell script
+      # @return [String] the script as single-line base64
+      # @api private
       def encode_command(script)
         encoded_script = script.encode("UTF-16LE", "UTF-8")
         Base64.strict_encode64(encoded_script)
       end
 
+      # The processor architecture Windows reports for this process.
+      #
+      # `PROCESSOR_ARCHITEW6432` is only set for a 32-bit process running under
+      # WOW64, where it holds the *machine's* architecture while
+      # `PROCESSOR_ARCHITECTURE` holds the emulated one -- so it wins when
+      # present.
+      #
+      # @return [String,nil] e.g. `"AMD64"`, or nil off Windows
+      # @api private
+      def os_architecture
+        ENV["PROCESSOR_ARCHITEW6432"] || ENV["PROCESSOR_ARCHITECTURE"]
+      end
+
+      # Whether the running Ruby is a 64-bit build, determined from the size of
+      # a packed pointer.
+      #
+      # @return [true,false] whether this Ruby is 64-bit
+      # @api private
+      def ruby_64bit?
+        ["foo"].pack("p").size != 4
+      end
+
+      # Whether both the OS and the running Ruby are 64-bit.
+      #
+      # @return [true,false] whether this is a 64-bit Ruby on a 64-bit Windows
+      # @api private
       def is_64bit?
-        os_arch = ENV["PROCESSOR_ARCHITEW6432"] || ENV["PROCESSOR_ARCHITECTURE"]
-        ruby_arch = ["foo"].pack("p").size == 4 ? 32 : 64
-        os_arch == "AMD64" && ruby_arch == 64
+        os_architecture == "AMD64" && ruby_64bit?
       end
 
+      # Whether both the OS and the running Ruby are 32-bit.
+      #
+      # Note this is deliberately *not* the negation of {#is_64bit?}: a 32-bit
+      # Ruby on a 64-bit Windows (the WOW64 case) is neither.
+      #
+      # @return [true,false] whether this is a 32-bit Ruby on a 32-bit Windows
+      # @api private
       def is_32bit?
-        os_arch = ENV["PROCESSOR_ARCHITEW6432"] || ENV["PROCESSOR_ARCHITECTURE"]
-        ruby_arch = ["foo"].pack("p").size == 4 ? 32 : 64
-        os_arch != "AMD64" && ruby_arch == 32
+        os_architecture != "AMD64" && !ruby_64bit?
       end
 
+      # Path to a PowerShell that matches the machine's architecture.
+      #
+      # When Ruby and Windows agree on bitness, the real `System32` PowerShell
+      # is correct. When they disagree -- a 32-bit Ruby on 64-bit Windows --
+      # `System32` would be silently redirected by WOW64 to the 32-bit
+      # PowerShell, which cannot see the Hyper-V cmdlets; `Sysnative` is the
+      # alias that reaches the 64-bit one.
+      #
+      # @return [String] absolute path to powershell.exe
+      # @api private
       def powershell_64_bit
         if is_64bit? || is_32bit?
           'c:\windows\system32\windowspowershell\v1.0\powershell.exe'
@@ -47,6 +107,12 @@ module Kitchen
         end
       end
 
+      # Builds a full `powershell.exe` command line that dot-sources the gem's
+      # `support/hyperv.ps1` helper functions and then runs `script`.
+      #
+      # @param script [String] a PowerShell script
+      # @return [String] a command line ready to hand to a shell
+      # @api private
       def wrap_command(script)
         base_script_path = File.join(File.dirname(__FILE__), "/../../../support/hyperv.ps1")
         debug("Loading functions from #{base_script_path}")
@@ -60,6 +126,7 @@ module Kitchen
       #
       # @param cmd [String] command to run locally
       # @param options [Hash] options hash
+      # @return [Hash,nil] the parsed JSON the script emitted, if any
       # @see Kitchen::ShellOut.run_command
       # @api private
       def run_ps(cmd, options = {})
@@ -70,6 +137,13 @@ module Kitchen
         execute_command wrapped_command, options
       end
 
+      # Runs a command locally and parses its output as JSON.
+      #
+      # @param cmd [String] command to run locally
+      # @param options [Hash] options passed through to `Mixlib::ShellOut`
+      # @return [Hash,Array,nil] the parsed output, or nil if there was none
+      # @raise [RuntimeError] if the command exited non-zero
+      # @api private
       def execute_command(cmd, options = {})
         debug("#Local Command BEGIN (#{cmd})")
         sh = Mixlib::ShellOut.new(cmd, options)
@@ -81,10 +155,21 @@ module Kitchen
         JSON.parse(stdout) if stdout.length > 2
       end
 
+      # Strips the interactive prompt lines PowerShell interleaves with real
+      # output, which would otherwise make the result unparseable as JSON.
+      #
+      # @param stdout [String] raw standard output
+      # @return [String] output with `PS ...>` lines removed
+      # @api private
       def sanitize_stdout(stdout)
         stdout.split("\n").select { |s| !s.start_with?("PS") }.join("\n")
       end
 
+      # Asks Hyper-V for the virtual switch new VMs should be attached to.
+      #
+      # @return [String] the name of the switch
+      # @raise [RuntimeError] if no usable switch could be determined
+      # @api private
       def hyperv_switch
         default_switch_object = run_ps hyperv_default_switch_ps
         if default_switch_object.nil? ||
@@ -96,6 +181,11 @@ module Kitchen
         default_switch_object["Name"]
       end
 
+      # The PowerShell that {#hyperv_switch} runs. Honours
+      # `KITCHEN_HYPERV_SWITCH`; without it, `Get-DefaultVMSwitch` picks one.
+      #
+      # @return [String] a PowerShell script emitting a JSON switch object
+      # @api private
       def hyperv_default_switch_ps
         <<-VMSWITCH
           Get-DefaultVMSwitch #{ENV["KITCHEN_HYPERV_SWITCH"]} | ConvertTo-Json
@@ -104,6 +194,11 @@ module Kitchen
 
       private
 
+      # Renders a Ruby Array as a PowerShell array literal.
+      #
+      # @param list [Array,nil] the values to render
+      # @return [String] e.g. `@("a", "b")`, or `@()` when empty
+      # @api private
       def ruby_array_to_ps_array(list)
         return "@()" if list.nil? || list.empty?
 

data/lib/kitchen/driver/vagrant.rb

@@ -80,6 +80,8 @@ module Kitchen
 
       default_config :synced_folders, []
 
+      default_config :env, []
+
       default_config :use_cached_chef_client, false
 
       default_config :vagrant_binary, "vagrant"
@@ -125,6 +127,11 @@ module Kitchen
         info("Vagrant instance #{instance.to_str} created.")
       end
 
+      # The box this Instance should use when the user has not named one.
+      #
+      # Platforms the Bento project builds are mapped onto their `bento/`
+      # box; anything else is assumed to name a box directly.
+      #
       # @return [String,nil] the Vagrant box for this Instance
       def default_box
         if bento_box?(instance.platform.name)
@@ -134,6 +141,11 @@ module Kitchen
         end
       end
 
+      # The box URL this Instance should use when the user has not named one.
+      #
+      # Always nil: modern Vagrant resolves boxes through Vagrant Cloud, so an
+      # explicit URL is only needed for privately hosted boxes.
+      #
       # @return [String,nil] the Vagrant box URL for this Instance
       def default_box_url
         nil
@@ -155,6 +167,12 @@ module Kitchen
         state.delete(:hostname)
       end
 
+      # Packages a created instance into a redistributable `.box` file in the
+      # current working directory, then destroys the instance.
+      #
+      # @param state [Hash] mutable instance state
+      # @raise [UserError] if the instance has not been created
+      # @raise [ActionFailed] if the action could not be completed
       def package(state)
         if state[:hostname].nil?
           raise UserError, "Vagrant instance not created!"
@@ -214,9 +232,11 @@ module Kitchen
         instance.transport.name.downcase =~ /win_?rm/
       end
 
-      # Setting up the `cache_directory` to store omnibus packages in cache
-      # and share a local folder to that directory so that we don't pull them
-      # down every single time
+      # The guest-side directory that the host's omnibus package cache should
+      # be shared into, so repeated converges do not re-download packages.
+      #
+      # @return [String,false] the guest path, or false if caching does not
+      #   apply to this box and provider combination
       def cache_directory
         if enable_cache?
           config[:cache_directory]
@@ -227,7 +247,10 @@ module Kitchen
 
       protected
 
+      # Where users are pointed when Vagrant is missing or too old.
       WEBSITE = "https://developer.hashicorp.com/vagrant/install".freeze
+
+      # The oldest Vagrant this driver supports.
       MIN_VER = "2.4.0".freeze
 
       class << self
@@ -236,10 +259,11 @@ module Kitchen
         attr_accessor :vagrant_version
       end
 
-      # Retuns whether or not a platform name could have a correcponding Bento
+      # Returns whether or not a platform name could have a corresponding Bento
       # box produced by the Bento project.
       # (https://github.com/chef/bento).
       #
+      # @param name [String] a Test Kitchen platform name
       # @return [TrueClass,FalseClass] whether or not the name could be a Bento
       #   box
       # @api private
@@ -248,8 +272,13 @@ module Kitchen
       end
 
       # Returns whether or not the we expect the box to work with shared folders
-      # by matching against a whitelist of bento boxes
-      # @return [TrueClass,FalseClass] whether or not the box shoud work with
+      # by matching against a whitelist of bento boxes.
+      #
+      # Providers without usable shared folder support are excluded outright,
+      # whatever the box.
+      #
+      # @param box [String] the Vagrant box name
+      # @return [TrueClass,FalseClass] whether or not the box should work with
       #   shared folders
       # @api private
       def safe_share?(box)
@@ -259,7 +288,11 @@ module Kitchen
       end
 
       # Return true if we found the criteria to enable the cache_directory
-      # functionality
+      # functionality.
+      #
+      # @return [TrueClass,FalseClass] whether the package cache should be
+      #   shared into the guest
+      # @api private
       def enable_cache?
         return false unless config[:cache_directory]
         return true if safe_share?(config[:box])
@@ -269,8 +302,10 @@ module Kitchen
         false
       end
 
-      # Renders and writes out a Vagrantfile dedicated to this instance.
+      # Renders and writes out a Vagrantfile dedicated to this instance. A
+      # no-op if this action has already written one.
       #
+      # @return [void]
       # @api private
       def create_vagrantfile
         return if @vagrantfile_created
@@ -286,6 +321,7 @@ module Kitchen
       # Logs the Vagrantfile's contents to the debug log level.
       #
       # @param vagrantfile [String] path to the Vagrantfile
+      # @return [void]
       # @api private
       def debug_vagrantfile(vagrantfile)
         return unless logger.debug?
@@ -295,8 +331,10 @@ module Kitchen
         debug("------------")
       end
 
-      # Setup path for CA cert
+      # Expands `:box_download_ca_cert` relative to the kitchen root, so a
+      # `.kitchen.yml` can refer to a cert alongside itself.
       #
+      # @return [void]
       # @api private
       def finalize_ca_cert!
         unless config[:box_download_ca_cert].nil?
@@ -306,9 +344,14 @@ module Kitchen
         end
       end
 
-      # Create vagrant command to update box to the latest version
+      # Replaces a truthy `:box_auto_update` with the `vagrant box update`
+      # command line that {#run_box_auto_update} should run. A falsey value is
+      # left alone so that `box_auto_update: false` stays disabled.
+      #
+      # @return [void]
+      # @api private
       def finalize_box_auto_update!
-        return if config[:box_auto_update].nil?
+        return unless config[:box_auto_update]
 
         cmd = "#{config[:vagrant_binary]} box update --box #{config[:box]}"
         cmd += " --architecture #{config[:box_arch]}" if config[:box_arch]
@@ -317,9 +360,14 @@ module Kitchen
         config[:box_auto_update] = cmd
       end
 
-      # Create vagrant command to remove older versions of the box
+      # Replaces a truthy `:box_auto_prune` with the `vagrant box prune`
+      # command line that {#run_box_auto_prune} should run. A falsey value is
+      # left alone so that `box_auto_prune: false` stays disabled.
+      #
+      # @return [void]
+      # @api private
       def finalize_box_auto_prune!
-        return if config[:box_auto_prune].nil?
+        return unless config[:box_auto_prune]
 
         cmd = "#{config[:vagrant_binary]} box prune --force --keep-active-boxes --name #{config[:box]}"
         cmd += " --provider #{config[:provider]}" if config[:provider]
@@ -336,8 +384,45 @@ module Kitchen
           .gsub("{{vagrant_root}}", vagrant_root)
       end
 
-      # Replaces an `%{instance_name}` tokens in the synced folder items.
+      # Formats synced folder options for use in the Vagrantfile.
+      # Accepts either a Hash (converts to Ruby hash syntax) or a String (returns as-is).
+      # Supports SMB options like smb_username, smb_password, etc.
+      #
+      # @param options [Hash, String, nil] synced folder options
+      # @return [String] formatted options string for Vagrantfile
+      # @api private
+      def format_synced_folder_options(options)
+        return "nil" if options.nil?
+        return options if options.is_a?(String)
+
+        # Convert Hash to Ruby hash literal syntax
+        if options.is_a?(Hash)
+          options.map { |k, v| "#{k}: #{v.inspect}" }.join(", ")
+        else
+          options.to_s
+        end
+      end
+
+      # Formats network options for use in the Vagrantfile.
+      #
+      # Accepts either a Hash (rendered as Ruby keyword syntax) or a String
+      # (passed through as-is, which is what {#finalize_network!} produces).
+      #
+      # @param options [Hash,String,#to_s] network options
+      # @return [String] formatted options string for Vagrantfile
+      # @api private
+      def format_network_options(options)
+        return options if options.is_a?(String)
+        return options.map { |k, v| "#{k}: #{v.inspect}" }.join(", ") if options.is_a?(Hash)
+
+        options.to_s
+      end
+
+      # Normalises every `:synced_folders` entry: expands the host path
+      # against the kitchen root, substitutes `%{instance_name}` tokens, and
+      # formats the options for the template.
       #
+      # @return [void]
       # @api private
       def finalize_synced_folders!
         config[:synced_folders] = config[:synced_folders]
@@ -348,15 +433,17 @@ module Kitchen
                 config[:kitchen_root]
               ),
               destination.gsub("%{instance_name}", instance.name),
-              options || "nil",
+              format_synced_folder_options(options),
             ]
           end
         add_extra_synced_folders!
       end
 
-      # We would like to sync a local folder to the instance so we can
-      # take advantage of the packages that we might have in cache,
-      # therefore we wont download a package we already have
+      # Shares the host's omnibus package cache into the guest so a converge
+      # does not re-download packages it already has.
+      #
+      # @return [void]
+      # @api private
       def add_extra_synced_folders!
         if cache_directory
           FileUtils.mkdir_p(local_kitchen_cache)
@@ -368,9 +455,12 @@ module Kitchen
         end
       end
 
-      # Truncates the length of `:vm_hostname` to 12 characters for
-      # Windows-based operating systems.
+      # Truncates an over-long `:vm_hostname` on Windows guests, where the
+      # NetBIOS name is capped at 15 characters. The final character of the
+      # original is kept as a suffix so that names which differ only in their
+      # tail do not collapse onto each other.
       #
+      # @return [void]
       # @api private
       def finalize_vm_hostname!
         string = config[:vm_hostname]
@@ -380,17 +470,20 @@ module Kitchen
         end
       end
 
-      # If Hyper-V and no network configuration
-      # check KITCHEN_HYPERV_SWITCH and fallback to helper method
-      # to select the best switch
+      # Gives a Hyper-V instance a default `public_network` bridged onto the
+      # switch named by `KITCHEN_HYPERV_SWITCH`, or whichever switch
+      # {HypervHelpers#hyperv_switch} considers best. Instances that already
+      # configure a network, and every other provider, are left alone.
+      #
+      # @return [void]
       # @api private
       def finalize_network!
-        if config[:provider] == "hyperv" && config[:network].empty?
-          config[:network].push([
-            "public_network",
-            "bridge: \"#{hyperv_switch}\"",
-            ])
-        end
+        return unless config[:provider] == "hyperv" && config[:network].empty?
+
+        # Deliberately a new Array rather than a push: `default_config` hands
+        # every instance the *same* Array object, so mutating it in place would
+        # leak this network into every other instance in the process.
+        config[:network] = [["public_network", %{bridge: "#{hyperv_switch}"}]]
       end
 
       # Renders the Vagrantfile ERb template.
@@ -433,6 +526,9 @@ module Kitchen
       # any bundler environment should we detect one.  Otherwise, subcommands
       # will inherit our bundled environment.
       # @see https://github.com/test-kitchen/kitchen-vagrant/issues/190
+      # @param cmd [String] command to run locally
+      # @param options [Hash] options hash
+      # @return [String] the standard output of the command
       # @see Kitchen::ShellOut#run_command
       # rubocop:disable Metrics/CyclomaticComplexity
       def run_command(cmd, options = {})
@@ -458,10 +554,10 @@ module Kitchen
         # is passed to a windows process with a PATH, Vagrant's batch installer
         # (https://github.com/mitchellh/vagrant-installers/blob/master/substrate
         # /modules/vagrant_installer/templates/windows_vagrant.bat.erb)
-        # does not efectively prepend the vagrant ruby path in a persistent
+        # does not effectively prepend the vagrant ruby path in a persistent
         # manner which causes vagrant to use the same ruby as test-kitchen and
         # then the environment is essentially corrupted leading to many errors
-        # and dispair
+        # and despair
         unless windows_host?
           gem_home = ENV["GEM_HOME"]
           if gem_home && (env["PATH"] || ENV["PATH"])
@@ -475,8 +571,11 @@ module Kitchen
       end
       # rubocop:enable Metrics/CyclomaticComplexity
 
-      # Check if a newer version of the vagrant box is available and warn the user
+      # Check if a newer version of the vagrant box is available and warn the
+      # user. Skipped when `:box_auto_update` is on, since the update happens
+      # regardless.
       #
+      # @return [void]
       # @api private
       def check_box_outdated
         # Skip if box_auto_update is enabled (they'll get the update anyway)
@@ -498,6 +597,7 @@ module Kitchen
       # Parse vagrant box outdated output and warn if a new version is available
       #
       # @param output [String] output from vagrant box outdated command
+      # @return [void]
       # @api private
       def warn_if_outdated(output)
         return unless box_is_outdated?(output)
@@ -528,15 +628,20 @@ module Kitchen
           output_downcase.include?("newer version of the box")
       end
 
+      # Characters that may legitimately appear inside a box version. Vagrant
+      # quotes versions in prose ("version '202401.31.0'.") and separates them
+      # with punctuation in tabular output ("Current: 1.2.3, Latest: 4.5.6"),
+      # so the version has to end on an alphanumeric to avoid swallowing the
+      # trailing quote, comma or full stop.
+      VERSION_PATTERN = /v?(\w[\w.+-]*\w|\w)/
+
       # Extract current version from vagrant box outdated output
       #
       # @param output [String] output from vagrant box outdated command
       # @return [String, nil] current version or nil if not found
       # @api private
       def extract_current_version(output)
-        match = output.match(/Current:\s+v?(\S+)/i) ||
-          output.match(/currently have version\s+'?v?([^'.\s]+)/i)
-        match ? match[1] : nil
+        extract_version(output, /Current:\s+/i, /currently have version\s+'?/i)
       end
 
       # Extract latest version from vagrant box outdated output
@@ -545,12 +650,32 @@ module Kitchen
       # @return [String, nil] latest version or nil if not found
       # @api private
       def extract_latest_version(output)
-        match = output.match(/Latest:\s+v?(\S+)/i) ||
-          output.match(/latest is version\s+'?v?([^'.\s]+)/i)
-        match ? match[1] : nil
+        extract_version(output, /Latest:\s+/i, /latest is version\s+'?/i)
       end
 
-      # Tell vagrant to update vagrant box to latest version
+      # Finds the first version number that follows any of the given prefixes.
+      #
+      # @param output [String] output from vagrant box outdated command
+      # @param prefixes [Array<Regexp>] prefixes to look for, in priority order
+      # @return [String, nil] the version, or nil if no prefix matched
+      # @api private
+      def extract_version(output, *prefixes)
+        prefixes.each do |prefix|
+          match = output.match(/#{prefix.source}#{VERSION_PATTERN.source}/i)
+          return match[1] if match
+        end
+        nil
+      end
+
+      # Runs the `vagrant box update` command built by
+      # {#finalize_box_auto_update!}, if any.
+      #
+      # A box that has never been downloaded cannot be updated; that specific
+      # failure is expected on a first run and is swallowed.
+      #
+      # @return [void]
+      # @raise [Kitchen::ShellOut::ShellCommandFailed] for any other failure
+      # @api private
       def run_box_auto_update
         if config[:box_auto_update]
           begin
@@ -563,15 +688,21 @@ module Kitchen
         end
       end
 
-      # Tell vagrant to remove older vagrant boxes
+      # Runs the `vagrant box prune` command built by
+      # {#finalize_box_auto_prune!}, if any.
+      #
+      # @return [void]
+      # @api private
       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.
+      # Runs `:pre_create_command`, if set, from the kitchen root -- before
+      # `vagrant up`, so it can prepare anything the Vagrantfile depends on.
       #
+      # @return [void]
       # @api private
       def run_pre_create_command
         if config[:pre_create_command]
@@ -582,6 +713,8 @@ module Kitchen
       # Runs a local command without streaming the stdout to the logger.
       #
       # @param cmd [String] command to run locally
+      # @param options [Hash] options hash
+      # @return [String] the standard output of the command
       # @api private
       def run_silently(cmd, options = {})
         merged = {
@@ -592,6 +725,7 @@ module Kitchen
 
       # Runs the `vagrant up` command locally.
       #
+      # @return [void]
       # @api private
       def run_vagrant_up
         cmd = "#{config[:vagrant_binary]} up"
@@ -600,9 +734,11 @@ module Kitchen
         run(cmd)
       end
 
-      # Updates any state after creation.
+      # Records the connection details Vagrant reports for the new machine
+      # into the instance state, so the transport can reach it.
       #
       # @param state [Hash] mutable instance state
+      # @return [void]
       # @api private
       def update_state(state)
         hash = winrm_transport? ? vagrant_config(:winrm) : vagrant_config(:ssh)

data/lib/kitchen/driver/vagrant_version.rb

@@ -17,9 +17,10 @@
 
 module Kitchen
 
+  # Test Kitchen driver plugins.
   module Driver
 
     # Version string for Vagrant Kitchen driver
-    VAGRANT_VERSION = "2.2.0".freeze
+    VAGRANT_VERSION = "2.3.0".freeze
   end
 end

data/templates/Vagrantfile.erb

@@ -85,7 +85,7 @@ Vagrant.configure("2") do |c|
 <% end %>
 
 <% Array(config[:network]).each do |opts| %>
-  c.vm.network(:<%= opts[0] %>, <%= opts[1].map { |k, v| "#{k}: #{v.inspect}" }.join(', ') %>)
+  c.vm.network(:<%= opts[0] %>, <%= format_network_options(opts[1]) %>)
 <% end %>
 
   c.vm.synced_folder ".", "/vagrant", disabled: true
@@ -97,6 +97,8 @@ Vagrant.configure("2") do |c|
 <% case config[:provider]
    when "virtualbox" %>
     p.name = "kitchen-<%= File.basename(config[:kitchen_root]) %>-<%= instance.name %>-<%= SecureRandom.uuid %>"
+<% when "tart" %>
+    p.name = "kitchen-<%= File.basename(config[:kitchen_root]) %>-<%= instance.name %>"
 <% end %>
 
 <% case config[:provider]
@@ -253,4 +255,14 @@ Vagrant.configure("2") do |c|
   <% end %>
 <% end %>
   end
+
+<% if config[:env] && !config[:env].empty? %>
+  # Set environment variables
+  <% config[:env].each do |env_var| %>
+  c.vm.provision "shell", inline: <<-SHELL
+    echo 'export <%= env_var.gsub("'", "'\\\\''") %>' >> /etc/profile.d/kitchen.sh
+  SHELL
+  <% end %>
+  c.vm.provision "shell", inline: "chmod +x /etc/profile.d/kitchen.sh", run: "once"
+<% end %>
 end

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-vagrant
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Fletcher Nichol
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-11-09 00:00:00.000000000 Z
+date: 2026-08-22 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: base64
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: test-kitchen
   requirement: !ruby/object:Gem::Requirement
@@ -19,7 +33,7 @@ dependencies:
         version: '1.4'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4'
+        version: '5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +43,7 @@ dependencies:
         version: '1.4'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4'
+        version: '5'
 description: Kitchen::Driver::Vagrant - A HashiCorp Vagrant Driver for Test Kitchen.
 email:
 - fnichol@nichol.ca
@@ -55,7 +69,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.0'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="