Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: validating read-only user before creation ...................................................................... packaging: setup: validating read-only user before creation The new implementation validates provided username before it creates a readonly user. The user provided cannot be one of the reserved users (postgres, engine, engine_history and engine_reports), it cannot be empty and it should not exist in the DB. Change-Id: Ic404e565a765c82d42f92a211a59d06c8a59d807 Bug-Url: https://bugzilla.redhat.com/1006948 Signed-off-by: Alex Lourie <alourie@redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 41 insertions(+), 5 deletions(-) Approvals: Alex Lourie: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19944 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic404e565a765c82d42f92a211a59d06c8a59d807 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie@redhat.com> Gerrit-Reviewer: Alex Lourie <alourie@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary@redhat.com>