Moti Asayag has uploaded a new change for review.

Change subject: restapi: Associate a host with its host provider
......................................................................

restapi: Associate a host with its host provider

The Host element is extended with the host external provider
reference which is used to query a specific host for the
available errata.

Change-Id: I27668d74d2b77a9e3e282cca606503c910158f66
Bug-Url: https://bugzilla.redhat.com/1157377
Signed-off-by: Moti Asayag <masa...@redhat.com>
---
M 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
3 files changed, 15 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/56/37556/1

diff --git 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
index a574291..df3a064 100644
--- 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
+++ 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
@@ -1940,6 +1940,7 @@
           <xs:element name="numa_supported" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>
           <xs:element name="live_snapshot_support" type="xs:boolean" 
minOccurs="0" maxOccurs="1" />
           <xs:element ref="katello_errata"  minOccurs="0" maxOccurs="1"/>
+          <xs:element ref="external_host_provider"  minOccurs="0" 
maxOccurs="1"/>
         </xs:sequence>
       </xs:extension>
     </xs:complexContent>
diff --git 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
index 6a9ce90..572e138 100644
--- 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
+++ 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
@@ -2848,7 +2848,7 @@
           host.power_management.options.option--COLLECTION: {option.name: 
'xs:string', option.value: 'xs:string'},
           host.power_management.pm_proxy--COLLECTION: {propietary : 
'xs:string'}, host.power_management.agents.agent--COLLECTION:{type: 'xs:string',
           address: 'xs:string', username: 'xs:string', password: 'xs:string', 
options.option--COLLECTION: {option.name: 'xs:string', option.value: 
'xs:string'}},
-          host.power_management.kdump_detection: 'xs:boolean'}
+          host.power_management.kdump_detection: 'xs:boolean', 
host.external_host_provider.id: 'xs:string'}
         description: update the specified host in the system. This is 
deprecated and is provided only for backwards compatibility
       - mandatoryArguments: {}
         optionalArguments: {host.name: 'xs:string', host.comment: 'xs:string', 
host.address: 'xs:string',
diff --git 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
index 5396c50..d7fb9ba 100644
--- 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
+++ 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
@@ -12,6 +12,7 @@
 import org.ovirt.engine.api.model.Cluster;
 import org.ovirt.engine.api.model.CpuTopology;
 import org.ovirt.engine.api.model.Display;
+import org.ovirt.engine.api.model.ExternalHostProvider;
 import org.ovirt.engine.api.model.HardwareInformation;
 import org.ovirt.engine.api.model.Hook;
 import org.ovirt.engine.api.model.Hooks;
@@ -21,24 +22,24 @@
 import org.ovirt.engine.api.model.HostType;
 import org.ovirt.engine.api.model.HostedEngine;
 import org.ovirt.engine.api.model.IscsiDetails;
-import org.ovirt.engine.api.model.KdumpStatus;
 import org.ovirt.engine.api.model.KSM;
+import org.ovirt.engine.api.model.KdumpStatus;
 import org.ovirt.engine.api.model.OperatingSystem;
 import org.ovirt.engine.api.model.Option;
 import org.ovirt.engine.api.model.Options;
 import org.ovirt.engine.api.model.PmProxies;
 import org.ovirt.engine.api.model.PmProxy;
 import org.ovirt.engine.api.model.PowerManagement;
+import org.ovirt.engine.api.model.SELinux;
 import org.ovirt.engine.api.model.SELinuxMode;
 import org.ovirt.engine.api.model.SPM;
 import org.ovirt.engine.api.model.SSH;
+import org.ovirt.engine.api.model.SpmState;
 import org.ovirt.engine.api.model.StorageManager;
 import org.ovirt.engine.api.model.TransparentHugePages;
 import org.ovirt.engine.api.model.User;
 import org.ovirt.engine.api.model.Version;
 import org.ovirt.engine.api.model.VmSummary;
-import org.ovirt.engine.api.model.SELinux;
-import org.ovirt.engine.api.model.SpmState;
 import org.ovirt.engine.api.restapi.model.AuthenticationMethod;
 import org.ovirt.engine.api.restapi.utils.GuidUtils;
 import org.ovirt.engine.core.common.action.VdsOperationActionParameters;
@@ -106,6 +107,10 @@
         }
         if (model.isSetComment()) {
             entity.setComment(model.getComment());
+        }
+        if (model.isSetExternalHostProvider()) {
+            String providerId = model.getExternalHostProvider().getId();
+            entity.setHostProviderId(providerId == null ? null : 
GuidUtils.asGuid(providerId));
         }
         return entity;
     }
@@ -288,6 +293,11 @@
         model.setNumaSupported(entity.isNumaSupport());
 
         model.setLiveSnapshotSupport(entity.getLiveSnapshotSupport());
+
+        if (entity.getHostProviderId() != null) {
+            model.setExternalHostProvider(new ExternalHostProvider());
+            
model.getExternalHostProvider().setId(entity.getHostProviderId().toString());
+        }
         return model;
     }
 


-- 
To view, visit http://gerrit.ovirt.org/37556
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I27668d74d2b77a9e3e282cca606503c910158f66
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to