Merge pull request fog#855 from abenari/master

[oVirt] volume size in GB accessor

fog.gemspec

@@ -54,7 +54,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency('rdoc')
   s.add_development_dependency('thor')
   s.add_development_dependency('rspec', '~>1.3.1')
-  s.add_development_dependency('rbovirt', '>=0.0.10')
+  s.add_development_dependency('rbovirt', '>=0.0.11')
   s.add_development_dependency('shindo', '~>0.3.4')
   s.add_development_dependency('virtualbox', '~>0.9.1')
   s.add_development_dependency('fission')

lib/fog/ovirt/models/compute/volume.rb

@@ -4,6 +4,7 @@ class Ovirt
 
       class Volume < Fog::Model
         attr_accessor :raw
+        DISK_SIZE_TO_GB = 1073741824
         identity :id
 
         attribute :storage_domain
@@ -13,6 +14,15 @@ class Volume < Fog::Model
         attribute :interface
         attribute :format
         attribute :sparse
+        attribute :size_gb
+
+        def size_gb
+          attributes[:size_gb] ||= attributes[:size].to_i / DISK_SIZE_TO_GB if attributes[:size]
+        end
+
+        def size_gb= s
+          attributes[:size] = s.to_i * DISK_SIZE_TO_GB if s
+        end
 
         def to_s
           id

lib/fog/ovirt/requests/compute/add_volume.rb

@@ -2,10 +2,10 @@ module Fog
   module Compute
     class Ovirt
       class Real
-
+        DISK_SIZE_TO_GB = 1073741824
         def add_volume(id, options = {})
           raise ArgumentError, "instance id is a required parameter" unless id
-
+          options[:size]=options[:size_gb].to_i*DISK_SIZE_TO_GB if options[:size_gb]
           client.add_volume(id, options)
         end