ovirt-engine-sdk 4.0.12 → 4.1.0.alpha2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 47b9f2b19a80d7bbe6e9d6b8b6872221d338dbb7
-  data.tar.gz: 7aff41222648ebb5773f883a4e403d484968e509
+  metadata.gz: 01492a98ce30994bf24efc6a20606a0632a43a92
+  data.tar.gz: b1febb7296118d3039f7014e2311c89a9a5b9f52
 SHA512:
-  metadata.gz: 60d56c33e999b1bcc53ac3dc3d8585bcccedc22968084b47f00f979e5e968155657954c81e5f7bd76d98f62d9f790987156f9164822c8cb48c0a09605c45aae2
-  data.tar.gz: 8941c7e72a6073f32e7d650b03ce2e504a11ccccea94dbf869a3757f8a69d40ac8066e14bc45da8eff854a7a1705144926f34aa050594b765277e0f13fe82734
+  metadata.gz: 258f0e168ebf10d9a860e5d8b0f236dd151359d446f8eff7a8ad132815119e984586feee94919c9e89d68f0595376fbe9aa6d3296acaca50e3fb966a0481052b
+  data.tar.gz: 14622b76ff510f1b30ab42a40f5badbb8b8bd772f73bc992fbd4581d940c79804b7925067587fc39b6d45216941cfbb7b4de81989cc58fa4df4148ce77fb9a39

data/CHANGES.adoc

@@ -2,69 +2,86 @@
 
 This document describes the relevant changes between releases of the SDK.
 
-== 4.0.12 / Mar 3 2017
+== 4.1.0-alpha2 / Jan 12 2017
+
+Update to model 4.1.16:
+
+* Add `succeeded` parameter to the operation that end an external job.
+
+* Add `initial_size` attribute to the `Disk` type.
 
 Bug fixes:
 
-* Don't avoid reserved words in enum values text.
+* Fix writing 'id' attribute for enum values.
+  https://bugzilla.redhat.com/1408839[#1408839].
 
-* Red correctly attributes with enum values
-  https://bugzilla.redhat.com/1428642[#1428642].
+* Enable installation in Vagrant with embedded Ruby
+  https://bugzilla.redhat.com/1411594[#1411594].
 
-== 4.0.11 / Mar 2 2017
+== 4.1.0-alpha1 / Dec 19 2016
 
-New features:
+Update to model 4.1.24:
 
-* Add support for custom headers and query parameters.
+* Added new `v4` value to the `StorageFormat` enum.
 
-* Ignore unsupported enum values.
+* Add `seal` operation to the service that manages templates.
 
-* Check that URL has been provided.
+* Add `progress` attribute to the `Disk` and `Step` types.
 
-* Send INFO log messages with summaries of requests and responses.
+* Add `allow_partial_import` parameter to the operations that import
+  virtual machines and templates.
 
-Bug fixes:
+* Add `ticket` operation to the service that manages the graphics
+  console of a virtual machine.
 
-* Fix writing of lists of structs.
+* Added `urandom` to the `RngSource` enumerated type.
 
-== 4.0.10 / Feb 9 2017
+* Added `migratable` flag to the `VnicProfile` type.
 
-Bug fixes:
+* Add `gluster_tuned_profile` to the `Cluster` type.
 
-* Use `curl-config` to detect `libcurl`, if possible.
+* Add `skip_if_gluster_bricks_up` and `skip_if_gluster_quorum_not_met`
+  attributes to the `FencingPolicy` type.
 
-== 4.0.9 / Feb 1 2017
+* Add the `ImageTransferDirection` enumerated type and the `direction`
+  attribute to the `ImageTransfer` type.
 
-Bug fixes:
+* Replace the generic `GraphicsConsole` and `GraphicsConsoles` with
+  specific services for virtual machines, templates and instance
+  types.
 
-* Don't require `json` 1.8, as it makes the SDK incompatible with
-  applications that need version 2.0 or newer.
+* Deprecate the `legacy` USB type.
 
-== 4.0.8 / Jan 27 2017
+* Add `remoteviewerconnectionfile` action to the `GraphicsConsole`
+  service.
 
-Update to model 4.0.41:
+* Add `max` attribute to the `MemoryPolicy` type.
 
-* Add `initial_size` attribute to the `Disk` type.
+* Make `Ip.version` optional.
 
-New features:
+* Add the `active_slave` link to the `Bonding` type.
 
-* Accept CA files and strings.
+* Add DNS configuration support to `Network` and `NetworkAttachment`.
 
