fog-ovirt 1.2.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8782e80f74881843f3d5cfdf3537289c1f828190f9285905989d75f486252d13
-  data.tar.gz: b9088bc35338d04d9b6408ad91ff11db4623a998ca84346e2bce9c726b449e28
+  metadata.gz: '0018dbcebca7b9c47941e86933df6d71f9106c80348689ab8ff3e355c5bb550a'
+  data.tar.gz: 312123f1ff82516363de62ed8cc2617117f3a8a9bd11cb655244332bbcf3b6dc
 SHA512:
-  metadata.gz: b25cf3dc201edb0a1b53fadab1d10182d3a7de3ecd26fed18c1e3c19ccad9d2855465f6e8e4f551aa4fcb685058db544e9baf730a90934bb7968e2dda3b770ca
-  data.tar.gz: e2423a8441309249034df7311e1884265fc5a1c747e3d3f20cb3a182747b8626d3137dd73b23bf02c589af499377a05f318fe8616a242db9f5a31a9f30a5a0a7
+  metadata.gz: b3218482b5f4b274c684c906688e59112ea1a67d78a437c89502840ab8a6edecdf5e311072cf8470f88c8752f31527e53e479924290b6f0323a73f3579d7ff27
+  data.tar.gz: e5aa103e4a011613b64a86d2150a4a936d13e0f331bb287c404a912bcce618431d32ecb8d6ddb3c57b2b8c456840d89dd83fba1735663552eecbb27ed49cf77f

data/.rubocop.yml

@@ -21,3 +21,13 @@ Style/SignalException:
 
 Layout/EmptyLineAfterGuardClause:
   Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+AllCops:
+  Exclude:
+  - 'lib/fog/ovirt/compute.rb'

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2019-05-22 10:36:37 +0300 using RuboCop version 0.70.0.
+# on 2020-10-15 17:29:26 +0300 using RuboCop version 0.81.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -11,11 +11,18 @@
 Metrics/ClassLength:
   Max: 150
 
-# Offense count: 10
+# Offense count: 8
 # Configuration parameters: CountComments, ExcludedMethods.
 Metrics/MethodLength:
   Max: 26
 
+# Offense count: 1
+# Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts, AllowedAcronyms.
+# AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS
+Naming/FileName:
+  Exclude:
+    - 'Gemfile.local.rb'
+
 # Offense count: 1
 Style/ClassVars:
   Exclude:
@@ -26,7 +33,7 @@ Style/CommentedKeyword:
   Exclude:
     - 'lib/fog/ovirt/models/compute/server.rb'
 
-# Offense count: 188
+# Offense count: 98
 Style/Documentation:
   Enabled: false
 
@@ -36,13 +43,18 @@ Style/DoubleNegation:
     - 'lib/fog/ovirt/models/compute/server.rb'
     - 'tests/helpers/succeeds_helper.rb'
 
-# Offense count: 133
+# Offense count: 84
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.
-# SupportedStyles: always, never
+# SupportedStyles: always, always_true, never
 Style/FrozenStringLiteralComment:
   Enabled: false
 
+# Offense count: 3
+Style/MissingRespondToMissing:
+  Exclude:
+    - 'lib/fog/ovirt/compute.rb'
+
 # Offense count: 4
 Style/MultilineTernaryOperator:
   Exclude:

data/Gemfile

@@ -1,3 +1,2 @@
 source "https://rubygems.org"
-
 gemspec

data/README.md

@@ -40,9 +40,7 @@ compute = Fog::Compute.new(
 
 ## API Support
 
-This gem fully supports API v3 of oVirt.
-
-Version 1.0.2 adds support API V4 as well.
+This gem fully supports API V4 of oVirt.
 The first version does not give full support.
 The supported requests for API V4 are:
 
@@ -72,9 +70,7 @@ This example shows the usage as a fog provider:
         :ovirt_url        => url,
         :ovirt_datacenter => datacenter,
         :public_key       => public_key,
-        :api_version      => 'v4'
       )    
