fog-xenserver 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
----
-SHA1:
-  metadata.gz: f0269325f7c966d090346e8338c3afe76376885c
-  data.tar.gz: 2adc377168002f1925552dc513c0d45f87c87f20
-SHA512:
-  metadata.gz: 9db549d9311be20e980a0e054e553b98ff1b5da43ef49101aa1ec443b00e54b6fddfb6a2a1db9f17d98c1b85839fb64f5cbdae447e78169ed93082694d548216
-  data.tar.gz: 8c343a5de1ac2d4e5faefa4f3620cda73624f235f2a46b1d5382ef8d3231a3a723acc5989fc6b41b025103fa56a8b8bbaa974c0025aeeccf543be0321d484a20
+--- 
+SHA512: 
+  data.tar.gz: d0ac34b63502b7b35a6bc805c2b2a820f9994a3a2e4a88d2e41b4042dd5bd53522abd4cb9f8c28ec74fcc5c3fbe1230bb8efa8f28c6ba279b11fb381f7221608
+  metadata.gz: 522acc12e5a3405b1d2194c04201c2c4de4d963a7f3c0f91415fedc0b73ba6530fb41e16425de27260be369fbf11407b071299eeed9f8e10a06f219ce4028996
+SHA1: 
+  data.tar.gz: e52e7c087cd6034f4ebaf9af941c21feb77ca749
+  metadata.gz: e0a54caf0e1052f809c836a58facebac1151ea51

data/.rubocop

@@ -1,26 +1,23 @@
-MethodCalledOnDoEndBlock:
-  Enabled: true
-
-HashSyntax:
+Metrics/LineLength:
   Enabled: false
 
-LineLength:
+Style/EachWithObject:
   Enabled: false
 
-SingleSpaceBeforeFirstArg:
-  Enabled: false
+Style/Encoding:
+  EnforcedStyle: when_needed
 
-SpaceInsideBrackets:
+Style/FormatString:
   Enabled: false
 
-StringLiterals:
+Style/HashSyntax:
   Enabled: false
 
-SymbolArray:
-  Enabled: true
+Style/SignalException:
+  EnforcedStyle: only_raise
 
-TrailingBlankLines:
-  Enabled: false
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
 
-TrailingWhitespace:
+Metrics/LineLength
   Enabled: false

data/lib/fog/bin/xenserver.rb

@@ -14,7 +14,7 @@ class XenServer < Fog::Bin
         hash[key] = case key
                       when :compute
                         Fog::Logger.warning("XenServer[:compute] is not recommended, use Compute[:xenserver] for portability")
-                        Fog::Compute.new(:provider => 'XenServer')
+                        Fog::Compute.new(:provider => "XenServer")
                       else
                         raise ArgumentError, "Unrecognized service: #{key.inspect}"
                     end

data/lib/fog/compute/xen_server.rb

@@ -1,11 +1,11 @@
-require 'fog/core'
+require "fog/core"
 
 module Fog
   module Compute
     class XenServer < Fog::Service
-      autoload :Models, 'fog/compute/xen_server/models'
-      autoload :Real, 'fog/compute/xen_server/real'
-      autoload :Mock, 'fog/compute/xen_server/mock'
+      autoload :Models, File.expand_path("../xen_server/models", __FILE__)
+      autoload :Real, File.expand_path("../xen_server/real", __FILE__)
+      autoload :Mock, File.expand_path("../xen_server/mock", __FILE__)
 
       requires :xenserver_username
       requires :xenserver_password
@@ -13,7 +13,7 @@ module Fog
       recognizes :xenserver_defaults
       recognizes :xenserver_timeout
 
-      model_path 'fog/compute/xen_server/models'
+      model_path "fog/compute/xen_server/models"
       model :blob
       collection :blobs
       model :bond
@@ -85,7 +85,7 @@ module Fog
       model :vtpm
       collection :vtpms
 
