diff options
author | Vratko Polak <vrpolak@cisco.com> | 2019-07-10 16:18:26 +0200 |
---|---|---|
committer | Peter Mikus <pmikus@cisco.com> | 2019-07-10 20:16:59 +0000 |
commit | 7b0e8a9dee0f874c4b7317e9132064dbab8fd34a (patch) | |
tree | 6363b0625c9b00a1e9eea0bd45d027353dc464f2 | |
parent | 82094363f6077e1b28845719db3a6191c0c93a99 (diff) |
Fix few pylint violations
+ Stop attempting to check test/ as module inits are not there.
Change-Id: Ia4e498061be3e3118b07e98c9c2f761f2454653e
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
-rw-r--r-- | resources/libraries/bash/entry/check/pylint.sh | 2 | ||||
-rw-r--r-- | resources/libraries/python/IPUtil.py | 8 | ||||
-rw-r--r-- | resources/libraries/python/IPsecUtil.py | 4 | ||||
-rw-r--r-- | resources/libraries/python/TestConfig.py | 6 |
4 files changed, 9 insertions, 11 deletions
diff --git a/resources/libraries/bash/entry/check/pylint.sh b/resources/libraries/bash/entry/check/pylint.sh index a3cad9a0d7..919e140f28 100644 --- a/resources/libraries/bash/entry/check/pylint.sh +++ b/resources/libraries/bash/entry/check/pylint.sh @@ -29,7 +29,7 @@ source "${BASH_FUNCTION_DIR}/common.sh" || { echo "Source failed." >&2 exit 1 } -pylint_args=("--rcfile=pylint.cfg" "resources/" "tests/") +pylint_args=("--rcfile=pylint.cfg" "resources/") if pylint "${pylint_args[@]}" > "pylint.log"; then warn warn "Pylint checker: PASS" diff --git a/resources/libraries/python/IPUtil.py b/resources/libraries/python/IPUtil.py index 17ba75d7fb..2dafebf5f6 100644 --- a/resources/libraries/python/IPUtil.py +++ b/resources/libraries/python/IPUtil.py @@ -161,13 +161,13 @@ class IPUtil(object): :param node: VPP node. :type node: dict """ - ip = ip_address(unicode(address)) + addr = ip_address(unicode(address)) PapiExecutor.run_cli_cmd( node, 'show {ip_ver} fib {addr}/{addr_len}'.format( - ip_ver='ip6' if ip.version == 6 else 'ip', - addr=ip, - addr_len=ip.max_prefixlen)) + ip_ver='ip6' if addr.version == 6 else 'ip', + addr=addr, + addr_len=addr.max_prefixlen)) @staticmethod def get_interface_vrf_table(node, interface, ip_version='ipv4'): diff --git a/resources/libraries/python/IPsecUtil.py b/resources/libraries/python/IPsecUtil.py index 123043828d..1a59172958 100644 --- a/resources/libraries/python/IPsecUtil.py +++ b/resources/libraries/python/IPsecUtil.py @@ -705,9 +705,7 @@ class IPsecUtil(object): remote_crypto_key=ckey, integ=integ, laddr=ip_address(unicode(if1_ip_addr)) + i * addr_incr, - raddr=ip_address(unicode(if2_ip_addr)), - uifc=Topology.get_interface_name(nodes['DUT1'], - if1_key))) + raddr=ip_address(unicode(if2_ip_addr)))) tmp_f2.write( 'ipsec_tunnel_if_add_del ' 'local_spi {local_spi} ' diff --git a/resources/libraries/python/TestConfig.py b/resources/libraries/python/TestConfig.py index 1f8af5d457..a5bd5650a2 100644 --- a/resources/libraries/python/TestConfig.py +++ b/resources/libraries/python/TestConfig.py @@ -132,11 +132,11 @@ class TestConfig(object): ip=src_ip, ip_len=128 if src_ip.version == 6 else 32)) commands.append( - 'vxlan_add_del_tunnel src {src_ip} dst {dst_ip} vni {vni}\n' - .format(src_ip=src_ip, dst_ip=dst_ip, + 'vxlan_add_del_tunnel src {srcip} dst {dstip} vni {vni}\n'\ + .format(srcip=src_ip, dstip=dst_ip, vni=vni_start + i)) commands.append( - 'create_vlan_subif sw_if_index {sw_idx} vlan {vlan}\n' + 'create_vlan_subif sw_if_index {sw_idx} vlan {vlan}\n'\ .format(sw_idx=Topology.get_interface_sw_index( node, node_vlan_if), vlan=i + 1)) VatExecutor().write_and_execute_script(node, tmp_fn, commands) |