Alex Lourie has uploaded a new change for review.

Change subject: packaging: setup: call psql with correct env
......................................................................

packaging: setup: call psql with correct env

Change-Id: I5464894fca921a7d3fdd41fd554dcb6b9805a785
Bug-Url: https://bugzilla.redhat.com/991760
Signed-off-by: Alex Lourie <alou...@redhat.com>
---
M packaging/common_utils.py
M packaging/ovirt-engine-dwh-setup.py
2 files changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-dwh refs/changes/25/17625/1

diff --git a/packaging/common_utils.py b/packaging/common_utils.py
index af886d6..f3c37f5 100755
--- a/packaging/common_utils.py
+++ b/packaging/common_utils.py
@@ -871,7 +871,7 @@
     return doBackup
 
 @transactionDisplay("Backing up the DB")
-def backupDB(backup_file, db_dict):
+def backupDB(backup_file, db_dict, PGPASS_FILE):
     """
     Backup postgres db
     Args:  file - a target file to backup to
@@ -885,6 +885,7 @@
         EXEC_PGDUMP,
         '-C',
         '-E',
+        '-w',
         'UTF8',
         '--disable-dollar-quoting',
         '--disable-triggers',
@@ -900,6 +901,7 @@
         cmdList=cmd,
         failOnError=True,
         msg='Error during DB backup.',
+        envDict={'ENGINE_PGPASS': PGPASS_FILE}
     )
     logging.debug("%s DB Backup completed successfully", db_dict['name'])
 
diff --git a/packaging/ovirt-engine-dwh-setup.py 
b/packaging/ovirt-engine-dwh-setup.py
index 6126559..d5e2671 100755
--- a/packaging/ovirt-engine-dwh-setup.py
+++ b/packaging/ovirt-engine-dwh-setup.py
@@ -297,6 +297,7 @@
                         utils.backupDB(
                             backupFile,
                             db_dict,
+                            PGPASS_TEMP,
                         )
                 except UserWarning:
                     print 'User decided to stop setup. Exiting.'


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5464894fca921a7d3fdd41fd554dcb6b9805a785
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alou...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to