-== 4.0.7 / Jan 12 2017
+* Add the `uses_scsi_reservation` attribute to the `DiskAttachment`
+  type.
 
-New features:
+* Add the `sparsify` method to `DiskService`.
 
-* Add HTTP proxy support.
+* Add the `discard_max_size` and `discard_zeroes_data` to the `LogicalUnit`
+  type.
 
-* Require Ruby 2.
+* Add `logical_name` attribute to the disk attachment type.
 
-Bug fixes:
+* Add the `upgradecheck` action to the host service.
 
-* Fix writing `id` attribute for enum values
-  https://bugzilla.redhat.com/1408839[#1408839].
+* Add the `original_template` link to the virtual machine type.
 
-* Enable installation in Vagrant with embedded Ruby
-  https://bugzilla.redhat.com/1411594[#1411594].
+New features:
+
+* Require Ruby 2.
+
+* Enable compression by default https://bugzilla.redhat.com/1370464[#1370464].
 
 == 4.0.6 / Oct 18 2016
 

data/ext/ovirtsdk4c/extconf.rb

@@ -1,7 +1,7 @@
 #!/usr/bin/ruby
 
 #
-# Copyright (c) 2015-2017 Red Hat, Inc.
+# Copyright (c) 2015-2016 Red Hat, Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -18,25 +18,15 @@
 
 require 'mkmf'
 
-# Check if "libxml2" is available:
-xml2_config = find_executable('xml2-config')
-if xml2_config
-  cflags = `#{xml2_config} --cflags`.strip
-  libs = `#{xml2_config} --libs`.strip
-  $CPPFLAGS = "#{cflags} #{$CPPFLAGS}"
-  $LDFLAGS = "#{libs} #{$LDFLAGS}"
-elsif !pkg_config('libxml2')
+# Check that "libxml2" is available:
+unless find_executable('xml2-config')
   raise 'The "libxml2" package isn\'t available.'
 end
+$CPPFLAGS = "#{`xml2-config --cflags`.strip} #{$CPPFLAGS}"
+$LDFLAGS = "#{`xml2-config --libs`.strip} #{$LDFLAGS}"
 
-# Check if "libcurl" is available:
-curl_config = find_executable('curl-config')
-if curl_config
-  cflags = `#{curl_config} --cflags`.strip
-  libs = `#{curl_config} --libs`.strip
-  $CPPFLAGS = "#{cflags} #{$CPPFLAGS}"
-  $LDFLAGS = "#{libs} #{$LDFLAGS}"
-elsif !pkg_config('libcurl')
+# Check that "libcurl" is available:
+unless pkg_config('libcurl')
   raise 'The "libcurl" package isn\'t available.'
 end
 

data/ext/ovirtsdk4c/ov_http_client.c

@@ -19,7 +19,6 @@ limitations under the License.
 
 #include <ctype.h>
 #include <curl/curl.h>
-#include <stdarg.h>
 #include <stdbool.h>
 #include <stdlib.h>
 #include <string.h>
@@ -49,8 +48,6 @@ static VALUE PROXY_PASSWORD_SYMBOL;
 /* Method identifiers: */
 static ID DEBUG_ID;
 static ID ENCODE_WWW_FORM_ID;
-static ID INFO_ID;
-static ID INFO_Q_ID;
 static ID READ_ID;
 static ID STRING_ID;
 static ID STRING_IO_ID;
@@ -552,22 +549,6 @@ static void ov_http_client_perform_cancel(void* data) {
     perform_context->cancel = true;
 }
 
-static void ov_http_client_log_info(VALUE log, const char* format, ...) {
-    VALUE enabled;
-    VALUE message;
-    va_list args;
-
-    if (!NIL_P(log)) {
-        enabled = rb_funcall(log, INFO_Q_ID, 0);
-        if (RTEST(enabled)) {
-            va_start(args, format);
-            message = rb_vsprintf(format, args);
-            rb_funcall(log, INFO_ID, 1, message);
-            va_end(args);
-        }
-    }
-}
-
 static VALUE ov_http_client_send(VALUE self, VALUE request, VALUE response) {
     VALUE header;
     VALUE url;
@@ -645,14 +626,6 @@ static VALUE ov_http_client_send(VALUE self, VALUE request, VALUE response) {
     }
     curl_easy_setopt(object->curl, CURLOPT_HTTPHEADER, headers);
 
-    /* Send a summary of the request to the log: */
-    ov_http_client_log_info(
-        object->log,
-        "Sending '%"PRIsVALUE"' request to URL '%"PRIsVALUE"'.",
-        request_object->method,
-        url
-    );
-
     /* Performing the request is a potentially lengthy and blocking operation, so we need to make sure that it runs
        without the global interpreter lock acquired as much as possible: */
     perform_context.object = object;
@@ -701,13 +674,6 @@ static VALUE ov_http_client_send(VALUE self, VALUE request, VALUE response) {
     /* Get the response body: */
     response_object->body = rb_funcall(perform_context.out, STRING_ID, 0);
 
-    /* Send a summary of the response to the log: */
-    ov_http_client_log_info(
-        object->log,
-        "Received response code '%"PRIsVALUE"'.",
-        response_object->code
-    );
-
     return Qnil;
 }
 
@@ -746,8 +712,6 @@ void ov_http_client_define(void) {
     /* Define the method identifiers: */
     DEBUG_ID           = rb_intern("debug");
     ENCODE_WWW_FORM_ID = rb_intern("encode_www_form");
-    INFO_ID            = rb_intern("info");
-    INFO_Q_ID          = rb_intern("info?");
     READ_ID            = rb_intern("read");
     STRING_ID          = rb_intern("string");
     STRING_IO_ID       = rb_intern("StringIO");

data/lib/ovirtsdk4/connection.rb

@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2015-2017 Red Hat, Inc.
+# Copyright (c) 2015-2016 Red Hat, Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -15,7 +15,6 @@
 #
 
 require 'json'
-require 'tempfile'
 require 'uri'
 
 module OvirtSDK4
@@ -55,14 +54,8 @@ module OvirtSDK4
     #   name should be checked.
     #
     # @option opts [String] :ca_file The name of a PEM file containing the trusted CA certificates. The certificate
-    #   presented by the server will be verified using these CA certificates. If neither this nor the `ca_certs`
-    #   options are provided, then the system wide CA certificates store is used. If both options are provided,
-    #   then the certificates from both options will be trusted.
-    #
-    # @option opts [Array<String>] :ca_certs An array of strings containing the trusted CA certificates, in PEM
-    #   format. The certificate presented by the server will be verified using these CA certificates. If neither this
-    #   nor the `ca_file` options are provided, then the system wide CA certificates store is used. If both options
-    #   are provided, then the certificates from both options will be trusted.
+    #   presented by the server will be verified using these CA certificates. If not set then the system wide CA
+    #   certificates store is used.
     #
     # @option opts [Boolean] :debug (false) A boolean flag indicating if debug output should be generated. If the
     #   values is `true` and the `log` parameter isn't `nil` then the data sent to and received from the server will be
@@ -77,9 +70,11 @@ module OvirtSDK4
     #   (the default) means wait for ever. If the timeout expires before the response is received an exception will be
     #   raised.
     #
-    # @option opts [Boolean] :compress (false) A boolean flag indicating if the SDK should ask the server to send
+    # @option opts [Boolean] :compress (true) A boolean flag indicating if the SDK should ask the server to send
     #   compressed responses. Note that this is a hint for the server, and that it may return uncompressed data even
-    #   when this parameter is set to `true`.
+    #   when this parameter is set to `true`. Also, compression will be automatically disabled when the `debug`
+    #   parameter is set to `true`, as otherwise the debug output will be compressed as well, and then it isn't
+    #   useful.
     #
     # @option opts [String] :proxy_url A string containing the protocol, address and port number of the proxy server
     #   to use to connect to the server. For example, in order to use the HTTP proxy `proxy.example.com` that is
@@ -90,11 +85,6 @@ module OvirtSDK4
     #
     # @option opts [String] :proxy_password The password of the user to authenticate to the proxy server.
     #
-    # @option opts [Hash] :headers Custom HTTP headers to send with all requests. The keys of the hash can be
-    #   strings of symbols, and they will be used as the names of the headers. The values of the hash will be used
-    #   as the names of the headers. If the same header is provided here and in the `headers` parameter of a specific
-    #   method call, then the `headers` parameter of the specific method call will have precendence.
-    #
     def initialize(opts = {})
       # Get the values of the parameters and assign default values:
       @url = opts[:url]
@@ -103,38 +93,23 @@ module OvirtSDK4
       @token = opts[:token]
       @insecure = opts[:insecure] || false
       @ca_file = opts[:ca_file]
-      @ca_certs = opts[:ca_certs]
       @debug = opts[:debug] || false
       @log = opts[:log]
       @kerberos = opts[:kerberos] || false
       @timeout = opts[:timeout] || 0
-      @compress = opts[:compress] || false
+      @compress = opts[:compress] || true
       @proxy_url = opts[:proxy_url]
       @proxy_username = opts[:proxy_username]
       @proxy_password = opts[:proxy_password]
-      @headers = opts[:headers]
-
-      # Check that the URL has been provided:
-      raise ArgumentError, "The 'url' option is mandatory" unless @url
-
-      # Create a temporary file to store the CA certificates, and populate it with the contents of the 'ca_file' and
-      # 'ca_certs' options. The file will be removed when the connection is closed.
-      @ca_store = nil
-      if @ca_file || @ca_certs
-        @ca_store = Tempfile.new('ca_store')
-        @ca_store.write(::File.read(@ca_file)) if @ca_file
-        if @ca_certs
-          @ca_certs.each do |ca_cert|
-            @ca_store.write(ca_cert)
-          end
-        end
-        @ca_store.close
-      end
+
+      # Automatically disable compression when debug is enabled, as otherwise the debug output generated by
+      # libcurl is also compressed, and that isn't useful for debugging:
+      @compress = false if @debug
 
       # Create the HTTP client:
       @client = HttpClient.new(
         insecure: @insecure,
-        ca_file: @ca_store ? @ca_store.path : nil,
+        ca_file: @ca_file,
         debug: @debug,
         log: @log,
         timeout: @timeout,
@@ -195,9 +170,6 @@ module OvirtSDK4
         request.headers['All-Content'] = all_content unless all_content.nil?
       end
 
-      # Add the global headers, but without replacing the values that may already exist:
-      request.headers.merge!(@headers) { |_name, local, _global| local } if @headers
-
       # Set the authentication token:
       @token ||= create_access_token
       request.token = @token
@@ -278,9 +250,6 @@ module OvirtSDK4
         body: URI.encode_www_form(parameters)
       )
 
-      # Add the global headers:
-      request.headers.merge!(@headers) if @headers
-
       # Create an empty response:
       response = HttpResponse.new
 
@@ -436,9 +405,6 @@ module OvirtSDK4
 
       # Close the HTTP client:
       @client.close if @client
-
-      # Remove the temporary file that contains the trusted CA certificates:
-      @ca_store.unlink if @ca_store
     end
   end
 end

data/lib/ovirtsdk4/reader.rb

@@ -121,7 +121,7 @@ module OvirtSDK4
     #
     # Converts the given text to a decimal value.
     #
-    # @return [Float]
+    # @return [Fixnum]
     #
     def self.parse_decimal(text)
       return nil if text.nil?
@@ -136,7 +136,7 @@ module OvirtSDK4
     # Reads a decimal value, assuming that the cursor is positioned at the start element that contains the value.
     #
     # @param reader [XmlReader]
-    # @return [Float]
+    # @return [Fixnum]
     #
     def self.read_decimal(reader)
       Reader.parse_decimal(reader.read_element)
@@ -147,7 +147,7 @@ module OvirtSDK4
     # values.
     #
     # @param reader [XmlReader]
-    # @return [Array<Float>]
+    # @return [Array<Fixnum>]
     #
     def self.read_decimals(reader)
       reader.read_elements.map { |text| Reader.parse_decimal(text) }
@@ -189,43 +189,6 @@ module OvirtSDK4
       reader.read_elements.map { |text| Reader.parse_date(text) }
     end
 
-    #
-    # Converts the given text to an enum.
-    #
-    # @param enum_module [Module]
-    # @param text [String]
-    # @return [String]
-    #
-    def self.parse_enum(enum_module, text)
-      return nil unless text
-      values = enum_module.constants.map { |const| enum_module.const_get(const) }
-      values.detect { |value| value.casecmp(text).zero? }
-    end
-
-    #
-    # Reads a enum value, assuming that the cursor is positioned at the
-    # start element that contains the value.
-    #
-    # @param enum_module [Module]
-    # @param reader [XmlReader]
-    # @return [Array<String>]
-    #
-    def self.read_enum(enum_module, reader)
-      Reader.parse_enum(enum_module, reader.read_element)
-    end
-
-    #
-    # Reads a list of enum values, assuming that the cursor is positioned
-    # at the start element of the element that contains the first value.
-    #
-    # @param enum_module [Module]
-    # @param reader [XmlReader]
-    # @return [Array<String>]
-    #
-    def self.read_enums(enum_module, reader)
-      reader.read_elements.map { |text| Reader.parse_enum(enum_module, text) }
-    end
-
     #
     # This hash stores for each known tag a reference to the method that read the object corresponding for that tag. For
     # example, for the `vm` tag it will contain a reference to the `VmReader.read_one` method, and for the `vms` tag