chef-provisioning-opennebula 0.4.7 → 0.4.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 04a47001d99e4d12e8d704f0f441e80582955566
-  data.tar.gz: d7a148664849bd390c62620eb80434077a4bcb0d
+  metadata.gz: 798eb6e4b57c623beb0ae99b82260ddcd73e31be
+  data.tar.gz: 04bcf4eb734161c936cbfbd3a47c07ab1bb4f9d3
 SHA512:
-  metadata.gz: 0c92fd314d1d51900a3d6ea3ce52316fa6ebf5123c988fe815b2e1fdb2a926ddc790cd9b657f51cd6799c6790a91ced34b040e3c71a681a6d555e787d8c8cef7
-  data.tar.gz: 209eb0984670a5ecfe358c8db13369952ccec9f988b2698dc3eca504df14988a32610cf560a4fae163830ebf85e7d51588f7ad8bf7d0e5947c1bfcb5e3dd7ed3
+  metadata.gz: d24fd26f66e346ac1049f4c5b36c3197402629cc5a7906caf789ffcfd6514b176e9d36af65a2b47b97ab125255975533f7efb9f5fc199c71cf13509bd8d65d3a
+  data.tar.gz: ce67ffb81da9cddb2dce49b5af2909562c870ecce4b5833fd7543883e0b9b85055498fc266d0458baa3979d3e667892bf4adc6231723315f59190e6556de226b

data/lib/chef/provider/one_image.rb

@@ -49,7 +49,100 @@ class Chef
 
       action :allocate do
         if exists?
