Yedidyah Bar David has posted comments on this change. Change subject: packaging: setup: legacy: use postgres db user ......................................................................
Patch Set 4: (1 comment) .................................................... File packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py Line 61: config = configfile.ConfigFile([ Line 62: osetupcons.FileLocations.LEGACY_OVIRT_ENGINE_SYSCONFIG Line 63: ]) Line 64: legacy_user = config.get('ENGINE_DB_USER') Line 65: self.logger.debug('legacy ENGINE_DB_USER: %s' % legacy_user) if ConfigFile is changed to fail if given a non-existing argument, setup will fail. Line 66: with open( Line 67: osetupcons.FileLocations.LEGACY_PSQL_PASS_FILE, Line 68: 'r', Line 69: ) as f: -- To view, visit http://gerrit.ovirt.org/23055 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I391cc532fd1673db9f6adb29fc0cce052992fda4 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <d...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches