Bug 1029050 - Reinstall host by rest api fails on root_password field requirement
Summary: Reinstall host by rest api fails on root_password field requirement
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 3.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.4.0
Assignee: Yaniv Bronhaim
QA Contact: sefi litmanovich
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-11 14:40 UTC by Yaniv Bronhaim
Modified: 2016-02-10 19:28 UTC (History)
9 users (show)

Fixed In Version: ovirt-3.4.0-alpha1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-06-12 14:04:34 UTC
oVirt Team: Infra
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 21589 0 None None None Never

Description Yaniv Bronhaim 2013-11-11 14:40:05 UTC
Description of problem:
Request for reinstall action requires <root_password></root_password> as part of the request although we connect by public key 

--- the request
<action>                                                  
<root_password></root_password>   <-- without those
<ssh>                                                                          
<port>22</port>                                                                
<authentication_method>PublicKey</authentication_method>                        
</ssh>                                                                          
</action>
---

returns the failure from BZ 987982 (rootPassword is required)

How reproducible:
100%

Steps to Reproduce:
1. add host, put it on maintainace
1. run the above REST API action

Actual results:
Failing the operation on required field rootPassword 


Expected results:
Should work without any warning if pk is verified

Additional info:
we keep validate rootPassword field in BackendHostMapper-
 validateParameters(action, "rootPassword");

It shouldn't be there at all

Comment 1 Sandro Bonazzola 2014-01-14 08:43:17 UTC
ovirt 3.4.0 alpha has been released

Comment 2 sefi litmanovich 2014-01-30 17:58:37 UTC
Verified on RHEL6.5, ovirt-engine-3.4.0-0.5

Comment 3 Itamar Heim 2014-06-12 14:04:34 UTC
Closing as part of 3.4.0


Note You need to log in before you can comment on or make changes to this bug.