-The :api_version can be sent 'v3' or 'v4' to determine which API version to use. 
 
 
 Feedback is welcome. Please feel free to open issues for the V4 support and contribute.
@@ -85,4 +81,4 @@ Please refer to [CONTRIBUTING.md](CONTRIBUTING.md).
 
 ## License
 
-Please refer to [LICENSE.md](LICENSE.md).
+Please refer to [LICENSE.md](LICENSE.md).

data/Rakefile

@@ -9,14 +9,8 @@ require "bundler/gem_tasks"
 
 mock = ENV["FOG_MOCK"] || "true"
 
-begin
-  require "rubocop/rake_task"
-  RuboCop::RakeTask.new do |t|
-    t.patterns = Dir["{lib,spec,tests}/**/*"]
-  end
-rescue StandardError
-  puts "Rubocop not loaded"
-end
+require "rubocop/rake_task"
+RuboCop::RakeTask.new
 
 task :test do
   sh("export FOG_MOCK=#{mock} && bundle exec shindont tests/ovirt")

data/fog-ovirt.gemspec

@@ -17,13 +17,14 @@ Gem::Specification.new do |spec|
 
   spec.files         = `git ls-files -z`.split("\x0")
   spec.test_files    = spec.files.grep(%r{^tests\/})
-
   spec.require_paths = ["lib"]
+  spec.required_ruby_version = ">= 2.0.0"
 
+  spec.add_dependency("activesupport")
   spec.add_dependency("fog-core")
   spec.add_dependency("fog-json")
   spec.add_dependency("fog-xml")
-  spec.add_dependency("ovirt-engine-sdk", ">= 4.1.3")
+  spec.add_dependency("ovirt-engine-sdk", ">= 4.3.1")
   spec.add_dependency("rbovirt", "~> 0.1.5")
 
   spec.add_development_dependency "bundler"

data/lib/fog/ovirt/compute.rb

@@ -50,22 +50,13 @@ module Fog
         end
       end
 
-      require "fog/ovirt/compute/v3"
       require "fog/ovirt/compute/v4"
 
       def self.new(options = {})
         super(options)
-
-        # rubocop:disable Style/ConditionalAssignment
-        if options[:api_version] == "v4"
-          @client = Fog::Ovirt::Compute::V4.new(options)
-        else
-          @client = Fog::Ovirt::Compute::V3.new(options)
-        end
-        # rubocop:enable Style/ConditionalAssignment
+        @client = Fog::Ovirt::Compute::V4.new(options)
       end
 
-      # rubocop:disable Style/MethodMissingSuper, Style/MissingRespondToMissing
       def method_missing(symbol, *args)
         @client.__send__(symbol, *args)
       end
@@ -73,20 +64,13 @@ module Fog
       def respond_to?(symbol, include_all = false)
         @client.respond_to?(symbol, include_all)
       end
-      # rubocop:enable Style/MethodMissingSuper, Style/MissingRespondToMissing
 
       class Mock
         def initialize(options = {})
-          if options[:api_version] == "v4"
-            Fog::Ovirt::Compute::V4::Mock.send(:include, Fog::Ovirt::Compute::Collections)
-            @client = Fog::Ovirt::Compute::V4::Mock.new(options)
-          else
-            Fog::Ovirt::Compute::V3::Mock.send(:include, Fog::Ovirt::Compute::Collections)
-            @client = Fog::Ovirt::Compute::V3::Mock.new(options)
-          end
+          Fog::Ovirt::Compute::V4::Mock.include Fog::Ovirt::Compute::Collections
+          @client = Fog::Ovirt::Compute::V4::Mock.new(options)
         end
 
-        # rubocop:disable Style/MethodMissingSuper, Style/MissingRespondToMissing
         def method_missing(symbol, *args)
           @client.__send__(symbol, *args)
         end
@@ -94,21 +78,14 @@ module Fog
         def respond_to?(symbol, include_all = false)
           @client.respond_to?(symbol, include_all)
         end
-        # rubocop:enable Style/MethodMissingSuper, Style/MissingRespondToMissing
       end
 
       class Real
         def initialize(options = {})
