Alex Lourie has posted comments on this change.

Change subject: core:  Installation fails with message of...
......................................................................


Patch Set 1:

(1 comment)

....................................................
File packaging/dbscripts/dbfunctions.sh
Line 104:     # before execution check if actually required
Line 105:     #
Line 106:     CMD="select count(*) from pg_proc where proname = 'uuid_nil';"
Line 107:     if [ "$(execute_command "$CMD" ${DATABASE} ${SERVERNAME} ${PORT} 
| sed -e 's/ //g' -e '/^$/d')" != 0 ]; then
Line 108:         psql -w -U postgres -h ${SERVERNAME} -p ${PORT} -f 
drop_old_uuid_functions.sql ${DATABASE} > /dev/null
@Allon:

I think dropping uuid requires superuser. ${USERNAME} has no such permissions.
Line 109:     fi
Line 110: }
Line 111: 
Line 112: #drops sps before upgrade or refresh operations


-- 
To view, visit http://gerrit.ovirt.org/18267
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4a2d406ff71f312805c8e55c1fe58886c591e1f6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to