kitchen-hyperv 0.5.5 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9258c479984814c2d9aaf91a3686a9c5d917f190cbc48627e65d69bd911a5b61
-  data.tar.gz: 43a7870274649b11ca8597213fe6ec7c93ed18b16f5c4b90d1bfd80642fb5344
+  metadata.gz: 2081b5e1d3d52389d2136e68798b265bdbfef2fe0a3bea713e9eca1f595dfffb
+  data.tar.gz: b39c49767592691ef4783c5e4492c865d55712b5c0a6a7e7c0dc501010fac6d1
 SHA512:
-  metadata.gz: ca0012d2cc41220779907a19b573f64ff69893f1cff7d28a68b41c80ab48ee52eb3a7d6ddb30d2c408139f2a4ec327479766ce207d65b64928b4e6dd393a21b0
-  data.tar.gz: 2994a51e52ddde357eacca301732b7ec7af40accd963ae47c0484a3aaeb9c1245e15f352c8ab64ec6aa9661328a052c8213b4400a259fb00c0e24a30c5ae591f
+  metadata.gz: a017678ac18c138fc504b8169a9e662000280fe1a2a0c7dbe092b45c98e4beab978504b831b5f6e3afc4f15b46733ef58bde12a5f23b76cdb06f32cca658c3b4
+  data.tar.gz: 0eb92487319eef5e8f35ae3a4ff9826aae8dd463087091fd93f59efdc9365524352f6996e9bcba40c4b55831f91f3a7e59c4d2a90a2fd7af21330eee167ca735

data/Gemfile

@@ -10,10 +10,6 @@ group :integration do
   gem "kitchen-vagrant"
 end
 
-group :changelog do
-  gem "github_changelog_generator", "1.15.2"
-end
-
 group :debug do
   gem "pry"
   gem "pry-byebug"

data/Rakefile

@@ -1,5 +1,3 @@
-# -*- encoding: utf-8 -*-
-
 require "bundler/gem_tasks"
 require "kitchen/driver/hyperv_version"
 
@@ -27,7 +25,7 @@ desc "Run all quality tasks"
 task quality: :style
 
 begin
-  require "yard"
+  require "yard" unless defined?(YARD)
   YARD::Rake::YardocTask.new
 rescue LoadError
   puts "yard is not available. (sudo) gem install yard to generate yard documentation."

data/kitchen-hyperv.gemspec

@@ -1,5 +1,4 @@
-# encoding: utf-8
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 require "kitchen/driver/hyperv_version"
@@ -17,11 +16,14 @@ Gem::Specification.new do |spec|
   spec.files         = %w{LICENSE kitchen-hyperv.gemspec Gemfile Rakefile support/hyperv.ps1} + Dir.glob("lib/**/*")
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler"
+  spec.required_ruby_version = ">= 2.6"
+
   spec.add_development_dependency "rake"
-  spec.add_development_dependency "minitest", "~> 5.3", "< 5.15"
+  spec.add_development_dependency "minitest", "~> 5.3", "< 5.16"
   spec.add_development_dependency "minitest-stub-const"
   spec.add_development_dependency "mocha", "~> 1.1"
 
-  spec.add_dependency "test-kitchen", ">= 1.4", "< 3"
+  spec.add_dependency "test-kitchen", ">= 1.4", "< 4"
+  spec.add_dependency "train", "~> 3.5"
+  spec.add_dependency "train-winrm", "~> 0.2"
 end

data/lib/kitchen/driver/hyperv.rb

@@ -20,9 +20,11 @@ require "kitchen"
 require "kitchen/driver"
 require_relative "hyperv_version"
 require_relative "powershell"
-require "mixlib/shellout"
-require "fileutils"
-require "json"
+require "mixlib/shellout" unless defined?(Mixlib::ShellOut)
+require "fileutils" unless defined?(FileUtils)
+require "json" unless defined?(JSON)
+require "train" unless defined?(Train)
+require "train-winrm" unless defined?(TrainPlugins::WinRM)
 
 module Kitchen
 
@@ -34,8 +36,9 @@ module Kitchen
       kitchen_driver_api_version 2
       plugin_version Kitchen::Driver::HYPERV_VERSION
 
-      default_config :parent_vhd_folder
-      default_config :parent_vhd_name
+      required_config :parent_vhd_folder
+      required_config :parent_vhd_name
+
       default_config :memory_startup_bytes, 536_870_912
       default_config :dynamic_memory_min_bytes, 536_870_912
       default_config :dynamic_memory_max_bytes, 2_147_483_648
