Oved Ourfali has submitted this change and it was merged. Change subject: tools: Fix issue with adding permission for manage-domains user ...................................................................... tools: Fix issue with adding permission for manage-domains user Fixes parameters of attach_user_to_role, which were broken due to refactoring made in I04e9c6fdb6723cd369908d40bb8a4a19f74d10f3. Change-Id: Ief31a99408706ee7a4974c93b63043c64f020d2b Bug-Url: https://bugzilla.redhat.com/1275043 Signed-off-by: Martin Perina <mperina@redhat.com> --- M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomainsDAOImpl.java 1 file changed, 6 insertions(+), 5 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Oved Ourfali: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47621 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief31a99408706ee7a4974c93b63043c64f020d2b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Eyal Edri <eedri@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: automation@ovirt.org