Piotr Kliczewski 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
Option --add-permissions in engine-manage-domains was removed, because
user LDAP user specified during engine-manage-domains execution should
be used only to access LDAP server and not as an oVirt administrator.
Change-Id: Ief31a99408706ee7a4974c93b63043c64f020d2b
Bug-Url:
https://bugzilla.redhat.com/1268076
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomains.java
M
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomainsArguments.java
M
backend/manager/modules/builtin-extensions/src/main/resources/manage-domains-help.properties
M
backend/manager/modules/builtin-extensions/src/main/resources/org/ovirt/engine/extensions/aaa/builtin/tools/arguments.properties
M packaging/man/man8/engine-manage-domains.8
5 files changed, 11 insertions(+), 37 deletions(-)
Approvals:
Piotr Kliczewski: Passed CI tests
Martin Peřina: Verified
Moti Asayag: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/47617
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ief31a99408706ee7a4974c93b63043c64f020d2b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>