@@ -60,6 +63,13 @@ module Kitchen
         File.extname(driver[:parent_vhd_name])
       end
 
+      default_config :hyperv_server, nil
+      default_config :hyperv_username, nil
+      default_config :hyperv_password, nil
+      default_config :hyperv_ssl, false
+      default_config :hyperv_insecure, true
+      default_config :remote_vm_path, 'C:\Users\Public\Documents\Hyper-V'
+
       include Kitchen::Driver::PowerShellScripts
 
       def create(state)
@@ -94,8 +104,8 @@ module Kitchen
       private
 
       def validate_vm_settings
-        raise "Missing parent_vhd_folder" unless vhd_folder?
-        raise "Missing parent_vhd_name" unless vhd?
+        raise "Missing parent_vhd_folder" unless vhd_folder? || remote_hyperv
+        raise "Missing parent_vhd_name" unless vhd? || remote_hyperv
 
         if config[:dynamic_memory]
           startup_bytes = config[:memory_startup_bytes]
@@ -160,6 +170,8 @@ module Kitchen
 
         info("Creating virtual machine for #{instance.name}.")
         new_vm_object = run_ps new_vm_ps
+        raise "Unable to create virtual machine for #{instance.name}." if new_vm_object.nil?
+
         @state[:id] = new_vm_object["Id"]
         info("Created virtual machine for #{instance.name}.")
       end
@@ -244,6 +256,8 @@ module Kitchen
       end
 
       def remove_differencing_disk
+        return unless differencing_disk_exists
+
         info("Removing the differencing disk for #{instance.name}.")
         FileUtils.rm(differencing_disk_path)
         info("Removed the differencing disk for #{instance.name}.")
@@ -269,12 +283,18 @@ module Kitchen
         @kitchen_vm_path ||= File.join(config[:kitchen_root], ".kitchen/#{instance.name}")
       end
 
+      def remote_kitchen_vm_path
+        config[:remote_vm_path]
+      end
+
       def boot_iso_path
         @boot_iso_path ||= config[:boot_iso_path]
       end
 
       def differencing_disk_path
-        @differencing_disk_path ||= File.join(kitchen_vm_path, "diff" + "#{config[:disk_type]}")
+        kitchen_vm_base = remote_hyperv ? remote_kitchen_vm_path : kitchen_vm_path
+
+        @differencing_disk_path ||= File.join(kitchen_vm_base, "diff" + "#{config[:disk_type]}")
       end
 
       def additional_disk_path(disk_name, disk_type)
@@ -292,6 +312,44 @@ module Kitchen
       def vhd?
         config[:parent_vhd_name] && File.exist?(parent_vhd_path)
       end
+
+      def remote_hyperv
+        !!config[:hyperv_server]
+      end
+
+      def connection
+        return @connection if @connection
+
+        backend = remote_hyperv ? "winrm" : "local"
+
+        train = Train.create(backend, {
+                              host:        config[:hyperv_server],
+                              user:        config[:hyperv_username],
+                              password:    config[:hyperv_password],
+                              ssl:         config[:hyperv_ssl],
+                              self_signed: config[:hyperv_insecure],
+                            })
+        @connection = train.connection
+
+        # Copy support PS1
+        @connection.upload(local_script_path, remote_script_path)
+
+        @connection
+      end
+
+      def base_script_path
+        return remote_script_path if remote_hyperv
+
+        local_script_path
+      end
+
+      def local_script_path
+        File.join(File.dirname(__FILE__), "/../../../support/hyperv.ps1")
+      end
+
+      def remote_script_path
+        File.join(config[:kitchen_root], "kitchen-hyperv", "hyperv.ps1")
+      end
     end
   end
 end

data/lib/kitchen/driver/hyperv_version.rb

@@ -17,6 +17,6 @@
 
 module Kitchen
   module Driver
-    HYPERV_VERSION = "0.5.5".freeze
+    HYPERV_VERSION = "0.7.1".freeze
   end
 end

data/lib/kitchen/driver/powershell.rb

@@ -15,9 +15,10 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require "mixlib/shellout"
-require "fileutils"
-require "json"
+require "mixlib/shellout" unless defined?(Mixlib::ShellOut)
+require "benchmark" unless defined?(Benchmark)
+require "fileutils" unless defined?(FileUtils)
+require "json" unless defined?(JSON)
 
 module Kitchen
   module Driver
@@ -28,6 +29,8 @@ module Kitchen
       end
 
       def is_64bit?
+        return true if remote_hyperv
+
         os_arch = ENV["PROCESSOR_ARCHITEW6432"] || ENV["PROCESSOR_ARCHITECTURE"]
         ruby_arch = ["foo"].pack("p").size == 4 ? 32 : 64
         os_arch == "AMD64" && ruby_arch == 64
