ovirt-engine-sdk 4.2.0.alpha3 → 4.2.0.alpha4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 80eb53f5a009746e0161ca6038d089370c372df8
-  data.tar.gz: 6285abc6584ca83aec8df2be1dfabca6d2485afa
+  metadata.gz: 230614a52420a3f291acff2868274b00220cfe6a
+  data.tar.gz: 4480ebcee091051f2acaf8dbca74e9d3903e4196
 SHA512:
-  metadata.gz: 123a67164cf851403136b4af9362d8c3c73826b30a6573bae326827929c4f9df5f46642a02a8f10a16634ae1782f77f3e202ed10a9d2f3069b5d6fbcf95f4616
-  data.tar.gz: 9ca5ab17eba09f5a7f1ae1e73d66008a9ccfc76d0f77f589c6cbaa12c5a9f9e0e072f83ded518d68111c6db63fd7d3e4cc5cbeecb4defad008dc08318977504e
+  metadata.gz: ae02f9d1e89464b19f72438d726f4d30a416fe30d18db16fc3f4fec2f08230f7ba3e64326d2e17a638b23de0ce0405725121793c3736385d30b34b59e2ebb5ea
+  data.tar.gz: d3e7722a5ecba2632d1f8259f43d61691463e63be959c8f66bb1289f4a61cf3dc1a774aa0dc2a0df13bd8376a37bb01ae7210860027e9f0d769cd6eff412cf40

data/CHANGES.adoc

@@ -2,6 +2,51 @@
 
 This document describes the relevant changes between releases of the SDK.
 
+== 4.2.0-alpha4 / Oct 23 2017
+
+Update to model 4.2.23:
+
+* Add new services and types to enable automatic provisioning of
+  external network providers during host installation.
+
+* Explicitly indicate that CPU type is mandtory when adding a cluster.
+
+* Add `image` parameter to `HostService.upgrade`
+  https://bugzilla.redhat.com/1488434[#1488434].
+
+* Add `storageErrorResumeBehaviour` to virtual machines, templates and
+  instance types https://bugzilla.redhat.com/1317450[#1317450].
+
+* Don't require name or identifier for adding SSH public key
+  https://bugzilla.redhat.com/1497641[#1497641].
+
+* Add `reboot` parameter to the `upgrade` method of the service that
+  manages a host.
+
+* Add `registrationConfiguration` parameter to the operations that
+  register virtual machines and templates.
+
+* Add documentation explaining how to add snapshots with disk
+  attachments.
+
+* Add documentation explaining how to add a template with disks in
+  specific storage domains https://bugzilla.redhat.com/1492614[#1492614].
+
+* Add `vnicProfileMappings` parameter to the operation that registers a
+  template.
+
+New features:
+
+* Check types of service method parameters
+  https://bugzilla.redhat.com/1378113[#1378113].
+
+* Add support for multiple threads
+  https://bugzilla.redhat.com/1496846[#1496846].
+
+Bug fixes:
+
+* Check types of HTTP client constructor parameters.
+
 == 4.2.0-alpha3 / Sep 25 2017
 
 Update to model 4.2.19:

data/ext/ovirtsdk4c/ov_http_client.c

@@ -50,7 +50,7 @@ static VALUE PROXY_PASSWORD_SYMBOL;
 static VALUE PROXY_URL_SYMBOL;
 static VALUE PROXY_USERNAME_SYMBOL;
 static VALUE TIMEOUT_SYMBOL;
-static VALUE USERNAME_SYMBOL;
+static VALUE COOKIES_SYMBOL;
 
 /* Method identifiers: */
 static ID COMPARE_BY_IDENTITY_ID;
@@ -154,6 +154,7 @@ static void ov_http_client_free(void* vptr) {
     /* Release the resources used by libcurl: */
     if (ptr->handle != NULL) {
         curl_multi_cleanup(ptr->handle);
+        curl_share_cleanup(ptr->share);
         ptr->handle = NULL;
     }
 
@@ -162,6 +163,7 @@ static void ov_http_client_free(void* vptr) {
     ov_string_free(ptr->proxy_url);
     ov_string_free(ptr->proxy_username);
     ov_string_free(ptr->proxy_password);
+    ov_string_free(ptr->cookies);
 
     /* Free this object: */
     xfree(ptr);
@@ -186,6 +188,20 @@ static VALUE ov_http_client_alloc(VALUE klass) {
     ov_http_client_object* ptr;
 
     ptr = ALLOC(ov_http_client_object);
+    ptr->handle = NULL;
+    ptr->share = NULL;
+    ptr->log = Qnil;
+    ptr->pending = Qnil;
+    ptr->completed = Qnil;
+    ptr->compress = false;
+    ptr->debug = false;
+    ptr->insecure = false;
+    ptr->ca_file = NULL;
+    ptr->proxy_url = NULL;
+    ptr->proxy_username = NULL;
+    ptr->proxy_password = NULL;
+    ptr->timeout = 0;
+    ptr->cookies = NULL;
     return TypedData_Wrap_Struct(klass, &ov_http_client_type, ptr);
 }
 
@@ -466,7 +482,13 @@ static VALUE ov_http_client_initialize(int argc, VALUE* argv, VALUE self) {
 
     /* Get the value of the 'ca_file' parameter: */
     opt = rb_hash_aref(opts, CA_FILE_SYMBOL);
-    ptr->ca_file = ov_string_dup(opt);
+    if (NIL_P(opt)) {
+        ptr->ca_file = NULL;
+    }
+    else {
+        Check_Type(opt, T_STRING);
+        ptr->ca_file = ov_string_dup(opt);
+    }
 
     /* Get the value of the 'insecure' parameter: */
     opt = rb_hash_aref(opts, INSECURE_SYMBOL);
@@ -492,15 +514,33 @@ static VALUE ov_http_client_initialize(int argc, VALUE* argv, VALUE self) {
 
     /* Get the value of the 'proxy_url' parameter: */
     opt = rb_hash_aref(opts, PROXY_URL_SYMBOL);
-    ptr->proxy_url = ov_string_dup(opt);
+    if (NIL_P(opt)) {
+        ptr->proxy_url = NULL;
+    }
+    else {
+        Check_Type(opt, T_STRING);
+        ptr->proxy_url = ov_string_dup(opt);
+    }
 
     /* Get the value of the 'proxy_username' parameter: */
     opt = rb_hash_aref(opts, PROXY_USERNAME_SYMBOL);
-    ptr->proxy_username = ov_string_dup(opt);
+    if (NIL_P(opt)) {
+        ptr->proxy_username = NULL;
+    }
+    else {
+        Check_Type(opt, T_STRING);
+        ptr->proxy_username = ov_string_dup(opt);
+    }
 
     /* Get the value of the 'proxy_password' parameter: */
     opt = rb_hash_aref(opts, PROXY_PASSWORD_SYMBOL);
-    ptr->proxy_password = ov_string_dup(opt);
+    if (NIL_P(opt)) {
+        ptr->proxy_password = NULL;
+    }
+    else {
+        Check_Type(opt, T_STRING);
+        ptr->proxy_password = ov_string_dup(opt);
+    }
 
     /* Get the value of the 'log' parameter: */
     opt = rb_hash_aref(opts, LOG_SYMBOL);
@@ -526,6 +566,20 @@ static VALUE ov_http_client_initialize(int argc, VALUE* argv, VALUE self) {
         connections = NUM2LONG(opt);
     }
 
+   /* Get the value of the 'cookies' parameter. If it is a string it will be used as the path of the file where the
+      cookies will be stored. If it is any other thing it will be treated as a boolean flag indicating if cookies
+      should be enabled but not loaded/saved from/to any file. */
+    opt = rb_hash_aref(opts, COOKIES_SYMBOL);
+    if (TYPE(opt) == T_STRING) {
+        ptr->cookies = ov_string_dup(opt);
+    }
+    else if (RTEST(opt)) {
+        ptr->cookies = ov_string_dup(rb_str_new2(""));
+    }
+    else {
+        ptr->cookies = NULL;
+    }
+
     /* Create the hash that contains the transfers are pending an completed. Both use the identity of the request
        as key. */
     ptr->completed = rb_funcall(rb_hash_new(), COMPARE_BY_IDENTITY_ID, 0);
@@ -534,7 +588,16 @@ static VALUE ov_http_client_initialize(int argc, VALUE* argv, VALUE self) {
     /* Create the libcurl multi handle: */
     ptr->handle = curl_multi_init();
     if (ptr->handle == NULL) {
-        rb_raise(ov_error_class, "Can't create libcurl object");
+        rb_raise(ov_error_class, "Can't create libcurl multi object");
+    }
+
+    /* Create the libcurl share handle in order to share cookie data: */
+    ptr->share = curl_share_init();
+    if (ptr->share == NULL) {
+        rb_raise(ov_error_class, "Can't create libcurl share object");
+    }
+    if (ptr->cookies != NULL) {
+        curl_share_setopt(ptr->share, CURLSHOPT_SHARE, CURL_LOCK_DATA_COOKIE);
     }
 
     /* Enable pipelining: */
@@ -738,6 +801,13 @@ static void ov_http_client_prepare_handle(ov_http_client_object* client_ptr, ov_
     VALUE url;
     int timeout;
 
+    /* Configure sharing of cookies with other handlers created by the client: */
+    curl_easy_setopt(handle, CURLOPT_SHARE, client_ptr->share);
+    if (client_ptr->cookies != NULL && strlen(client_ptr->cookies) > 0) {
+        curl_easy_setopt(handle, CURLOPT_COOKIEFILE, client_ptr->cookies);
+        curl_easy_setopt(handle, CURLOPT_COOKIEJAR, client_ptr->cookies);
+    }
+
     /* Configure TLS parameters: */
     if (client_ptr->insecure) {
         curl_easy_setopt(handle, CURLOPT_SSL_VERIFYPEER, 0L);
@@ -968,7 +1038,7 @@ void ov_http_client_define(void) {
     PROXY_URL_SYMBOL      = ID2SYM(rb_intern("proxy_url"));
     PROXY_USERNAME_SYMBOL = ID2SYM(rb_intern("proxy_username"));
     TIMEOUT_SYMBOL        = ID2SYM(rb_intern("timeout"));
-    USERNAME_SYMBOL       = ID2SYM(rb_intern("username"));
+    COOKIES_SYMBOL        = ID2SYM(rb_intern("cookies"));
 
     /* Define the method identifiers: */
     COMPARE_BY_IDENTITY_ID = rb_intern("compare_by_identity");

data/ext/ovirtsdk4c/ov_http_client.h

@@ -28,9 +28,12 @@ extern VALUE ov_http_client_class;
 
 /* Content: */
 typedef struct {
-    /* The libcurl multi handle: */
+    /* The libcurl multi handle, used to implement multiple simultaneous requests: */
     CURLM* handle;
 
+    /* The libcurl share handle, used to share cookie data between multiple requests: */
+    CURLSH* share;
+
     /* The logger: */
     VALUE log;
 
@@ -51,6 +54,7 @@ typedef struct {
     char* proxy_username;
     char* proxy_password;
     int timeout;
+    char* cookies;
 } ov_http_client_object;
 
 /* Macro to get the pointer: */

data/lib/ovirtsdk4/connection.rb

@@ -16,6 +16,7 @@
 
 require 'json'
 require 'tempfile'
+require 'thread'
 require 'uri'
 
 module OvirtSDK4
@@ -145,6 +146,9 @@ module OvirtSDK4
         @ca_store.close
       end
 
+      # Create the mutex that will be used to prevents simultaneous access to the same HTTP client by multiple threads:
+      @mutex = Mutex.new
+
       # Create the HTTP client:
       @client = HttpClient.new(
         insecure: @insecure,
@@ -184,67 +188,26 @@ module OvirtSDK4
     end
 
     #
-    # Sends an HTTP request.
+    # Sends an HTTP request, making sure that multiple threads are coordinated correctly.
     #
     # @param request [HttpRequest] The request object containing the details of the HTTP request to send.
     #
     # @api private
     #
     def send(request)
-      # Add the base URL to the request:
-      request.url = request.url.nil? ? request.url = @url : "#{@url}/#{request.url}"
-
-      # Set the headers common to all requests:
-      request.headers.merge!(
-        'User-Agent'   => "RubySDK/#{VERSION}",
-        'Version'      => '4',
-        'Content-Type' => 'application/xml',
-        'Accept'       => 'application/xml'
-      )
-
-      # Older versions of the engine (before 4.1) required the 'all_content' as an HTTP header instead of a query
-      # parameter. In order to better support those older versions of the engine we need to check if this parameter is
-      # included in the request, and add the corresponding header.
-      unless request.query.nil?
-        all_content = request.query['all_content']
-        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
-
-      # Send the request:
-      @client.send(request)
+      @mutex.synchronize { internal_send(request) }
     end
 
     #
-    # Waits for the response to the given request.
+    # Waits for the response to the given request, making sure that multiple threads are coordinated correctly.
     #
     # @param request [HttpRequest] The request object whose corresponding response you want to wait for.
-    # @return [Response] A request object containing the details of the HTTP response received.
+    # @return [HttpResponse] A request object containing the details of the HTTP response received.
     #
     # @api private
     #
     def wait(request)
-      # Wait for the response:
-      response = @client.wait(request)
-      raise response if response.is_a?(Exception)
-
-      # If the request failed because of authentication, and it wasn't a request to the SSO service, then the
-      # most likely cause is an expired SSO token. In this case we need to request a new token, and try the original
-      # request again, but only once. It if fails again, we just return the failed response.
-      if response.code == 401 && request.token
-        @token = create_access_token
-        request.token = @token
-        @client.send(request)
-        response = @client.wait(request)
-      end
-
-      response
+      @mutex.synchronize { internal_wait(request) }
     end
 
     #
@@ -330,17 +293,10 @@ module OvirtSDK4
     end
 
     #
-    # Releases the resources used by this connection.
+    # Releases the resources used by this connection, making sure that multiple threads are coordinated correctly.
     #
     def close
-      # Revoke the SSO access token:
-      revoke_access_token if @token
-
-      # Close the HTTP client:
-      @client.close if @client
-
-      # Remove the temporary file that contains the trusted CA certificates:
-      @ca_store.unlink if @ca_store
+      @mutex.synchronize { internal_close }
     end
 
     #
@@ -625,5 +581,85 @@ module OvirtSDK4
       end
       "#{code}: #{description}" if code
     end
+
+    #
+    # Sends an HTTP request.
+    #
+    # @param request [HttpRequest] The request object containing the details of the HTTP request to send.
+    #
+    # @api private
+    #
+    def internal_send(request)
+      # Add the base URL to the request:
+      request.url = request.url.nil? ? request.url = @url : "#{@url}/#{request.url}"
+
+      # Set the headers common to all requests:
+      request.headers.merge!(
+        'User-Agent'   => "RubySDK/#{VERSION}",
+        'Version'      => '4',
+        'Content-Type' => 'application/xml',
+        'Accept'       => 'application/xml'
+      )
+
+      # Older versions of the engine (before 4.1) required the 'all_content' as an HTTP header instead of a query
+      # parameter. In order to better support those older versions of the engine we need to check if this parameter is
+      # included in the request, and add the corresponding header.
+      unless request.query.nil?
+        all_content = request.query['all_content']
+        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
+
+      # Send the request:
+      @client.send(request)
+    end
+
+    #
+    # Waits for the response to the given request.
+    #
+    # @param request [HttpRequest] The request object whose corresponding response you want to wait for.
+    # @return [Response] A request object containing the details of the HTTP response received.
+    #
+    # @api private
+    #
+    def internal_wait(request)
+      # Wait for the response:
+      response = @client.wait(request)
+      raise response if response.is_a?(Exception)
+
+      # If the request failed because of authentication, and it wasn't a request to the SSO service, then the
+      # most likely cause is an expired SSO token. In this case we need to request a new token, and try the original
+      # request again, but only once. It if fails again, we just return the failed response.
+      if response.code == 401 && request.token
+        @token = create_access_token
+        request.token = @token
+        @client.send(request)
+        response = @client.wait(request)
+      end
+
+      response
+    end
+
+    #
+    # Releases the resources used by this connection.
+    #
+    # @api private
+    #
+    def internal_close
+      # Revoke the SSO access token:
+      revoke_access_token if @token
+
+      # 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/readers.rb

@@ -78,6 +78,8 @@ module OvirtSDK4
       object.reason = value if not value.nil?
       value = reader.get_attribute('reassign_bad_macs')
       object.reassign_bad_macs = value if not value.nil?
+      value = reader.get_attribute('reboot')
+      object.reboot = value if not value.nil?
       value = reader.get_attribute('remote_viewer_connection_file')
       object.remote_viewer_connection_file = value if not value.nil?
       value = reader.get_attribute('resolution_type')
@@ -209,6 +211,10 @@ module OvirtSDK4
           object.reason = Reader.read_string(reader)
         when 'reassign_bad_macs'
           object.reassign_bad_macs = Reader.read_boolean(reader)
+        when 'reboot'
+          object.reboot = Reader.read_boolean(reader)
+        when 'registration_configuration'
+          object.registration_configuration = RegistrationConfigurationReader.read_one(reader)
         when 'remote_viewer_connection_file'
           object.remote_viewer_connection_file = Reader.read_string(reader)
         when 'removed_bonds'
@@ -2036,6 +2042,8 @@ module OvirtSDK4
           object.data_center = DataCenterReader.read_one(reader)
         when 'enabled_features'
           object.enabled_features = ClusterFeatureReader.read_many(reader)
+        when 'external_network_providers'
+          object.external_network_providers = ExternalProviderReader.read_many(reader)
         when 'gluster_hooks'
           object.gluster_hooks = GlusterHookReader.read_many(reader)
         when 'gluster_volumes'
@@ -2104,6 +2112,8 @@ module OvirtSDK4
           object.cpu_profiles = list
         when 'enabledfeatures'
           object.enabled_features = list
+        when 'externalnetworkproviders'
+          object.external_network_providers = list
         when 'glusterhooks'
           object.gluster_hooks = list
         when 'glustervolumes'
@@ -4884,6 +4894,84 @@ module OvirtSDK4
     
   end
   
+  class ExternalNetworkProviderConfigurationReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = ExternalNetworkProviderConfiguration.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      value = reader.get_attribute('comment')
+      object.comment = value if not value.nil?
+      value = reader.get_attribute('description')
+      object.description = value if not value.nil?
+      value = reader.get_attribute('id')
+      object.id = value if not value.nil?
+      value = reader.get_attribute('name')
+      object.name = value if not value.nil?
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'comment'
+          object.comment = Reader.read_string(reader)
+        when 'description'
+          object.description = Reader.read_string(reader)
+        when 'id'
+          object.id = Reader.read_string(reader)
+        when 'name'
+          object.name = Reader.read_string(reader)
+        when 'external_network_provider'
+          object.external_network_provider = ExternalProviderReader.read_one(reader)
+        when 'host'
+          object.host = HostReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
   class ExternalProviderReader < Reader
     
     def self.read_one(reader)
@@ -7202,6 +7290,8 @@ module OvirtSDK4
           object.devices = DeviceReader.read_many(reader)
         when 'external_host_provider'
           object.external_host_provider = ExternalHostProviderReader.read_one(reader)
+        when 'external_network_provider_configurations'
+          object.external_network_provider_configurations = ExternalNetworkProviderConfigurationReader.read_many(reader)
         when 'hooks'
           object.hooks = HookReader.read_many(reader)
         when 'katello_errata'
@@ -7276,6 +7366,8 @@ module OvirtSDK4
           object.agents = list
         when 'devices'
           object.devices = list
+        when 'externalnetworkproviderconfigurations'
+          object.external_network_provider_configurations = list
         when 'hooks'
           object.hooks = list
         when 'katelloerrata'
@@ -8420,6 +8512,8 @@ module OvirtSDK4
       object.stateless = value if not value.nil?
       value = reader.get_attribute('status')
       object.status = value if not value.nil?
+      value = reader.get_attribute('storage_error_resume_behaviour')
+      object.storage_error_resume_behaviour = value if not value.nil?
       value = reader.get_attribute('tunnel_migration')
       object.tunnel_migration = value if not value.nil?
       value = reader.get_attribute('type')
@@ -8503,6 +8597,8 @@ module OvirtSDK4
           object.stateless = Reader.read_boolean(reader)
         when 'status'
           object.status = Reader.read_enum(TemplateStatus, reader)
+        when 'storage_error_resume_behaviour'
+          object.storage_error_resume_behaviour = Reader.read_enum(VmStorageErrorResumeBehaviour, reader)
         when 'time_zone'
           object.time_zone = TimeZoneReader.read_one(reader)
         when 'tunnel_migration'
@@ -13756,6 +13852,448 @@ module OvirtSDK4
     
   end
   
+  class RegistrationAffinityGroupMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationAffinityGroupMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = AffinityGroupReader.read_one(reader)
+        when 'to'
+          object.to = AffinityGroupReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationAffinityLabelMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationAffinityLabelMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = AffinityLabelReader.read_one(reader)
+        when 'to'
+          object.to = AffinityLabelReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationClusterMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationClusterMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = ClusterReader.read_one(reader)
+        when 'to'
+          object.to = ClusterReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationConfigurationReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationConfiguration.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'affinity_group_mappings'
+          object.affinity_group_mappings = RegistrationAffinityGroupMappingReader.read_many(reader)
+        when 'affinity_label_mappings'
+          object.affinity_label_mappings = RegistrationAffinityLabelMappingReader.read_many(reader)
+        when 'cluster_mappings'
+          object.cluster_mappings = RegistrationClusterMappingReader.read_many(reader)
+        when 'domain_mappings'
+          object.domain_mappings = RegistrationDomainMappingReader.read_many(reader)
+        when 'lun_mappings'
+          object.lun_mappings = RegistrationLunMappingReader.read_many(reader)
+        when 'role_mappings'
+          object.role_mappings = RegistrationRoleMappingReader.read_many(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationDomainMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationDomainMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = DomainReader.read_one(reader)
+        when 'to'
+          object.to = DomainReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationLunMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationLunMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = DiskReader.read_one(reader)
+        when 'to'
+          object.to = DiskReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
+  class RegistrationRoleMappingReader < Reader
+    
+    def self.read_one(reader)
+      # Do nothing if there aren't more tags:
+      return nil unless reader.forward
+      
+      # Create the object:
+      object = RegistrationRoleMapping.new
+      
+      # Process the attributes:
+      object.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return object if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        case reader.node_name
+        when 'from'
+          object.from = RoleReader.read_one(reader)
+        when 'to'
+          object.to = RoleReader.read_one(reader)
+        else
+          reader.next_element
+        end
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return object
+    end
+    
+    
+    def self.read_many(reader)
+      # Do nothing if there aren't more tags:
+      list = List.new
+      return list unless reader.forward
+      
+      # Process the attributes:
+      list.href = reader.get_attribute('href')
+      
+      # Discard the start tag:
+      empty = reader.empty_element?
+      reader.read
+      return list if empty
+      
+      # Process the inner elements:
+      while reader.forward do
+        list << read_one(reader)
+      end
+      
+      # Discard the end tag:
+      reader.read
+      
+      return list
+    end
+    
+  end
+  
   class ReportedConfigurationReader < Reader
     
     def self.read_one(reader)
@@ -14697,6 +15235,8 @@ module OvirtSDK4
       object.stop_reason = value if not value.nil?
       value = reader.get_attribute('stop_time')
       object.stop_time = value if not value.nil?
+      value = reader.get_attribute('storage_error_resume_behaviour')
+      object.storage_error_resume_behaviour = value if not value.nil?
       value = reader.get_attribute('tunnel_migration')
       object.tunnel_migration = value if not value.nil?
       value = reader.get_attribute('type')
@@ -14816,6 +15356,8 @@ module OvirtSDK4
           object.stop_reason = Reader.read_string(reader)
         when 'stop_time'
           object.stop_time = Reader.read_date(reader)
+        when 'storage_error_resume_behaviour'
+          object.storage_error_resume_behaviour = Reader.read_enum(VmStorageErrorResumeBehaviour, reader)
         when 'time_zone'
           object.time_zone = TimeZoneReader.read_one(reader)
         when 'tunnel_migration'
@@ -16316,6 +16858,8 @@ module OvirtSDK4
       object.stateless = value if not value.nil?
       value = reader.get_attribute('status')
       object.status = value if not value.nil?
+      value = reader.get_attribute('storage_error_resume_behaviour')
+      object.storage_error_resume_behaviour = value if not value.nil?
       value = reader.get_attribute('tunnel_migration')
       object.tunnel_migration = value if not value.nil?
       value = reader.get_attribute('type')
@@ -16399,6 +16943,8 @@ module OvirtSDK4
           object.stateless = Reader.read_boolean(reader)
         when 'status'
           object.status = Reader.read_enum(TemplateStatus, reader)
+        when 'storage_error_resume_behaviour'
+          object.storage_error_resume_behaviour = Reader.read_enum(VmStorageErrorResumeBehaviour, reader)
         when 'time_zone'
           object.time_zone = TimeZoneReader.read_one(reader)
         when 'tunnel_migration'
@@ -17659,6 +18205,8 @@ module OvirtSDK4
       object.stop_reason = value if not value.nil?
       value = reader.get_attribute('stop_time')
       object.stop_time = value if not value.nil?
+      value = reader.get_attribute('storage_error_resume_behaviour')
+      object.storage_error_resume_behaviour = value if not value.nil?
       value = reader.get_attribute('tunnel_migration')
       object.tunnel_migration = value if not value.nil?
       value = reader.get_attribute('type')
@@ -17770,6 +18318,8 @@ module OvirtSDK4
           object.stop_reason = Reader.read_string(reader)
         when 'stop_time'
           object.stop_time = Reader.read_date(reader)
+        when 'storage_error_resume_behaviour'
+          object.storage_error_resume_behaviour = Reader.read_enum(VmStorageErrorResumeBehaviour, reader)
         when 'time_zone'
           object.time_zone = TimeZoneReader.read_one(reader)
         when 'tunnel_migration'
@@ -17966,6 +18516,8 @@ module OvirtSDK4
       object.start_paused = value if not value.nil?
       value = reader.get_attribute('stateless')
       object.stateless = value if not value.nil?
+      value = reader.get_attribute('storage_error_resume_behaviour')
+      object.storage_error_resume_behaviour = value if not value.nil?
       value = reader.get_attribute('tunnel_migration')
       object.tunnel_migration = value if not value.nil?
       value = reader.get_attribute('type')
@@ -18047,6 +18599,8 @@ module OvirtSDK4
           object.start_paused = Reader.read_boolean(reader)
         when 'stateless'
           object.stateless = Reader.read_boolean(reader)
+        when 'storage_error_resume_behaviour'
+          object.storage_error_resume_behaviour = Reader.read_enum(VmStorageErrorResumeBehaviour, reader)
         when 'time_zone'
           object.time_zone = TimeZoneReader.read_one(reader)
         when 'tunnel_migration'
@@ -19001,6 +19555,8 @@ module OvirtSDK4
   Reader.register('external_host_groups', ExternalHostGroupReader.method(:read_many))
   Reader.register('external_host_provider', ExternalHostProviderReader.method(:read_one))
   Reader.register('external_host_providers', ExternalHostProviderReader.method(:read_many))
+  Reader.register('external_network_provider_configuration', ExternalNetworkProviderConfigurationReader.method(:read_one))
+  Reader.register('external_network_provider_configurations', ExternalNetworkProviderConfigurationReader.method(:read_many))
   Reader.register('external_provider', ExternalProviderReader.method(:read_one))
   Reader.register('external_providers', ExternalProviderReader.method(:read_many))
   Reader.register('external_vm_import', ExternalVmImportReader.method(:read_one))
@@ -19195,6 +19751,20 @@ module OvirtSDK4
   Reader.register('ranges', RangeReader.method(:read_many))
   Reader.register('rate', RateReader.method(:read_one))
   Reader.register('rates', RateReader.method(:read_many))
+  Reader.register('registration_affinity_group_mapping', RegistrationAffinityGroupMappingReader.method(:read_one))
+  Reader.register('registration_affinity_group_mappings', RegistrationAffinityGroupMappingReader.method(:read_many))
+  Reader.register('registration_affinity_label_mapping', RegistrationAffinityLabelMappingReader.method(:read_one))
+  Reader.register('registration_affinity_label_mappings', RegistrationAffinityLabelMappingReader.method(:read_many))
+  Reader.register('registration_cluster_mapping', RegistrationClusterMappingReader.method(:read_one))
+  Reader.register('registration_cluster_mappings', RegistrationClusterMappingReader.method(:read_many))
+  Reader.register('registration_configuration', RegistrationConfigurationReader.method(:read_one))
+  Reader.register('registration_configurations', RegistrationConfigurationReader.method(:read_many))
+  Reader.register('registration_domain_mapping', RegistrationDomainMappingReader.method(:read_one))
+  Reader.register('registration_domain_mappings', RegistrationDomainMappingReader.method(:read_many))
+  Reader.register('registration_lun_mapping', RegistrationLunMappingReader.method(:read_one))
+  Reader.register('registration_lun_mappings', RegistrationLunMappingReader.method(:read_many))
+  Reader.register('registration_role_mapping', RegistrationRoleMappingReader.method(:read_one))
+  Reader.register('registration_role_mappings', RegistrationRoleMappingReader.method(:read_many))
   Reader.register('reported_configuration', ReportedConfigurationReader.method(:read_one))
   Reader.register('reported_configurations', ReportedConfigurationReader.method(:read_many))
   Reader.register('reported_device', ReportedDeviceReader.method(:read_one))