Ofer Schreiber has submitted this change and it was merged.
Change subject: packaging: setup: clear PGPASSWORD when interacting with database
......................................................................
packaging: setup: clear PGPASSWORD when interacting with database
when both PGPASSWORD and PGPASSFILE are specified the PGPASSWORD takes
precedence, so we should unset/zero it to avoid errors when user has
PGPASSWORD within environment and executes setup.
Change-Id: Ie02939fe51eb4308d76a65e9eb1eb86df4a6801c
Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com>
---
M packaging/dbscripts/dbcustomfunctions.sh
M packaging/setup/ovirt_engine_setup/database.py
2 files changed, 3 insertions(+), 0 deletions(-)
Approvals:
Ofer Schreiber: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/18704
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie02939fe51eb4308d76a65e9eb1eb86df4a6801c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Spenser Shumaker <sshumake(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>