Change in ovirt-engine[master]: packaging: setup: clear PGPASSWORD when interacting with dat...

oschreib at redhat.com oschreib at redhat.com
Mon Sep 2 08:16:16 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Spenser Shumaker <sshumake at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>



More information about the Engine-commits mailing list