From e7ae1fec8e378fa7805d6f2034df14b98ea0b850 Mon Sep 17 00:00:00 2001 From: Alexey Stepanov Date: Mon, 14 Mar 2016 15:11:29 +0300 Subject: [PATCH] Pylint: fix trivial issues 4 Change-Id: I512c6faa3870b62a1516fd7aa5f5dd0979f745c7 fix: dangerous default values Related-bug: #1556791 --- fuelweb_test/helpers/utils.py | 4 +++- fuelweb_test/tests/test_net_templates_base.py | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/fuelweb_test/helpers/utils.py b/fuelweb_test/helpers/utils.py index 5ad05b85e..a1e980ec4 100644 --- a/fuelweb_test/helpers/utils.py +++ b/fuelweb_test/helpers/utils.py @@ -496,7 +496,9 @@ def get_network_template(template_name): @logwrap -def get_net_settings(remote, skip_interfaces=set()): +def get_net_settings(remote, skip_interfaces=None): + if skip_interfaces is None: + skip_interfaces = set() net_settings = dict() interface_cmd = ('awk \'$1~/:/{split($1,iface,":"); print iface[1]}\'' ' /proc/net/dev') diff --git a/fuelweb_test/tests/test_net_templates_base.py b/fuelweb_test/tests/test_net_templates_base.py index c32ee847f..201dcf542 100644 --- a/fuelweb_test/tests/test_net_templates_base.py +++ b/fuelweb_test/tests/test_net_templates_base.py @@ -83,7 +83,9 @@ class TestNetworkTemplatesBase(TestBasic): @staticmethod @logwrap def get_template_ep_for_role(template, role, nodegroup='default', - skip_net_roles=set()): + skip_net_roles=None): + if skip_net_roles is None: + skip_net_roles = set() tmpl = template['adv_net_template'][nodegroup] endpoints = set() networks = set()