-          if options[:api_version] == "v4"
-            Fog::Ovirt::Compute::V4::Real.send(:include, Fog::Ovirt::Compute::Collections)
-            @client = Fog::Ovirt::Compute::V4::Real.new(options)
-          else
-            Fog::Ovirt::Compute::V3::Real.send(:include, Fog::Ovirt::Compute::Collections)
-            @client = Fog::Ovirt::Compute::V3::Real.new(options)
-          end
+          Fog::Ovirt::Compute::V4::Real.include Fog::Ovirt::Compute::Collections
+          @client = Fog::Ovirt::Compute::V4::Real.new(options)
         end
 
-        # rubocop:disable Style/MethodMissingSuper, Style/MissingRespondToMissing
         def method_missing(symbol, *args)
           @client.send(symbol, *args)
         end
@@ -116,7 +93,6 @@ module Fog
         def respond_to?(symbol, include_all = false)
           @client.respond_to?(symbol, include_all)
         end
-        # rubocop:enable Style/MethodMissingSuper, Style/MissingRespondToMissing
       end
     end
   end

data/lib/fog/ovirt/compute/v4.rb

@@ -1,3 +1,6 @@
+require "active_support"
+require "active_support/core_ext"
+
 module Fog
   module Ovirt
     class Compute
@@ -33,6 +36,7 @@ module Fog
         request :list_template_volumes
         request :list_volumes
         request :add_volume
+        request :get_volume
         request :destroy_volume
         request :update_volume
         request :get_api_version
@@ -41,9 +45,9 @@ module Fog
         request :list_operating_systems
 
         module Shared
-          # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+          # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
           # converts an OVIRT object into an hash for fog to consume.
-          def ovirt_attrs(obj)
+          def shared_ovirt_attrs(obj)
             opts = {}
             # TODO: stop using instance_variables, they will change
             obj.instance_variables.each do |v|
@@ -51,7 +55,7 @@ module Fog
               value = obj.instance_variable_get(v)
 
               if key == :network
-                opts[key] = obj.vnic_profile.present? ? client.follow_link(obj.vnic_profile).network.id : value
+                opts[key] = obj.vnic_profile.present? ? yield(obj) : value
                 next
               end
 
@@ -69,9 +73,10 @@ module Fog
             end
             opts
           end
-          # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
 
-          # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+          # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+
+          # rubocop:disable Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
           def get_attr_value(value, opts)
             case value
             when OvirtSDK4::List
@@ -102,29 +107,31 @@ module Fog
             when OvirtSDK4::Console
               { :enabled => value.enabled }
             else
-              if value.class.respond_to?(:parent) && value.class.parent == OvirtSDK4
+              if value.class.respond_to?(:module_parent) && value.class.module_parent == OvirtSDK4
                 value.id if value.respond_to?(:id)
               else
                 value.to_s.strip
               end
             end
           end
-          # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
-
-          def convert_string_to_bool(opts)
-            return opts unless opts.present?
-            opts.each do |key, value|
-              if value == "true"
-                opts[key] = true
-              elsif value == "false"
-                opts[key] = false
-              elsif value.is_a? Hash
-                convert_string_to_bool(value)
-              elsif value.is_a? Array
-                value.map { |item| convert_string_to_bool(item) }
+          # rubocop:enable Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+
+          def convert_string_to_bool(value)
+            case value
+            when "true"
+              true
+            when "false"
+              false
+            when Array
+              value.map { |elem| convert_string_to_bool(elem) }
+            when Hash
+              value.each do |key, elem|
+                value[key] = convert_string_to_bool(elem)
               end
+              value
+            else
+              value
             end
-            opts
           end
         end
 
@@ -135,6 +142,13 @@ module Fog
             require "ovirtsdk4"
           end
 
+          def ovirt_attrs(obj)
+            shared_ovirt_attrs(obj) do
+              xml = read_xml("vnic_profile.xml")
+              Nokogiri::XML(xml).xpath("/vnic_profile/network/@id").to_s
+            end
+          end
+
           private
 
           def client
