Change in ovirt-engine[master]: api: Fix approve API to allow deploy with public key
oourfali at redhat.com
oourfali at redhat.com
Mon Jun 2 18:29:09 UTC 2014
Oved Ourfali has submitted this change and it was merged.
Change subject: api: Fix approve API to allow deploy with public key
......................................................................
api: Fix approve API to allow deploy with public key
Currently we don't set the authentication method and engine searches for
password validation. Approve API should use PK auth as default except
password and authentication method are set.
(Apperently approve call by API was never tested and currently fails
on "empty password")
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101565
Change-Id: I9816d907d45ec7b4ee066fa369ca1150e688084f
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java
2 files changed, 26 insertions(+), 2 deletions(-)
Approvals:
Amador Pahim: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/27622
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9816d907d45ec7b4ee066fa369ca1150e688084f
Gerrit-PatchSet: 15
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Amador Pahim <apahim at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list