-      request_path 'fog/compute/xen_server/requests'
+      request_path "fog/compute/xen_server/requests"
 
       # host
       request :apply_edition_host
@@ -351,4 +351,4 @@ module Fog
       end
     end
   end
-end
+end

data/lib/fog/compute/xen_server/models.rb

@@ -2,80 +2,80 @@ module Fog
   module Compute
     class XenServer
       module Models
-        autoload :Blob, 'fog/compute/xen_server/models/blob'
-        autoload :Blobs, 'fog/compute/xen_server/models/blobs'
-        autoload :Bond, 'fog/compute/xen_server/models/bond'
-        autoload :Bonds, 'fog/compute/xen_server/models/bonds'
-        autoload :ClassMethods, 'fog/compute/xen_server/models/class_methods'
-        autoload :Collection, 'fog/compute/xen_server/models/collection'
-        autoload :Console, 'fog/compute/xen_server/models/console'
-        autoload :Consoles, 'fog/compute/xen_server/models/consoles'
-        autoload :CrashDump, 'fog/compute/xen_server/models/crash_dump'
-        autoload :CrashDumps, 'fog/compute/xen_server/models/crash_dumps'
-        autoload :DrTask, 'fog/compute/xen_server/models/dr_task'
-        autoload :DrTasks, 'fog/compute/xen_server/models/dr_tasks'
-        autoload :GpuGroup, 'fog/compute/xen_server/models/gpu_group'
-        autoload :GpuGroups, 'fog/compute/xen_server/models/gpu_groups'
-        autoload :GuestMetrics, 'fog/compute/xen_server/models/guest_metrics'
-        autoload :GuestsMetrics, 'fog/compute/xen_server/models/guests_metrics'
-        autoload :Host, 'fog/compute/xen_server/models/host'
-        autoload :HostCpu, 'fog/compute/xen_server/models/host_cpu'
-        autoload :HostCpus, 'fog/compute/xen_server/models/host_cpus'
-        autoload :HostCrashDump, 'fog/compute/xen_server/models/host_crash_dump'
-        autoload :HostCrashDumps, 'fog/compute/xen_server/models/host_crash_dumps'
-        autoload :HostMetrics, 'fog/compute/xen_server/models/host_metrics'
-        autoload :HostPatch, 'fog/compute/xen_server/models/host_patch'
-        autoload :HostPatchs, 'fog/compute/xen_server/models/host_patchs'
-        autoload :Hosts, 'fog/compute/xen_server/models/hosts'
-        autoload :HostsMetrics, 'fog/compute/xen_server/models/hosts_metrics'
-        autoload :InstanceMethods, 'fog/compute/xen_server/models/instance_methods'
-        autoload :Model, 'fog/compute/xen_server/models/model'
-        autoload :Network, 'fog/compute/xen_server/models/network'
-        autoload :Networks, 'fog/compute/xen_server/models/networks'
-        autoload :Pbd, 'fog/compute/xen_server/models/pbd'
-        autoload :Pbds, 'fog/compute/xen_server/models/pbds'
-        autoload :Pci, 'fog/compute/xen_server/models/pci'
-        autoload :Pcis, 'fog/compute/xen_server/models/pcis'
-        autoload :Pgpu, 'fog/compute/xen_server/models/pgpu'
-        autoload :Pgpus, 'fog/compute/xen_server/models/pgpus'
-        autoload :Pif, 'fog/compute/xen_server/models/pif'
-        autoload :PifMetrics, 'fog/compute/xen_server/models/pif_metrics'
-        autoload :Pifs, 'fog/compute/xen_server/models/pifs'
-        autoload :PifsMetrics, 'fog/compute/xen_server/models/pifs_metrics'
-        autoload :Pool, 'fog/compute/xen_server/models/pool'
-        autoload :PoolPatch, 'fog/compute/xen_server/models/pool_patch'
-        autoload :PoolPatchs, 'fog/compute/xen_server/models/pool_patchs'
-        autoload :Pools, 'fog/compute/xen_server/models/pools'
-        autoload :Role, 'fog/compute/xen_server/models/role'
-        autoload :Roles, 'fog/compute/xen_server/models/roles'
-        autoload :Server, 'fog/compute/xen_server/models/server'
-        autoload :ServerAppliance, 'fog/compute/xen_server/models/server_appliance'
-        autoload :ServerAppliances, 'fog/compute/xen_server/models/server_appliances'
-        autoload :ServerMetrics, 'fog/compute/xen_server/models/server_metrics'
-        autoload :ServersMetrics, 'fog/compute/xen_server/models/servers_metrics'
-        autoload :Servers, 'fog/compute/xen_server/models/servers'
-        autoload :StorageManager, 'fog/compute/xen_server/models/storage_manager'
-        autoload :StorageManagers, 'fog/compute/xen_server/models/storage_managers'
-        autoload :StorageRepositories, 'fog/compute/xen_server/models/storage_repositories'
-        autoload :StorageRepository, 'fog/compute/xen_server/models/storage_repository'
-        autoload :Tunnel, 'fog/compute/xen_server/models/tunnel'
-        autoload :Tunnels, 'fog/compute/xen_server/models/tunnels'
-        autoload :Vbd, 'fog/compute/xen_server/models/vbd'
-        autoload :VbdMetrics, 'fog/compute/xen_server/models/vbd_metrics'
-        autoload :Vbds, 'fog/compute/xen_server/models/vbds'
-        autoload :VbdsMetrics, 'fog/compute/xen_server/models/vbds_metrics'
-        autoload :Vdi, 'fog/compute/xen_server/models/vdi'
-        autoload :Vdis, 'fog/compute/xen_server/models/vdis'
-        autoload :Vif, 'fog/compute/xen_server/models/vif'
-        autoload :VifMetrics, 'fog/compute/xen_server/models/vif_metrics'
-        autoload :Vifs, 'fog/compute/xen_server/models/vifs'
-        autoload :VifsMetrics, 'fog/compute/xen_server/models/vifs_metrics'
-        autoload :Vlan, 'fog/compute/xen_server/models/vlan'
-        autoload :Vlans, 'fog/compute/xen_server/models/vlans'
-        autoload :Vmpp, 'fog/compute/xen_server/models/vmpp'
-        autoload :Vmpps, 'fog/compute/xen_server/models/vmpps'
-        autoload :Vtpm, 'fog/compute/xen_server/models/vtpm'
-        autoload :Vtpms, 'fog/compute/xen_server/models/vtpms'
+        autoload :Blob, File.expand_path("../models/blob", __FILE__)
+        autoload :Blobs, File.expand_path("../models/blobs", __FILE__)
+        autoload :Bond, File.expand_path("../models/bond", __FILE__)
+        autoload :Bonds, File.expand_path("../models/bonds", __FILE__)
+        autoload :ClassMethods, File.expand_path("../models/class_methods", __FILE__)
+        autoload :Collection, File.expand_path("../models/collection", __FILE__)
+        autoload :Console, File.expand_path("../models/console", __FILE__)
+        autoload :Consoles, File.expand_path("../models/consoles", __FILE__)
+        autoload :CrashDump, File.expand_path("../models/crash_dump", __FILE__)
+        autoload :CrashDumps, File.expand_path("../models/crash_dumps", __FILE__)
+        autoload :DrTask, File.expand_path("../models/dr_task", __FILE__)
+        autoload :DrTasks, File.expand_path("../models/dr_tasks", __FILE__)
+        autoload :GpuGroup, File.expand_path("../models/gpu_group", __FILE__)
+        autoload :GpuGroups, File.expand_path("../models/gpu_groups", __FILE__)
+        autoload :GuestMetrics, File.expand_path("../models/guest_metrics", __FILE__)
+        autoload :GuestsMetrics, File.expand_path("../models/guests_metrics", __FILE__)
+        autoload :Host, File.expand_path("../models/host", __FILE__)
+        autoload :HostCpu, File.expand_path("../models/host_cpu", __FILE__)
+        autoload :HostCpus, File.expand_path("../models/host_cpus", __FILE__)
+        autoload :HostCrashDump, File.expand_path("../models/host_crash_dump", __FILE__)
+        autoload :HostCrashDumps, File.expand_path("../models/host_crash_dumps", __FILE__)
+        autoload :HostMetrics, File.expand_path("../models/host_metrics", __FILE__)
+        autoload :HostPatch, File.expand_path("../models/host_patch", __FILE__)
+        autoload :HostPatchs, File.expand_path("../models/host_patchs", __FILE__)
+        autoload :Hosts, File.expand_path("../models/hosts", __FILE__)
+        autoload :HostsMetrics, File.expand_path("../models/hosts_metrics", __FILE__)
+        autoload :InstanceMethods, File.expand_path("../models/instance_methods", __FILE__)
+        autoload :Model, File.expand_path("../models/model", __FILE__)
+        autoload :Network, File.expand_path("../models/network", __FILE__)
+        autoload :Networks, File.expand_path("../models/networks", __FILE__)
+        autoload :Pbd, File.expand_path("../models/pbd", __FILE__)
+        autoload :Pbds, File.expand_path("../models/pbds", __FILE__)
+        autoload :Pci, File.expand_path("../models/pci", __FILE__)
+        autoload :Pcis, File.expand_path("../models/pcis", __FILE__)
+        autoload :Pgpu, File.expand_path("../models/pgpu", __FILE__)
+        autoload :Pgpus, File.expand_path("../models/pgpus", __FILE__)
+        autoload :Pif, File.expand_path("../models/pif", __FILE__)
+        autoload :PifMetrics, File.expand_path("../models/pif_metrics", __FILE__)
+        autoload :Pifs, File.expand_path("../models/pifs", __FILE__)
+        autoload :PifsMetrics, File.expand_path("../models/pifs_metrics", __FILE__)
+        autoload :Pool, File.expand_path("../models/pool", __FILE__)
+        autoload :PoolPatch, File.expand_path("../models/pool_patch", __FILE__)
+        autoload :PoolPatchs, File.expand_path("../models/pool_patchs", __FILE__)
+        autoload :Pools, File.expand_path("../models/pools", __FILE__)
+        autoload :Role, File.expand_path("../models/role", __FILE__)
+        autoload :Roles, File.expand_path("../models/roles", __FILE__)
+        autoload :Server, File.expand_path("../models/server", __FILE__)
+        autoload :ServerAppliance, File.expand_path("../models/server_appliance", __FILE__)
+        autoload :ServerAppliances, File.expand_path("../models/server_appliances", __FILE__)
+        autoload :ServerMetrics, File.expand_path("../models/server_metrics", __FILE__)
+        autoload :ServersMetrics, File.expand_path("../models/servers_metrics", __FILE__)
+        autoload :Servers, File.expand_path("../models/servers", __FILE__)
+        autoload :StorageManager, File.expand_path("../models/storage_manager", __FILE__)
+        autoload :StorageManagers, File.expand_path("../models/storage_managers", __FILE__)
+        autoload :StorageRepositories, File.expand_path("../models/storage_repositories", __FILE__)
+        autoload :StorageRepository, File.expand_path("../models/storage_repository", __FILE__)
+        autoload :Tunnel, File.expand_path("../models/tunnel", __FILE__)
+        autoload :Tunnels, File.expand_path("../models/tunnels", __FILE__)
+        autoload :Vbd, File.expand_path("../models/vbd", __FILE__)
+        autoload :VbdMetrics, File.expand_path("../models/vbd_metrics", __FILE__)
+        autoload :Vbds, File.expand_path("../models/vbds", __FILE__)
+        autoload :VbdsMetrics, File.expand_path("../models/vbds_metrics", __FILE__)
+        autoload :Vdi, File.expand_path("../models/vdi", __FILE__)
+        autoload :Vdis, File.expand_path("../models/vdis", __FILE__)
+        autoload :Vif, File.expand_path("../models/vif", __FILE__)
+        autoload :VifMetrics, File.expand_path("../models/vif_metrics", __FILE__)
+        autoload :Vifs, File.expand_path("../models/vifs", __FILE__)
+        autoload :VifsMetrics, File.expand_path("../models/vifs_metrics", __FILE__)
+        autoload :Vlan, File.expand_path("../models/vlan", __FILE__)
+        autoload :Vlans, File.expand_path("../models/vlans", __FILE__)
+        autoload :Vmpp, File.expand_path("../models/vmpp", __FILE__)
+        autoload :Vmpps, File.expand_path("../models/vmpps", __FILE__)
+        autoload :Vtpm, File.expand_path("../models/vtpm", __FILE__)
+        autoload :Vtpms, File.expand_path("../models/vtpms", __FILE__)
       end
     end
   end

