kitchen-proxmox 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b86aaf1627109dc3fc11a425f023a03da6601eb6ef5c19e7d426654f40a1363
-  data.tar.gz: 71a7a993ce7729691b6887f84df0854023b32953b843fd7b0c1f29a3d07c253f
+  metadata.gz: 9431e030d0300d57c9fac8a20047375865cedf9f4a48b34b3039370f2d62a98a
+  data.tar.gz: 62e91255a7b9b2586eeef54bcd76bb7110b782ae8c948853ef0691a13e11df5e
 SHA512:
-  metadata.gz: 5c7640ad5482862220e38d3fb3ff3cdc3d6943b4656e5bdbb9c49169f5887c82c23044abcc6361e870625fda7fa944d7f81944e70d10255b095e230fbdd777b6
-  data.tar.gz: 9f919e44967c325720f05724b97318884719fd21b46717838d38199beebbb062c1a829c1a635092461d56fe9258c2f917e2ed97a5f18cf4af44d38b19b03c858
+  metadata.gz: e974ac8b7626ed1232e608073c259f5bb90c65aa47bdaf9327195a255dfbcbb7eb969a6cd88c5781c929835daa84f52a63e83aa6e291199de3d32889c244300b
+  data.tar.gz: 4b36857b3cfb877ff3bcf292049389cbe26cd43ab2393fb235d3236e8ad5d50ba74ed1ae55076961f97294f2df14e9a8d98d683a50afe4f37af5253ca3066f63

data/lib/kitchen/driver/proxmox/api_client.rb

@@ -5,6 +5,7 @@ require 'json'
 require 'uri'
 require 'openssl'
 require 'kitchen'
+require_relative 'errors'
 
 module Kitchen
   module Driver
@@ -145,7 +146,7 @@ module Kitchen
         end
 
         def handle_response(response)
-          raise "Proxmox API error #{response.code}: #{response.body}" unless response.is_a?(Net::HTTPSuccess)
+          raise ProxmoxErrors::ApiError.new(response.code, response.body) unless response.is_a?(Net::HTTPSuccess)
 
           JSON.parse(response.body)['data']
         end

data/lib/kitchen/driver/proxmox/errors.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Kitchen
+  module Driver
+    module ProxmoxErrors
+      # Structured error raised by ApiClient on non-2xx responses.
+      class ApiError < ::StandardError
+        attr_reader :status_code, :response_body
+
+        def initialize(status_code, response_body)
+          @status_code = status_code.to_i
+          @response_body = response_body
+          super("Proxmox API error #{status_code}: #{response_body}")
+        end
+
+        # Returns true when the error indicates a VMID is already in use.
+        def vmid_conflict?
+          return false unless status_code == 400 || status_code == 500
+
+          response_body.match?(/already exists/i) ||
+            response_body.match?(/unable to create VM \d+/i)
+        end
+      end
+    end
+  end
+end

data/lib/kitchen/driver/proxmox.rb

@@ -2,6 +2,7 @@
 
 require 'kitchen'
 require_relative 'proxmox_version'
+require_relative 'proxmox/errors'
 require_relative 'proxmox/api_client'
 
 module Kitchen
@@ -16,6 +17,8 @@ module Kitchen
 
       plugin_version Kitchen::Driver::PROXMOX_VERSION
 
+      ApiError = Kitchen::Driver::ProxmoxErrors::ApiError
+
       required_config :proxmox_url
       required_config :proxmox_token_id
       required_config :proxmox_token_secret
@@ -32,6 +35,7 @@ module Kitchen
       default_config :clone_timeout, 300
       default_config :start_timeout, 300
       default_config :ip_wait_timeout, 120
+      default_config :clone_retries, 5
 
       def create(state)
         return if state[:vm_id]
@@ -65,19 +69,43 @@ module Kitchen
       def clone_and_start(state)
         info("Creating Proxmox VM from template #{config[:template_id]}...")
 
-        vm_id = allocate_vm_id
-        vm_name = generate_vm_name(instance.name)
+        vm_id, vm_name = allocate_and_clone
+        state[:vm_id] = vm_id
+        state[:vm_name] = vm_name
 
-        clone_template(vm_id, vm_name)
         configure_hardware(vm_id)
         start_and_wait_for_ip(state, vm_id)
 
-        state[:vm_id] = vm_id
-        state[:vm_name] = vm_name
-
         info("Proxmox VM #{vm_name} (#{vm_id}) created.")
       end
 
+      def allocate_and_clone
+        retries = config[:clone_retries]
+        last_error = nil
+
+        retries.times do |attempt|
+          vm_id = allocate_vm_id
+          vm_name = generate_vm_name(instance.name)
+
+          begin
+            clone_template(vm_id, vm_name)
+            return [vm_id, vm_name]
+          rescue ApiError => e
+            raise unless e.vmid_conflict?
+
+            last_error = e
+            warn("VMID #{vm_id} conflict (attempt #{attempt + 1}/#{retries}), retrying...")
+            sleep backoff_delay(attempt)
+          end
+        end
+
+        raise last_error
+      end
+
+      def backoff_delay(attempt)
+        (0.5 * (2**attempt)) + rand(0.0..1.0)
+      end
+
       def allocate_vm_id
         Integer(api_client.next_vm_id)
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-proxmox
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Richard Nixon
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-04-30 00:00:00.000000000 Z
+date: 2026-05-03 00:00:00.000000000 Z
 dependencies: []
 description: A Test Kitchen driver for Proxmox VE. Manages VM lifecycle (create, destroy)
   via the Proxmox REST API. Supports cloning from templates.
@@ -22,6 +22,7 @@ files:
 - README.md
 - lib/kitchen/driver/proxmox.rb
 - lib/kitchen/driver/proxmox/api_client.rb
+- lib/kitchen/driver/proxmox/errors.rb
 - lib/kitchen/driver/proxmox_version.rb
 homepage: https://github.com/trickyearlobe-chef/kitchen-proxmox
 licenses: