kitchen-oci 1.15.0 → 1.15.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da622526e4097cf2ecc02289b0bb0764c83258f382341383077dd808d8efad0c
-  data.tar.gz: b8f0ce977128f24fdc25779bbe53b461657af9f1273ccac9548a9eb85a2bd56d
+  metadata.gz: 3e3fbf95593303d5f9538c59a7f07eb03dd1871b05b12ca0a82e5d2ed9b529b4
+  data.tar.gz: d0c4e60f05ebaf18293ce93bad6f6a52d5d9657de9a0d34b4bb1d61ddaf3d57e
 SHA512:
-  metadata.gz: 3507a51fbc01136b940ebc988c4a353f2615595ecbb0bfdf0487fa6d52bdad6f6fd185a990db215652f72a0e86fbe33813134287294700911380646f86f7ca4e
-  data.tar.gz: 1283e12168c453c49c2e59bfce062003c012d805e2e715ddab57cac03b0c710631d84334c835aaaeba51f0ab32ac91b0fef0e127a1de4857f46cbc90b52c8a60
+  metadata.gz: dac56f0b424e0b74ca273de0242d5f0883c95eee61c4e086a7cfa66c9382b9b933f5eb907099c83860f1a44c64c10277a0a656e56c755658a44b548b824c2875
+  data.tar.gz: 19adf812199ecbba332985656ffbcc67252107f3f3d1ae2d9fbd1401a9c05e00342bf3d0ba2285f76388ba3075b111de0cb54dab43b94de8424c9ff3359aa04c

data/lib/kitchen/driver/oci.rb

@@ -500,7 +500,7 @@ module Kitchen
         created_vol = []
         config[:volumes].each do |vol_settings|
           # convert to hash because otherwise it's an an OCI API Object and won't load
-          volume_attachment_type = vol_settings[:type].downcase || 'paravirtual'
+          volume_attachment_type = vol_settings[:type] ? vol_settings[:type].downcase : 'paravirtual'
           unless %w[iscsi paravirtual].include?(volume_attachment_type)
             info("invalid volume attachment type: #{volume_attachment_type}")
             next
@@ -519,7 +519,7 @@ module Kitchen
       end
 
       def volume_create(availability_domain, display_name, size_in_gbs, vpus_per_gb)
-        info("Creating volume <#{display_name}>...")
+        info("Creating <#{display_name}>...")
         result = blockstorage_api.create_volume(
           OCI::Core::Models::CreateVolumeDetails.new(
             compartment_id: compartment_id,
@@ -531,28 +531,29 @@ module Kitchen
         )
         get_volume_response = blockstorage_api.get_volume(result.data.id)
                                               .wait_until(:lifecycle_state, OCI::Core::Models::Volume::LIFECYCLE_STATE_AVAILABLE)
-        info("Finished creating volume <#{display_name}>.")
-        state_data = {
-          :id => get_volume_response.data.id
+        info("Finished creating <#{display_name}>.")
+        {
+          id: get_volume_response.data.id,
+          display_name: get_volume_response.data.display_name
         }
       end
 
       def volume_delete(volume_id)
-        info("Deleting volume: <#{volume_id}>...")
+        info("Deleting <#{volume_id}>...")
         blockstorage_api.delete_volume(volume_id)
         blockstorage_api.get_volume(volume_id)
                         .wait_until(:lifecycle_state, OCI::Core::Models::Volume::LIFECYCLE_STATE_TERMINATED)
-        info("Deleted volume: <#{volume_id}>")
+        info("Finished deleting <#{volume_id}>.")
       end
 
       def process_volume_attachments(state)
         attachments = []
         state[:volumes].each do |volume|
-          info("Attaching Volume: #{volume[:displayName]} - #{volume[:attachment_type]}")
+          info("Attaching <#{volume[:display_name]}>...")
           details = volume_create_attachment_details(volume, state[:server_id])
           attachment = volume_attach(details).to_hash
           attachments << attachment
-          info("Attached Volume #{volume[:displayName]} - #{volume[:attachment_type]}")
+          info("Finished attaching <#{volume[:display_name]}>.")
         end
         attachments
       end
@@ -564,7 +565,7 @@ module Kitchen
             volume_id: volume[:id],
             instance_id: instance_id
           )
-        elsif volume[:attachment_type].eq?('paravirtual')
+        elsif volume[:attachment_type].eql?('paravirtual')
           OCI::Core::Models::AttachParavirtualizedVolumeDetails.new(
             display_name: 'paravirtAttachment',
             volume_id: volume[:id],
@@ -579,19 +580,22 @@ module Kitchen
           comp_api.get_volume_attachment(result.data.id)
                   .wait_until(:lifecycle_state, OCI::Core::Models::VolumeAttachment::LIFECYCLE_STATE_ATTACHED)
         state_data = {
-          :id => get_volume_attachment_response.data.id,
-          :iqn_ipv4 => get_volume_attachment_response.data.ipv4,
-          :iqn => get_volume_attachment_response.data.iqn,
-          :port => get_volume_attachment_response.data.port,
+          id: get_volume_attachment_response.data.id
         }
+        if get_volume_attachment_response.data.attachment_type == 'iscsi'
+          state_data.store(:iqn_ipv4, get_volume_attachment_response.data.ipv4)
+          state_data.store(:iqn, get_volume_attachment_response.data.iqn)
+          state_data.store(:port, get_volume_attachment_response.data.port)
+        end
+        state_data
       end
 
       def volume_detach(volume_attachment)
-        info("Detaching volume: #{volume_attachment[:id]}")
+        info("Detaching <#{volume_attachment[:id]}>...")
         comp_api.detach_volume(volume_attachment[:id])
         comp_api.get_volume_attachment(volume_attachment[:id])
                 .wait_until(:lifecycle_state, OCI::Core::Models::VolumeAttachment::LIFECYCLE_STATE_DETACHED)
-        info("Detached volume: #{volume_attachment[:id]}")
+        info("Finished detaching <#{volume_attachment[:id]}>.")
       end
 
       #################

data/lib/kitchen/driver/oci_version.rb

@@ -20,6 +20,6 @@
 module Kitchen
   module Driver
     # Version string for Oracle OCI Kitchen driver
-    OCI_VERSION = '1.15.0'
+    OCI_VERSION = '1.15.1'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-oci
 version: !ruby/object:Gem::Version
-  version: 1.15.0
+  version: 1.15.1
 platform: ruby
 authors:
 - Stephen Pearson
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-01 00:00:00.000000000 Z
+date: 2024-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oci