data/lib/fog/compute/xen_server/models/instance_methods.rb

@@ -12,7 +12,7 @@ module Fog
           end
 
           def require_creation_attributes
-            requires *self.class.instance_variable_get('@require_before_save')
+            requires *self.class.instance_variable_get("@require_before_save")
           end
 
           def set_attribute(name, *val)

data/lib/fog/compute/xen_server/models/network.rb

@@ -16,9 +16,9 @@ module Fog
           attribute :bridge
           attribute :current_operations
           attribute :default_locking_mode
-          attribute :description,         :aliases => :name_description,  :default => '',  :as => :name_description
+          attribute :description,         :aliases => :name_description,  :default => "",  :as => :name_description
           attribute :mtu,                 :aliases => :MTU,                                :as => :MTU
-          attribute :name,                :aliases => :name_label,        :default => '',  :as => :name_label
+          attribute :name,                :aliases => :name_label,        :default => "",  :as => :name_label
           attribute :other_config,                                        :default => {}
           attribute :tags
           attribute :uuid

data/lib/fog/compute/xen_server/models/server.rb

@@ -1,4 +1,4 @@
-require 'fog/compute/models/server'
+require "fog/compute/models/server"
 
 module Fog
   module Compute
@@ -16,9 +16,9 @@ module Fog
 
           identity :reference
 