-          action_handler.report_progress "image '#{new_resource.name}' already exists - (up to date)"
+          # Image already exists, check whether we need/can update it
+          # OpenNebula Image example:
+          # ID             : 14202
+          # NAME           : ey-test
+          # USER           : Mandolin
+          # GROUP          : users
+          # DATASTORE      : orn-svc01-ds
+          # TYPE           : DATABLOCK
+          # REGISTER TIME  : 10/19 18:16:02
+          # PERSISTENT     : No
+          # SOURCE         : /var/lib/one/datastores/103/743dff63f337a0192b13f2963f92d741
+          # FSTYPE         : raw
+          # SIZE           : 1000M
+          # STATE          : rdy
+          # RUNNING_VMS    : 0
+          #
+          # PERMISSIONS
+          # OWNER          : um-
+          # GROUP          : ---
+          # OTHER          : ---
+          #
+          # IMAGE TEMPLATE
+          # DESCRIPTION="what a heck!"
+          # DEV_PREFIX="vd"
+          # DRIVER="qcow2"
+          # FSTYPE="ext4"
+          # SIZE="100"
+          #
+          # We can update many parameters and whatever we update goes into section below 'IMAGE TEMPLATE'.
+          # IMPORTANT: if the parameter we are updating exists above 'IMAGE TEMPLATE' then our modification has no effect.
+          # In other words a value for a parameter defined above wins over parameter defined below 'IMAGE TEMPLATE'.
+
+          on_image = @image.to_hash['IMAGE']
+
+          # We can update only following attributes. This is a map of one_image attributes to OpenNebula image attributes
+          attrs_map = { 'name' => 'NAME', 'size' => 'SIZE', 'datastore_id' => 'DATASTORE_ID', 'type' => 'TYPE', 'description' => 'DESCRIPTION', 'fs_type' => 'FSTYPE',
+                        'img_driver' => 'DRIVER', 'prefix' => 'DEV_PREFIX', 'persistent' => 'PERSISTENT', 'mode' => 'PERMISSIONS', 'disk_type' => 'DISK_TYPE' }
+
+          # Find out what attribute needs to be updated
+          attrs_to_update = {}
+          new_resource_hash = @new_resource.to_hash # hash keys are symbols
+          new_resource_hash.each do |k, v|
+            next if v.nil? || !attrs_map.key?(k.to_s)
+            v = v.to_s # everything is String in what we get in OpenNebula Image info
+            on_attr = attrs_map[k.to_s]
+
+            # For some one_image attributes provided in new_resource we need to find respective values in ON Image
+            case k
+            when :type
+              image_types = %w(OS CDROM DATABLOCK KERNEL RAMDISK CONTEXT)
+              on_image['TYPE'] = image_types[on_image['TYPE'].to_i] # convert Image Type Id into String
+            when :persistent
+              on_image['PERSISTENT'] = (on_image['PERSISTENT'] == '1' ? 'true' : 'false')
+            when :mode
+              perm = on_image['PERMISSIONS']
+              perm_octet_u = perm['OWNER_U'].to_i * 4 + perm['OWNER_M'].to_i * 2 + perm['OWNER_A'].to_i
+              perm_octet_g = perm['GROUP_U'].to_i * 4 + perm['GROUP_M'].to_i * 2 + perm['GROUP_A'].to_i
+              perm_octet_o = perm['OTHER_U'].to_i * 4 + perm['OTHER_M'].to_i * 2 + perm['OTHER_A'].to_i
+              on_image['PERMISSIONS'] = "#{perm_octet_u}#{perm_octet_g}#{perm_octet_o}"
+            when :disk_type
+              disk_types = %w(BLOCK CDROM FILE)
+              on_image['DISK_TYPE'] = disk_types[on_image['DISK_TYPE'].to_i] # convert Disk Type into String
+            end
+            next if on_image.key?(on_attr) && (v == on_image[on_attr])
+            next if on_image['TEMPLATE'].key?(on_attr) && (new_resource_hash[k] == on_image['TEMPLATE'][on_attr])
+            fail "Cannot update '#{on_attr}' as it is defined above 'IMAGE TEMPLATE' section." if on_image.key?(on_attr) && !on_image[on_attr].empty? && on_attr != 'PERMISSIONS'
+            attrs_to_update[on_attr] = v
+          end
+
+          unless attrs_to_update.empty?
+            # Prepare template to update
+            img_template = ''
+            attrs_to_update.each do |k, v|
+              next if k == 'PERMISSIONS' # needs special treatment
+              img_template << case k
+                              when 'SIZE', 'PERSISTENT'
+                                "#{k} = #{v}\n"
+                              when 'TYPE', 'DESCRIPTION', 'FSTYPE', 'DRIVER', 'DEV_PREFIX', 'DISK_TYPE'
+                                "#{k} = \"#{v}\"\n"
+                              end
+            end
+            # Perform actual update
+            description = "updated image '#{new_resource.name}'\n" + attrs_to_update.to_s
+            action_handler.perform_action description do
+              unless img_template == '' # can happen when we update only PERMISSIONS
+                rc = @image.update(img_template, true)
+                fail "failed to update image '#{new_resource.name}': #{rc.message}" if OpenNebula.is_error?(rc)
+              end
+              if attrs_to_update.key?('PERMISSIONS')
+                rc = @image.chmod_octet(attrs_to_update['PERMISSIONS'])
+                fail "failed to update image '#{new_resource.name}': #{rc.message}" if OpenNebula.is_error?(rc)
+              end
+            end
+          end
         else
           fail "'size' must be specified" unless new_resource.size
           fail "'datastore_id' must be specified" unless new_resource.datastore_id
@@ -81,7 +174,6 @@ class Chef
             @new_resource.updated_by_last_action(true)
           end
         when 'READY', 'USED', 'USED_PERS'
-          action_handler.report_progress "image '#{new_resource.name}' is already in #{@image.state_str} state - (up to date)"
         else
           fail "Image #{new_resource.name} is in unexpected state '#{@image.state_str}'"
         end

data/lib/chef/provisioning/opennebula_driver/driver.rb

@@ -208,7 +208,7 @@ class Chef
           return machine_spec unless instance.nil?
 
           unless machine_options[:bootstrap_options]
-            instance = retry_one("Retrying allocate_machine.instance_for (missing bootstrap options)") do
+            instance = @one.retry_one("Retrying allocate_machine.instance_for (missing bootstrap options)") do
               instance_for(machine_spec)
             end
             return machine_spec unless instance.nil?

data/lib/chef/provisioning/opennebula_driver/version.rb

@@ -24,7 +24,7 @@ class Chef
     # Extending module.
     #
     module OpenNebulaDriver
-      VERSION = '0.4.7'.freeze
+      VERSION = '0.4.8'.freeze
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chef-provisioning-opennebula
 version: !ruby/object:Gem::Version
-  version: 0.4.7
+  version: 0.4.8
 platform: ruby
 authors:
 - Andrew J. Brown
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-19 00:00:00.000000000 Z
+date: 2016-10-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef