aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjose.lausuch <jose.lausuch@ericsson.com>2017-02-14 14:37:05 +0100
committerjose.lausuch <jose.lausuch@ericsson.com>2017-02-14 16:58:05 +0100
commitf169fa760059f91ba68960a7f87ca915e7d0ddaf (patch)
tree6178525e8a96f13613846603489e042f05009947
parent2b191ef958106da17594ee8a4143a392984822b0 (diff)
Fix flake8 violations in releng
Change-Id: I45769ff9c9db2de9fbba69117a92de414dd7f651 Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
-rw-r--r--opnfv/deployment/example.py2
-rw-r--r--opnfv/deployment/manager.py2
-rw-r--r--opnfv/utils/Credentials.py2
-rw-r--r--opnfv/utils/constants.py1
-rw-r--r--opnfv/utils/ovs_logger.py9
-rw-r--r--opnfv/utils/ssh_utils.py8
6 files changed, 13 insertions, 11 deletions
diff --git a/opnfv/deployment/example.py b/opnfv/deployment/example.py
index 6a76eb9..f456832 100644
--- a/opnfv/deployment/example.py
+++ b/opnfv/deployment/example.py
@@ -18,4 +18,4 @@ for node in nodes:
print("Hello, I am node '%s' and my ip is %s." %
(node.run_cmd('hostname'), node.ip))
-print handler.get_deployment_info()
+print(handler.get_deployment_info())
diff --git a/opnfv/deployment/manager.py b/opnfv/deployment/manager.py
index f0e4429..9f77ff0 100644
--- a/opnfv/deployment/manager.py
+++ b/opnfv/deployment/manager.py
@@ -56,7 +56,7 @@ class Deployment(object):
version = self.deployment_info['openstack_version'].split('.')[0]
name = os_versions[version]
return name
- except Exception as e:
+ except Exception:
return 'Unknown release'
def get_dict(self):
diff --git a/opnfv/utils/Credentials.py b/opnfv/utils/Credentials.py
index 6441b84..141ecbd 100644
--- a/opnfv/utils/Credentials.py
+++ b/opnfv/utils/Credentials.py
@@ -77,7 +77,7 @@ class Credentials(object):
creds_file = '/root/openrc'
try:
self.handler.get_file_from_controller(creds_file, target_path)
- except Exception, e:
+ except Exception as e:
self.logger.error(
"Cannot get %s from controller. %e" % (creds_file, e))
pass
diff --git a/opnfv/utils/constants.py b/opnfv/utils/constants.py
index ed83488..56008c3 100644
--- a/opnfv/utils/constants.py
+++ b/opnfv/utils/constants.py
@@ -14,6 +14,7 @@ EXIT_OK = 0
EXIT_RUN_ERROR = -1
EXIT_PUSH_TO_TEST_DB_ERROR = -2
+
class Constants(object):
INSTALLERS = ['apex', 'fuel', 'compass', 'joid', "daisy"]
VERSIONS = ['arno', 'brahmaputra', 'colorado', 'danube']
diff --git a/opnfv/utils/ovs_logger.py b/opnfv/utils/ovs_logger.py
index 3159609..75b4cec 100644
--- a/opnfv/utils/ovs_logger.py
+++ b/opnfv/utils/ovs_logger.py
@@ -16,6 +16,7 @@ logger = OPNFVLogger.Logger('ovs_logger').getLogger()
class OVSLogger(object):
+
def __init__(self, basedir, ft_resdir):
self.ovs_dir = basedir
self.ft_resdir = ft_resdir
@@ -32,7 +33,7 @@ class OVSLogger(object):
hosts = stdout.readline().strip().split(' ')
found_host = [h for h in hosts if h.startswith(host_prefix)][0]
return found_host
- except Exception, e:
+ except Exception as e:
logger.error(e)
def __dump_to_file(self, operation, host, text, timestamp=None):
@@ -55,7 +56,7 @@ class OVSLogger(object):
.format(cmd, host))
output = ''.join(stdout.readlines())
return output
- except Exception, e:
+ except Exception as e:
logger.error('[__remote_command(ssh_client, {0})]: {1}'
.format(cmd, e))
return None
@@ -78,7 +79,7 @@ class OVSLogger(object):
host = self.__ssh_host(ssh_conn)
self.__dump_to_file(operation, host, output, timestamp=timestamp)
return output
- except Exception, e:
+ except Exception as e:
logger.error('[ofctl_dump_flows(ssh_client, {0}, {1})]: {2}'
.format(br, choose_table, e))
return None
@@ -91,7 +92,7 @@ class OVSLogger(object):
host = self.__ssh_host(ssh_conn)
self.__dump_to_file(operation, host, output, timestamp=timestamp)
return output
- except Exception, e:
+ except Exception as e:
logger.error('[vsctl_show(ssh_client)]: {0}'.format(e))
return None
diff --git a/opnfv/utils/ssh_utils.py b/opnfv/utils/ssh_utils.py
index f900455..d17f5ae 100644
--- a/opnfv/utils/ssh_utils.py
+++ b/opnfv/utils/ssh_utils.py
@@ -47,7 +47,7 @@ def get_ssh_client(hostname,
password=password)
return client
- except Exception, e:
+ except Exception as e:
logger.error(e)
return None
@@ -57,7 +57,7 @@ def get_file(ssh_conn, src, dest):
sftp = ssh_conn.open_sftp()
sftp.get(src, dest)
return True
- except Exception, e:
+ except Exception as e:
logger.error("Error [get_file(ssh_conn, '%s', '%s']: %s" %
(src, dest, e))
return None
@@ -68,7 +68,7 @@ def put_file(ssh_conn, src, dest):
sftp = ssh_conn.open_sftp()
sftp.put(src, dest)
return True
- except Exception, e:
+ except Exception as e:
logger.error("Error [put_file(ssh_conn, '%s', '%s']: %s" %
(src, dest, e))
return None
@@ -128,5 +128,5 @@ class ProxyHopClient(paramiko.SSHClient):
pkey=proxy_key,
sock=self.proxy_channel)
os.remove(self.local_ssh_key)
- except Exception, e:
+ except Exception as e:
logger.error(e)