Bug 1029050

Summary: Reinstall host by rest api fails on root_password field requirement
Product: Red Hat Enterprise Virtualization Manager Reporter: Yaniv Bronhaim <ybronhei>
Component: ovirt-engineAssignee: Yaniv Bronhaim <ybronhei>
Status: CLOSED CURRENTRELEASE QA Contact: sefi litmanovich <slitmano>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.3.0CC: aberezin, acathrow, bazulay, emesika, iheim, lpeer, pstehlik, Rhev-m-bugs, yeylon
Target Milestone: ---   
Target Release: 3.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: ovirt-3.4.0-alpha1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-06-12 14:04:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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