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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Amador Pahim <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Ori Liel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