-          attribute :actions_after_crash,                                             :default => 'Restart'
-          attribute :actions_after_reboot,                                            :default => 'Restart'
-          attribute :actions_after_shutdown,                                          :default => 'Destroy'
+          attribute :actions_after_crash,                                             :default => "Restart"
+          attribute :actions_after_reboot,                                            :default => "Restart"
+          attribute :actions_after_shutdown,                                          :default => "Destroy"
           attribute :allowed_operations
           attribute :blobs
           attribute :blocked_operations
@@ -26,12 +26,12 @@ module Fog
           attribute :current_operations
           attribute :domarch
           attribute :domid
-          attribute :description,                 :aliases => :name_description,      :default => '',         :as => :name_description
+          attribute :description,                 :aliases => :name_description,      :default => "",         :as => :name_description
           attribute :generation_id
           attribute :ha_always_run
           attribute :ha_restart_priority
           attribute :hvm_boot_params,             :aliases => :HVM_boot_params,       :default => {},         :as => :HVM_boot_params
-          attribute :hvm_boot_policy,             :aliases => :HVM_boot_policy,       :default => '',         :as => :HVM_boot_policy
+          attribute :hvm_boot_policy,             :aliases => :HVM_boot_policy,       :default => "",         :as => :HVM_boot_policy
           attribute :hvm_shadow_multiplier,       :aliases => :HVM_shadow_multiplier,                         :as => :HVM_shadow_multiplier
           attribute :is_a_snapshot
           attribute :is_a_template,                                                   :default => true
