Yedidyah Bar David has posted comments on this change. Change subject: packaging: setup: fix ugprade with remote db ......................................................................
Patch Set 1: (1 comment) .................................................... File packaging/legacy-setup/ovirt-engine-dwh-setup.py Line 511: if not utils.localHost(db_dict["host"]): Line 512: # remote Line 513: if hasData: Line 514: # upgrade Line 515: db_dict = working_db_dict This was a bug, because working_db_dict did not have everything - e.g. "engine_db". I think we need such a fix also for reports, now pushing. Line 516: else: Line 517: print 'Remote installation is selected.\n' Line 518: Line 519: dbExists, tmpowned, tmphasData = utils.dbExists(db_dict, PGPASS_TEMP) -- To view, visit http://gerrit.ovirt.org/22961 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibd68ae47e3b1b21b9cb79e0c351706d166941c9a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: Ofer Schreiber <[email protected]> Gerrit-Reviewer: Yaniv Dary <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
