Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: setup: pep8 fixes
......................................................................

packaging: setup: pep8 fixes

One is minor, other is real - missing import. Affects only certain kinds
of failures, therefore not noticed in testing.

Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Signed-off-by: Yedidyah Bar David <d...@redhat.com>
---
M 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
2 files changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-dwh refs/changes/92/34992/1

diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
index 36cfe81..b6c1814 100644
--- 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
+++ 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
@@ -32,6 +32,7 @@
 from ovirt_engine import configfile
 
 
+from ovirt_engine_setup import constants as osetupcons
 from ovirt_engine_setup.dwh import constants as odwhcons
 from ovirt_engine_setup.engine_common import database
 from ovirt_engine_setup.engine_common \
diff --git 
a/packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py 
b/packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
index b692b7e..261af50 100644
--- 
a/packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
+++ 
b/packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
@@ -64,8 +64,8 @@
                 self.logger.debug('_ is %s' % _)
                 self.logger.warning(
                     _(
-                        'Not updating engine database to disconnect from dwh - 
'
-                        'seems like a different dwh was already setup for it'
+                        'Not updating engine database to disconnect from dwh '
+                        '- seems like a different dwh was already setup for it'
                     )
                 )
             else:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
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

Reply via email to