rbovirt 0.0.9 → 0.0.10

data/VERSION

@@ -1 +1 @@
-0.0.9
+0.0.10

data/lib/client/storage_domain_api.rb

@@ -8,8 +8,10 @@ module OVIRT
     def storagedomains(opts={})
       search= opts[:search] ||''
       http_get("/storagedomains?search=%s" % CGI.escape(search)).xpath('/storage_domains/storage_domain').collect do |sd|
-        OVIRT::StorageDomain::new(self, sd)
-      end
+        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.
+        (opts[:role].nil? || storage_domain.role == opts[:role]) ? storage_domain : nil
+      end.compact
     end
   end
 end

data/lib/client/vm_api.rb

@@ -43,10 +43,14 @@ module OVIRT
     end
 
     def add_volume(vm_id, opts={})
-      storage_domain_id = opts[:storage_domain] || storagedomains.first.id
+      storage_domain_id = opts[:storage_domain] || storagedomains(:role => 'data').first.id
       http_post("/vms/%s/disks" % vm_id, OVIRT::Volume.to_xml(storage_domain_id, opts))
     end
 
+    def destroy_volume(vm_id, vol_id)
+      http_delete("/vms/%s/disks/%s" % [vm_id, vol_id])
+    end
+
     def vm_action(id, action, opts={})
       xml_response = http_post("/vms/%s/%s" % [id, action],'<action/>', opts)
       return (xml_response/'action/status').first.text.strip.upcase=="COMPLETE"

data/lib/ovirt/storage_domain.rb

@@ -1,6 +1,6 @@
 module OVIRT
 class StorageDomain < BaseObject
-    attr_reader :available, :used, :kind, :address, :path
+    attr_reader :available, :used, :kind, :address, :path, :role
 
     def initialize(client, xml)
       super(client, xml[:id], xml[:href], (xml/'name').first.text)
@@ -11,8 +11,9 @@ class StorageDomain < BaseObject
     private
 
     def parse_xml_attributes!(xml)
-      @available = (xml/'available').first.text
-      @used = (xml/'used').first.text
+      @available = ((xml/'available').first.text rescue nil)
+      @used = ((xml/'used').first.text rescue nil)
+      @role = (xml/'type').first.text
       @kind = (xml/'storage/type').first.text
       @address = ((xml/'storage/address').first.text rescue nil)
       @path = ((xml/'storage/path').first.text rescue nil)

data/rbovirt.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{rbovirt}
-  s.version = "0.0.9"
+  s.version = "0.0.10"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Amos Benari"]
-  s.date = %q{2012-03-21}
+  s.date = %q{2012-04-08}
   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: 13
+  hash: 11
   prerelease: false
   segments: 
   - 0
   - 0
-  - 9
-  version: 0.0.9
+  - 10
+  version: 0.0.10
 platform: ruby
 authors: 
 - Amos Benari
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-21 00:00:00 +02:00
+date: 2012-04-08 00:00:00 +03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency