Yaniv Bronhaim has uploaded a new change for review.

Change subject: API: editing host update and add fields
......................................................................

API: editing host update and add fields

Updating ssh fields in adding new host:
* Setting password as optional parameters (when using pk the field can be
  omitted).
* Removing port, ip and authentications fields from updating operation, as
  updating always uses pk authentication.

Change-Id: I640cfca4057ffd65d37afc10bc5ac2e12f1aafd5
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=999640
Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com>
---
M 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
1 file changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/54/18754/1

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 9de8c7d..c174523 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
@@ -2202,7 +2202,7 @@
       parameterType: Host
       signatures:
       - mandatoryArguments: {}
-        optionalArguments: {host.name: 'xs:string', host.comment: 'xs:string', 
host.address: 'xs:string', host.root_password--DEPRECATED: 'xs:string',
+        optionalArguments: {host.name: 'xs:string', host.comment: 'xs:string', 
host.address: 'xs:string',
           host.display.address: 'xs:string', host.cluster.id|name: 
'xs:string', host.port: 'xs:int', host.storage_manager.priority: 'xs:int',
           host.power_management.type: 'xs:string', 
host.power_management.enabled: 'xs:boolean', host.power_management.address: 
'xs:string',
           host.power_management.username: 'xs:string', 
host.power_management.password: 'xs:string',
@@ -2211,8 +2211,7 @@
           address: 'xs:string', user_name: 'xs:string', password: 'xs:string', 
options.option--COLLECTION: {option.name: 'xs:string', option.value: 
'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', host.ssh.password: 'xs:string',
-          host.ssh.port: 'xs:int', host.ssh.fingerprint: 'xs:string', 
host.ssh.authentication_type: 'xs:string', host.display.address: 'xs:string',
+        optionalArguments: {host.name: 'xs:string', host.comment: 'xs:string', 
host.address: 'xs:string', host.ssh.fingerprint: 'xs:string', 
host.display.address: 'xs:string',
           host.cluster.id|name: 'xs:string', host.port: 'xs:int', 
host.storage_manager.priority: 'xs:int', host.power_management.type: 
'xs:string',
           host.power_management.enabled: 'xs:boolean', 
host.power_management.address: 'xs:string', host.power_management.username: 
'xs:string',
           host.power_management.password: 'xs:string', 
host.power_management.options.option--COLLECTION: {option.name: 'xs:string',
@@ -2239,8 +2238,9 @@
           host.power_management.pm_proxy--COLLECTION: {propietary : 
'xs:string'}, host.power_management.agents.agent--COLLECTION:{type: 'xs:string',
           address: 'xs:string', user_name: 'xs:string', password: 'xs:string', 
options.option--COLLECTION: {option.name: 'xs:string', option.value: 
'xs:string'}}, host.reboot_after_installation: 'xs:boolean', 
host.override_iptables: 'xs:boolean'}
         description: add a new host to the system providing the host root 
password. This has been deprecated and provided for backwards compatibility
-      - mandatoryArguments: {host.name: 'xs:string', host.address: 
'xs:string', host.ssh.password: 'xs:string', host.cluster.id|name: 'xs:string'}
-        optionalArguments: {host.comment: 'xs:string', host.ssh.port: 
'xs:int', host.ssh.fingerprint: 'xs:string', host.ssh.authentication_type: 
'xs:string', host.port: 'xs:int',
+      - mandatoryArguments: {host.name: 'xs:string', host.address: 
'xs:string', host.cluster.id|name: 'xs:string'}
+        optionalArguments: {host.comment: 'xs:string', host.ssh.port: 
'xs:int', host.ssh.fingerprint: 'xs:string', host.ssh.authentication_method: 
'xs:string',
+          host.ssh.user.user_name: 'xs:string', host.ssh.user.password: 
'xs:string', host.port: 'xs:int',
           host.display.address: 'xs:string', host.storage_manager.priority: 
'xs:int', host.power_management.type: 'xs:string',
           host.power_management.enabled: 'xs:boolean', 
host.power_management.address: 'xs:string', host.power_management.username: 
'xs:string',
           host.power_management.password: 'xs:string', 
host.power_management.options.option--COLLECTION: {option.name: 'xs:string', 
option.value: 'xs:string'},


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

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

Reply via email to