@@ -39,29 +39,29 @@ module Fog
           attribute :is_snapshot_from_vmpp
           attribute :last_booted_record
           attribute :last_boot_cpu_flags,         :aliases => :last_boot_CPU_flags,                           :as => :last_boot_CPU_flags
-          attribute :memory_dynamic_max,                                              :default => '536870912'
-          attribute :memory_dynamic_min,                                              :default => '536870912'
+          attribute :memory_dynamic_max,                                              :default => "536870912"
+          attribute :memory_dynamic_min,                                              :default => "536870912"
           attribute :memory_overhead
-          attribute :memory_static_max,                                               :default => '536870912'
-          attribute :memory_static_min,                                               :default => '536870912'
+          attribute :memory_static_max,                                               :default => "536870912"
+          attribute :memory_static_min,                                               :default => "536870912"
           attribute :memory_target
-          attribute :name,                        :aliases => :name_label,            :default => '',         :as => :name_label
+          attribute :name,                        :aliases => :name_label,            :default => "",         :as => :name_label
           attribute :order
           attribute :other_config,                                                    :default => {}
-          attribute :pci_bus,                     :aliases => :PCI_bus,               :default => '',         :as => :PCI_bus
-          attribute :platform,                                                        :default => { 'nx' => 'true',
-                                                                                                    'acpi' => 'true',
-                                                                                                    'apic' => 'true',
-                                                                                                    'pae' => 'true',
-                                                                                                    'viridian' => 'true' }
+          attribute :pci_bus,                     :aliases => :PCI_bus,               :default => "",         :as => :PCI_bus
+          attribute :platform,                                                        :default => { "nx" => "true",
+                                                                                                    "acpi" => "true",
+                                                                                                    "apic" => "true",
+                                                                                                    "pae" => "true",
+                                                                                                    "viridian" => "true" }
           attribute :power_state