@@ -151,7 +165,6 @@ module Fog
         class Real
           include Shared
 
-          # rubocop:disable Metrics/AbcSize
           def initialize(options = {})
             require "ovirtsdk4"
             username   = options[:ovirt_username]
@@ -172,7 +185,12 @@ module Fog
 
             @client = ExceptionWrapper.new(OvirtSDK4::Connection.new(connection_opts))
           end
-          # rubocop:enable Metrics/AbcSize
+
+          def ovirt_attrs(obj)
+            shared_ovirt_attrs(obj) do
+              client.follow_link(obj.vnic_profile).network.id
+            end
+          end
 
           def api_version
             api = client.system_service.get
@@ -180,11 +198,11 @@ module Fog
           end
 
           def datacenter
-            @datacenter ||= datacenter_hash[:id]
+            datacenter_hash[:id]
           end
 
           def datacenter_hash
-            @datacenter_hash ||= datacenters.find { |x| x[:id] == @datacenter } if @datacenter
+            @datacenter_hash ||= datacenters.find { |x| x[:id] == @datacenter } || datacenters.find { |x| x[:name] == @datacenter } if @datacenter
             @datacenter_hash ||= datacenters.first
           end
 

data/lib/fog/ovirt/models/compute/instance_type.rb

@@ -10,6 +10,7 @@ module Fog
         attribute :description
         attribute :memory
         attribute :cores
+        attribute :sockets
         attribute :creation_time
         attribute :os
         attribute :ha

data/lib/fog/ovirt/models/compute/interfaces.rb

@@ -9,7 +9,6 @@ module Fog
 
         attr_accessor :vm
 
-        # rubocop:disable Metrics/AbcSize
         def all(_filters = {})
           requires :vm
           if vm.is_a? Fog::Ovirt::Compute::Server
@@ -20,7 +19,6 @@ module Fog
             raise ::Fog::Ovirt::Errors::OvirtError, "interfaces should have vm or template"
           end
         end
-        # rubocop:enable Metrics/AbcSize
 
         def get(id)
           new service.get_interface(id)

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

@@ -121,7 +121,6 @@ module Fog
           vm_power_action(:start)
         end
 
-        # rubocop:disable Metrics/AbcSize
         def start_with_cloudinit(options = {})
           wait_for { !locked? } if options[:blocking]
           user_data = if options[:use_custom_script]
@@ -133,7 +132,6 @@ module Fog
           reload
           action_status
         end
-        # rubocop:enable Metrics/AbcSize
 
         def stop(_options = {})
           vm_power_action(:stop)

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

@@ -17,6 +17,7 @@ module Fog
         attribute :quota
         attribute :alias
         attribute :wipe_after_delete
+        attribute :name
 
         def size_gb
           attributes[:size_gb] ||= attributes[:size].to_i / Fog::Ovirt::Compute::DISK_SIZE_TO_GB if attributes[:size]

data/lib/fog/ovirt/models/compute/volumes.rb

@@ -9,7 +9,6 @@ module Fog
 
         attr_accessor :vm
 
-        # rubocop:disable Metrics/AbcSize
         def all(_filters = {})
           if vm.is_a? Fog::Ovirt::Compute::Server
             load service.list_vm_volumes(vm.id)
@@ -19,7 +18,6 @@ module Fog
             load service.list_volumes
           end
         end
-        # rubocop:enable Metrics/AbcSize
 
         def get(id)
           new service.get_volume(id)

data/lib/fog/ovirt/requests/compute/v4/add_interface.rb

@@ -3,7 +3,6 @@ module Fog
     class Compute
       class V4
         class Real
-          # rubocop:disable Metrics/AbcSize
           def add_interface(id, options = {})
             raise ArgumentError, "instance id is a required parameter" unless id
             vm = client.system_service.vms_service.vm_service(id)
@@ -26,7 +25,6 @@ module Fog
             interface = OvirtSDK4::Nic.new(options)
             nics_service.add(interface)
           end
-          # rubocop:enable Metrics/AbcSize
         end
 
         class Mock