Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: setup: test only config file db password
......................................................................

packaging: setup: test only config file db password

Testing also env breaks unattended clean setup.
Bug-Url: https://bugzilla.redhat.com/1094016

Change-Id: I7a721ce3322bab5ea972e43bd8155beb9b447f30
Signed-off-by: Yedidyah Bar David <[email protected]>
---
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py
1 file changed, 1 insertion(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-dwh refs/changes/68/29768/1

diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py
index fa511d8..572cf14 100644
--- 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py
+++ 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py
@@ -153,10 +153,7 @@
             odwhcons.FileLocations.OVIRT_ENGINE_DWHD_SERVICE_CONFIG_DEFAULTS,
             odwhcons.FileLocations.OVIRT_ENGINE_DWHD_SERVICE_CONFIG,
         ])
-        if (
-            config.get('DWH_DB_PASSWORD') or
-            self.environment.get(odwhcons.DBEnv.PASSWORD)
-        ):
+        if config.get('DWH_DB_PASSWORD'):
             try:
                 dbenv = {}
                 for e, k in (


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7a721ce3322bab5ea972e43bd8155beb9b447f30
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to