Moti Asayag has posted comments on this change.

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


Patch Set 6:

(1 comment)

http://gerrit.ovirt.org/#/c/37556/6/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml:

Line 2847:           host.power_management.username: 'xs:string', 
host.power_management.password: 'xs:string', 
host.power_management.automatic_pm_enabled: 'xs:boolean',
Line 2848:           host.power_management.options.option--COLLECTION: 
{option.name: 'xs:string', option.value: 'xs:string'},
Line 2849:           host.power_management.pm_proxy--COLLECTION: {propietary : 
'xs:string'}, host.power_management.agents.agent--COLLECTION:{type: 'xs:string',
Line 2850:           address: 'xs:string', username: 'xs:string', password: 
'xs:string', options.option--COLLECTION: {option.name: 'xs:string', 
option.value: 'xs:string'}},
Line 2851:           host.power_management.kdump_detection: 'xs:boolean', 
host.external_host_provider.id: 'xs:string'}
> Will the backend use this only for updates or also when adding a new host? 
>From backend side, when this field is used for adding a host, it should be in 
>the 'provision host' flow.

There is not support for supplying a host provider id, when there is no 
intention to use that provider for the host provisioning.

So if a simple (non-provisioned) host was added to the system, it shouldn't 
provider any provider-id on the 'add host' flow, but could be updated later-on.
Line 2852:         description: update the specified host in the system. This 
is deprecated and is provided only for backwards compatibility
Line 2853:       - mandatoryArguments: {}
Line 2854:         optionalArguments: {host.name: 'xs:string', host.comment: 
'xs:string', host.address: 'xs:string',
Line 2855:           host.ssh.port: 'xs:int', host.ssh.user.user_name: 
'xs:string', host.ssh.fingerprint: 'xs:string', host.display.address: 
'xs:string',


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I27668d74d2b77a9e3e282cca606503c910158f66
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to