Yedidyah Bar David has uploaded a new change for review. Change subject: packaging: setup: legacy: use postgres db user ......................................................................
packaging: setup: legacy: use postgres db user If legacy pgpass has only one line with 'postgres', do use it. Bug-Url: https://bugzilla.redhat.com/1049654 Change-Id: I391cc532fd1673db9f6adb29fc0cce052992fda4 Signed-off-by: Yedidyah Bar David <d...@redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py 1 file changed, 25 insertions(+), 25 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/55/23055/1 diff --git a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py index c4d7205..e1d6490 100644 --- a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py +++ b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/database.py @@ -61,32 +61,32 @@ ) as f: for l in f: l = l.rstrip('\n') - if ':%s:' % 'postgres' not in l: - d = l.split(':') - if len(d) == 5: - self._dbenv = { - osetupcons.DBEnv.HOST: d[0], - osetupcons.DBEnv.PORT: int(d[1]), - osetupcons.DBEnv.SECURED: None, - osetupcons.DBEnv.SECURED_HOST_VALIDATION: None, - osetupcons.DBEnv.DATABASE: ( - d[2] if d[2] != '*' - else - osetupcons.Defaults.DEFAULT_DB_DATABASE - ), - osetupcons.DBEnv.USER: d[3], - osetupcons.DBEnv.PASSWORD: d[4], - osetupcons.DBEnv.NEW_DATABASE: False, - } - self.environment[ - osetupcons.CoreEnv.LEGACY_PG_CREDS_FOUND - ] = True + d = l.split(':') + if len(d) == 5: + self._dbenv = { + osetupcons.DBEnv.HOST: d[0], + osetupcons.DBEnv.PORT: int(d[1]), + osetupcons.DBEnv.SECURED: None, + osetupcons.DBEnv.SECURED_HOST_VALIDATION: None, + osetupcons.DBEnv.DATABASE: ( + d[2] if d[2] != '*' + else + osetupcons.Defaults.DEFAULT_DB_DATABASE + ), + osetupcons.DBEnv.USER: d[3], + osetupcons.DBEnv.PASSWORD: d[4], + osetupcons.DBEnv.NEW_DATABASE: False, + } + self.environment[ + osetupcons.CoreEnv.LEGACY_PG_CREDS_FOUND + ] = True - # In legacy we always have engine user defined - # before the rhevm user, as that's how we - # upgraded from 3.0; in other installations - # we will not have more than 1 user (engine) - # As such, we should use the first user we find. + # In legacy we always have engine user defined + # before the rhevm user, as that's how we + # upgraded from 3.0; in other installations + # we will not have more than 1 user (engine) + # As such, we should use the first user we find. + if self._dbenv[osetupcons.DBEnv.USER] != 'postgres': break @plugin.event( -- To view, visit http://gerrit.ovirt.org/23055 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I391cc532fd1673db9f6adb29fc0cce052992fda4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <d...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
