Yaniv Dary has uploaded a new change for review.

Change subject: packaging: changed the db dict get method
......................................................................

packaging: changed the db dict get method

Engine now saved config values with quotes marks. This breaks
the current db dict get method. Added replace for these values.

Change-Id: I72e386787f7b074f365506dedb9976fed2554dc4
Signed-off-by: Yaniv Dary <[email protected]>
---
M packaging/ovirt-engine-reports-setup.py
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-reports refs/changes/01/20701/1

diff --git a/packaging/ovirt-engine-reports-setup.py 
b/packaging/ovirt-engine-reports-setup.py
index 9207d50..85113e5 100755
--- a/packaging/ovirt-engine-reports-setup.py
+++ b/packaging/ovirt-engine-reports-setup.py
@@ -265,8 +265,8 @@
                 dhandler.getParam('REPORTS_DATABASE') or
                 JRS_DB_NAME
             ),
-            'host': handler.getParam('ENGINE_DB_HOST'),
-            'port': handler.getParam('ENGINE_DB_PORT'),
+            'host': handler.getParam('ENGINE_DB_HOST').replace('"', ''),
+            'port': handler.getParam('ENGINE_DB_PORT').replace('"', ''),
             'username': (
                 dhandler.getParam('REPORTS_USER') or
                 REPORTS_DB_USER
@@ -276,10 +276,10 @@
                 utils.generatePassword()
             ),
             'engine_db': (
-                handler.getParam('ENGINE_DB_NAME') or
+                handler.getParam('ENGINE_DB_NAME').replace('"', '') or
                 ENGINE_DB_NAME
             ),
-            'engine_user': handler.getParam('ENGINE_DB_USER'),
+            'engine_user': handler.getParam('ENGINE_DB_USER').replace('"', ''),
             'engine_pass': handler.getParam('ENGINE_DB_PASSWORD').replace('"', 
''),
         }
         handler.close()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I72e386787f7b074f365506dedb9976fed2554dc4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to