foreman_kubevirt 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf485c758f474f165b3f447b543f6f125ae5a507a06c611baed7fc9c20c9db86
-  data.tar.gz: e16eca90c17c3f4f25d1686195f19609951002eac567174e826b32ad9650f76e
+  metadata.gz: 27a32cc9e111db27e1c4e447aeb8fe1c7d556a542999008f54d8593032fa7231
+  data.tar.gz: 411a38571296341bcd8cd0d79286d4caa3f15a35f359723ad7b80b3ebd522949
 SHA512:
-  metadata.gz: a9ece666c6b3af21d27a29709904f21c27cf78b234441b7e0494ca11026873e2d333974dfdd16892217ea6e6e8b7f6fa4280cbbc43e26c6927a080741d037ee0
-  data.tar.gz: 1ca9cfa2aa37f9d068f3231ea2e0f4361447f3edef30a76b50bc844500abe4080307970fa5d71c175b7d90aeb6f928b0b4030ae1d66ab7c0a9cd37c8f2bdb8a2
+  metadata.gz: afbf6ececc85edf062ccd38a1061b885b46c6cc4c5f1810883a12769d7e054825c0461dcfdabe16025624b567385ae48f2183b2791d23ae6901f462cae6952f9
+  data.tar.gz: a9965b82dcef601bc0f941ded45a2a9f8a359826d2328668a094f12951a9c92bc81263b4f4e5f0380f11f6cbf9cc12e4b879e5e7363b203ae1616ba7157499c4

data/app/models/concerns/fog_extensions/kubevirt/server.rb

@@ -14,7 +14,7 @@ module FogExtensions
           return fog_memory if fog_memory.is_a?(Numeric)
           return nil if fog_memory.blank?
 
-          ::Fog::Kubevirt::Utils::UnitConverter.convert(fog_memory, :b).to_i
+          ::Fog::Kubevirt::Utils::UnitConverter.convert("#{fog_memory}b", :b)
         rescue StandardError => e
           Rails.logger.warn("Failed to convert memory '#{fog_memory}': #{e.message}")
           nil
@@ -35,6 +35,10 @@ module FogExtensions
       def volumes_attributes=(attrs)
       end
 
+      def id
+        name
+      end
+
       def uuid
         name
       end

data/app/models/foreman_kubevirt/kubevirt.rb

@@ -280,10 +280,10 @@ module ForemanKubevirt
 
     def new_vm(attr = {})
       vm = super
-      interfaces = nested_attributes_for :interfaces, attr[:interfaces_attributes]
-      interfaces.map { |i| vm.interfaces << new_interface(i) }
-      volumes = nested_attributes_for :volumes, attr[:volumes_attributes]
-      volumes.map { |v| vm.volumes << new_volume(v) }
+      iface_nested_attrs = nested_attributes_for :interfaces, attr[:interfaces_attributes]
+      vm.interfaces = iface_nested_attrs.map { |i| new_interface(i) }
+      volume_nested_attrs = nested_attributes_for :volumes, attr[:volumes_attributes]
+      vm.volumes = volume_nested_attrs.map { |v| new_volume(v) }
       vm
     end
 

data/lib/foreman_kubevirt/version.rb

@@ -1,3 +1,3 @@
 module ForemanKubevirt
-  VERSION = '0.5.2'.freeze
+  VERSION = '0.5.3'.freeze
 end

data/test/models/compute_resources/kubevirt_test.rb

@@ -57,6 +57,8 @@ class ForemanKubevirtTest < ActiveSupport::TestCase
 
     test "raises an error for image based provisioning without an explicit boot volume" do
       record = new_kubevirt_vcr
+      client = mocked_client
+      record.stubs(:client).returns(client)
 
       error = assert_raises(Foreman::Exception) do
         record.create_vm({ :name => "test", :provision_method => 'image', :image_id => "default/template", :volumes_attributes => {}, :interfaces_attributes => { "0" => { "cni_provider" => "multus", "network" => "default/network" } } })
@@ -93,6 +95,8 @@ class ForemanKubevirtTest < ActiveSupport::TestCase
 
     test "raises an error for image based provisioning with only an extra data volume" do
       record = new_kubevirt_vcr
+      client = mocked_client
+      record.stubs(:client).returns(client)
 
       error = assert_raises(Foreman::Exception) do
         record.create_vm({ :name => "test", :provision_method => 'image', :image_id => "default/template", :volumes_attributes => { "0" => { :capacity => "10" } }, :interfaces_attributes => { "0" => { "cni_provider" => "multus", "network" => "default/network" } } })

data/test/unit/foreman_kubevirt_test.rb

@@ -161,6 +161,6 @@ class ForemanKubevirtTest < ActiveSupport::TestCase
     server = compute_resource.create_vm(vm_args)
 
     # verify default memory value is set
-    assert_equal "1024M", server.memory
+    assert_equal 1_024_000_000, server.memory
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_kubevirt
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - Moti Asayag