-          attribute :pv_args,                     :aliases => :PV_args,               :default => '-- quiet console=hvc0',  :as => :PV_args
-          attribute :pv_bootloader,               :aliases => :PV_bootloader,         :default => 'pygrub',                 :as => :PV_bootloader
-          attribute :pv_bootloader_args,          :aliases => :PV_bootloader_args,    :default => '',                       :as => :PV_bootloader_args
-          attribute :pv_kernel,                   :aliases => :PV_kernel,             :default => '',                       :as => :PV_kernel
-          attribute :pv_legacy_args,              :aliases => :PV_legacy_args,        :default => '',                       :as => :PV_legacy_args
-          attribute :pv_ramdisk,                  :aliases => :PV_ramdisk,            :default => '',                       :as => :PV_ramdisk
-          attribute :recommendations,                                                 :default => ''
+          attribute :pv_args,                     :aliases => :PV_args,               :default => "-- quiet console=hvc0",  :as => :PV_args
+          attribute :pv_bootloader,               :aliases => :PV_bootloader,         :default => "pygrub",                 :as => :PV_bootloader
+          attribute :pv_bootloader_args,          :aliases => :PV_bootloader_args,    :default => "",                       :as => :PV_bootloader_args
+          attribute :pv_kernel,                   :aliases => :PV_kernel,             :default => "",                       :as => :PV_kernel
+          attribute :pv_legacy_args,              :aliases => :PV_legacy_args,        :default => "",                       :as => :PV_legacy_args
+          attribute :pv_ramdisk,                  :aliases => :PV_ramdisk,            :default => "",                       :as => :PV_ramdisk
+          attribute :recommendations,                                                 :default => ""
           attribute :shutdown_delay
           attribute :snapshot_info
           attribute :snapshot_metadata
