Ofer Schreiber has submitted this change and it was merged. Change subject: setup: test effective user id instead of 'root' user name ...................................................................... setup: test effective user id instead of 'root' user name Root user or every other alias will have UID=0, better test for UID and not user name. In bash we have $UID, $EUID. In POSIX sh we have $(id -u). In Python, there is no need to convert uid to name in order to test if we run under root. Change-Id: Iad84de5277439b2a1aac7b5985954c1ad8ffca80 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com&gt; --- M backend/manager/dbscripts/engine-db-install.sh M packaging/fedora/setup/engine-cleanup.py M packaging/fedora/setup/engine-upgrade.py 3 files changed, 5 insertions(+), 5 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6051 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad84de5277439b2a1aac7b5985954c1ad8ffca80 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com&gt; Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com&gt; Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com&gt; Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com&gt;