Yedidyah Bar David has posted comments on this change. Change subject: packaging: setup: Allow dwh on separate host ......................................................................
Patch Set 16: (2 comments) http://gerrit.ovirt.org/#/c/27516/16/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py File packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py: Line 119: 'host': '', Line 120: 'port': '5432', Line 121: 'secured': '', Line 122: 'hostValidation': False, Line 123: 'user': '', Why not default to 'engine'? Line 124: 'password': None, Line 125: 'database': '', Line 126: }, Line 127: show_create_msg=False, Line 121: 'secured': '', Line 122: 'hostValidation': False, Line 123: 'user': '', Line 124: 'password': None, Line 125: 'database': '', same Line 126: }, Line 127: show_create_msg=False, Line 128: ) Line 129: -- To view, visit http://gerrit.ovirt.org/27516 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I06fc960481af258b3954a8968be3439393d3ebdb Gerrit-PatchSet: 16 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Shirly Radco <[email protected]> Gerrit-Reviewer: Simone Tiraboschi <[email protected]> Gerrit-Reviewer: Yaniv Dary <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