@@ -69,10 +69,10 @@ module Fog
           attribute :start_delay
           attribute :tags
           attribute :transportable_snapshot_id
-          attribute :user_version,                                                    :default => '0'
+          attribute :user_version,                                                    :default => "0"
           attribute :uuid
-          attribute :vcpus_at_startup,            :aliases => :VCPUs_at_startup,      :default => '1',    :as => :VCPUs_at_startup
-          attribute :vcpus_max,                   :aliases => :VCPUs_max,             :default => '1',    :as => :VCPUs_max
+          attribute :vcpus_at_startup,            :aliases => :VCPUs_at_startup,      :default => "1",    :as => :VCPUs_at_startup
+          attribute :vcpus_max,                   :aliases => :VCPUs_max,             :default => "1",    :as => :VCPUs_max
           attribute :vcpus_params,                :aliases => :VCPUs_params,          :default => {},     :as => :VCPUs_params
           attribute :version
           attribute :xenstore_data
@@ -113,12 +113,12 @@ module Fog
 
           def running?
             reload
-            power_state == 'Running'
+            power_state == "Running"
           end
 
           def halted?
             reload
-            power_state == 'Halted'
+            power_state == "Halted"
           end
 
           def destroy
@@ -149,11 +149,11 @@ module Fog
           end
 
           def can_be_cloned?
-            allowed_operations.include?('clone')
+            allowed_operations.include?("clone")
           end
 
           def clone(name)
-            raise 'Clone Operation not Allowed' unless can_be_cloned?
+            raise "Clone Operation not Allowed" unless can_be_cloned?
             self.reference = service.clone_vm(name, self.reference)
             reload
           end

data/lib/fog/compute/xen_server/models/servers.rb

@@ -1,4 +1,4 @@
-require 'fog/core/collection'
+require "fog/core/collection"
 
 module Fog
   module Compute
@@ -8,22 +8,22 @@ module Fog
           model Fog::Compute::XenServer::Models::Server
 
           def templates
-            Fog::Logger.deprecation 'This method is DEPRECATED. Call #templates directly on the connection instead.'
+            Fog::Logger.deprecation "This method is DEPRECATED. Call #templates directly on the connection instead."
             service.templates
           end
 
           def custom_templates
-            Fog::Logger.deprecation 'This method is DEPRECATED. Call #custom_templates directly on the connection instead.'
+            Fog::Logger.deprecation "This method is DEPRECATED. Call #custom_templates directly on the connection instead."
             service.custom_templates
           end
 
           def builtin_templates
-            Fog::Logger.deprecation 'This method is DEPRECATED. Call #builtin_templates directly on the connection instead.'
+            Fog::Logger.deprecation "This method is DEPRECATED. Call #builtin_templates directly on the connection instead."
             service.builtin_templates
           end
 
           def all(options = {})
-            data = service.get_records 'VM'
+            data = service.get_records "VM"
             # Exclude templates
             data.delete_if { |vm| vm[:is_control_domain] or vm[:is_a_template] }
             data.delete_if { |vm| vm[:is_a_snapshot] and !options[:include_snapshots] }

data/lib/fog/compute/xen_server/models/storage_repository.rb

@@ -13,14 +13,14 @@ module Fog
 
           attribute :allowed_operations
           attribute :blobs
-          attribute :content_type,                                          :default => 'user'
+          attribute :content_type,                                          :default => "user"
           attribute :current_operations
-          attribute :description,          :aliases => :name_description,   :default => ''
+          attribute :description,          :aliases => :name_description,   :default => ""
           attribute :introduced_by
           attribute :local_cache_enabled
           attribute :name,                 :aliases => :name_label
           attribute :other_config
-          attribute :physical_size,                                         :default => '0'
+          attribute :physical_size,                                         :default => "0"
           attribute :physical_utilisation
           attribute :shared,                                                :default => false
           attribute :sm_config,                                             :default => {}