@@ -48,7 +51,6 @@ module Kitchen
       end
 
       def wrap_command(script)
-        base_script_path = File.join(File.dirname(__FILE__), "/../../../support/hyperv.ps1")
         debug("Loading functions from #{base_script_path}")
         new_script = [ ". #{base_script_path}", "#{script}" ].join(";\n")
         debug("Wrapped script: #{new_script}")
@@ -71,11 +73,15 @@ module Kitchen
       end
 
       def execute_command(cmd, options = {})
-        debug("#Local Command BEGIN (#{cmd})")
-        sh = Mixlib::ShellOut.new(cmd, options)
-        sh.run_command
-        debug("Local Command END #{Util.duration(sh.execution_time)}")
-        raise "Failed: #{sh.stderr}" if sh.error?
+        debug("#Command BEGIN (#{cmd})")
+
+        sh = nil
+        bm = Benchmark.measure do
+          sh = connection.run_command(cmd, options)
+        end
+
+        debug("Command END #{Util.duration(bm.total)}")
+        raise "Failed: #{sh.stderr}" if sh.exit_status != 0
 
         stdout = sanitize_stdout(sh.stdout)
         JSON.parse(stdout) if stdout.length > 2
@@ -139,6 +145,7 @@ module Kitchen
         EOH
       end
 
+      # TODO: Report if VM has no IP address instead of silently waiting forever
       def vm_details_ps
         <<-DETAILS
 

data/support/hyperv.ps1

@@ -18,8 +18,8 @@ function New-DifferencingDisk {
         [parameter(Mandatory)]
         [ValidateNotNullOrEmpty()]
         [string[]]$Path,
-        [parameter(Mandatory)]        
-        [ValidateNotNullOrEmpty()] 
+        [parameter(Mandatory)]
+        [ValidateNotNullOrEmpty()]
         [string]$ParentPath
     )
     if (-not (Test-Path $Path)) {
@@ -73,9 +73,9 @@ function New-KitchenVM {
         $EnableGuestServices,
         $AdditionalDisks
     )
-    $null = $psboundparameters.remove('DisableSecureBoot')    
+    $null = $psboundparameters.remove('DisableSecureBoot')
     $null = $psboundparameters.remove('ProcessorCount')
-    $null = $psboundparameters.remove('StaticMacAddress') 
+    $null = $psboundparameters.remove('StaticMacAddress')
     $null = $psboundparameters.remove('UseDynamicMemory')
     $null = $psboundparameters.remove('DynamicMemoryMinBytes')
     $null = $psboundparameters.remove('DynamicMemoryMaxBytes')
@@ -99,7 +99,7 @@ function New-KitchenVM {
     if (-not [string]::IsNullOrEmpty($boot_iso_path)) {
         Mount-VMISO -Id $vm.Id -Path $boot_iso_path
     }
-    if ($StaticMacAddress -ne $null) {
+    if (-not [string]::IsNullOrEmpty($StaticMacAddress)) {
         Set-VMNetworkAdapter -VMName $vm.VMName -StaticMacAddress $StaticMacAddress
     }
     if ($EnableGuestServices -and (Get-command Enable-VMIntegrationService -ErrorAction SilentlyContinue)) {

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-hyperv
 version: !ruby/object:Gem::Version
-  version: 0.5.5
+  version: 0.7.1
 platform: ruby
 authors:
 - Steven Murawski
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-17 00:00:00.000000000 Z
+date: 2022-04-13 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -47,7 +33,7 @@ dependencies:
         version: '5.3'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.15'
+        version: '5.16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -57,7 +43,7 @@ dependencies:
         version: '5.3'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.15'
+        version: '5.16'
 - !ruby/object:Gem::Dependency
   name: minitest-stub-const
   requirement: !ruby/object:Gem::Requirement
@@ -95,7 +81,7 @@ dependencies:
         version: '1.4'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -105,7 +91,35 @@ dependencies:
         version: '1.4'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '4'
+- !ruby/object:Gem::Dependency
+  name: train
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+- !ruby/object:Gem::Dependency
+  name: train-winrm
+  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'
 description: Hyper-V Driver for Test-Kitchen
 email:
 - steven.murawski@gmail.com
@@ -125,7 +139,7 @@ homepage: https://github.com/test-kitchen/kitchen-hyperv
 licenses:
 - Apache-2.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -133,15 +147,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
 summary: Hyper-V Driver for Test-Kitchen
 test_files: []