rbovirt 0.0.10 → 0.0.11

data/VERSION

@@ -1 +1 @@
-0.0.10
+0.0.11

data/lib/client/storage_domain_api.rb

@@ -6,10 +6,10 @@ module OVIRT
     end
 
     def storagedomains(opts={})
-      search= opts[:search] ||''
+      search= opts[:search] || ("datacenter=%s" % current_datacenter.name)
       http_get("/storagedomains?search=%s" % CGI.escape(search)).xpath('/storage_domains/storage_domain').collect do |sd|
         storage_domain = OVIRT::StorageDomain::new(self, sd)
-        #filter by role is not supported by the search language. The work around is to list all then filter.
+        #filter by role is not supported by the search language. The work around is to list all, then filter.
         (opts[:role].nil? || storage_domain.role == opts[:role]) ? storage_domain : nil
       end.compact
     end

data/lib/client/vm_api.rb

@@ -37,13 +37,16 @@ module OVIRT
     end
 
     def vm_volumes vm_id
-      http_get("/vms/%s/disks" % vm_id, http_headers).xpath('/disks/disk').collect do |disk|
+      volumes = http_get("/vms/%s/disks" % vm_id, http_headers).xpath('/disks/disk').collect do |disk|
         OVIRT::Volume::new(self, disk)
       end
+      #this is a workaround to a bug that the list is not sorted by default.
+      volumes.sort{ |l, r| l.name <=> r.name }
     end
 
     def add_volume(vm_id, opts={})
-      storage_domain_id = opts[:storage_domain] || storagedomains(:role => 'data').first.id
+      search = opts[:search] || ("datacenter=%s" % current_datacenter.name)
+      storage_domain_id = opts[:storage_domain] || storagedomains(:role => 'data', :search => search).first.id
       http_post("/vms/%s/disks" % vm_id, OVIRT::Volume.to_xml(storage_domain_id, opts))
     end
 

data/lib/ovirt/volume.rb

@@ -16,7 +16,7 @@ module OVIRT
             storage_domain_(:id => storage_domain_id)
           }
           size_(opts[:size] || 8589934592)
-          type_(opts[:type] || 'system')
+          type_(opts[:type] || 'data')
           bootable_(opts[:bootable] || 'true')
           interface_(opts[:interface] || 'virtio')
           format_(opts[:format] || 'cow')
@@ -29,7 +29,7 @@ module OVIRT
     def parse_xml_attributes!(xml)
      @storage_domain = (xml/'storage_domains/storage_domain').first[:id]
      @size = (xml/'size').first.text
-     @disk_type = (xml/'type').first.text
+     @disk_type = ((xml/'type').first.text rescue nil)
      @bootable = (xml/'bootable').first.text
      @interface = (xml/'interface').first.text
      @format = (xml/'format').first.text

data/rbovirt.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{rbovirt}
-  s.version = "0.0.10"
+  s.version = "0.0.11"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Amos Benari"]
-  s.date = %q{2012-04-08}
+  s.date = %q{2012-04-16}
   s.description = %q{A Ruby client for oVirt REST API}
   s.email = %q{abenari@redhat.com}
   s.extra_rdoc_files = [

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rbovirt
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 9
   prerelease: false
   segments: 
   - 0
   - 0
-  - 10
-  version: 0.0.10
+  - 11
+  version: 0.0.11
 platform: ruby
 authors: 
 - Amos Benari
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-04-08 00:00:00 +03:00
+date: 2012-04-16 00:00:00 +03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency