From ae62a4348f7de53cfb91a5ae25a10723d5f9b398 Mon Sep 17 00:00:00 2001 From: Jan Gelety Date: Tue, 3 Sep 2019 22:18:17 +0200 Subject: CSIT-1597 API cleanup: tap cover API changes in VPP: https://gerrit.fd.io/r/c/vpp/+/21706 Change-Id: I16b3e3e29e7e7491cd19c067e8f3d8ced4947852 Signed-off-by: Jan Gelety --- resources/libraries/python/Tap.py | 56 ++++++++++++++++++++++++--------------- 1 file changed, 34 insertions(+), 22 deletions(-) (limited to 'resources/libraries/python') diff --git a/resources/libraries/python/Tap.py b/resources/libraries/python/Tap.py index 103eeffd9e..50e065f9b5 100644 --- a/resources/libraries/python/Tap.py +++ b/resources/libraries/python/Tap.py @@ -13,16 +13,22 @@ """Tap utilities library.""" -from ipaddress import ip_address +from enum import IntEnum + from robot.api import logger from resources.libraries.python.Constants import Constants -from resources.libraries.python.L2Util import L2Util from resources.libraries.python.InterfaceUtil import InterfaceUtil +from resources.libraries.python.L2Util import L2Util from resources.libraries.python.PapiExecutor import PapiSocketExecutor from resources.libraries.python.topology import Topology +class TapFlags(IntEnum): + """TAP interface flags.""" + TAP_FLAG_GSO = 1 + + class Tap(object): """Tap utilities.""" @@ -44,17 +50,18 @@ class Tap(object): cmd = 'tap_create_v2' args = dict( id=Constants.BITWISE_NON_ZERO, - use_random_mac=0 if mac else 1, - mac_address=L2Util.mac_to_bin(mac) if mac else 6 * b'\x00', - host_namespace=64 * b'\x00', - host_mac_addr=6 * b'\x00', - host_if_name_set=1, - host_if_name=tap_name + (64 - len(tap_name)) * b'\x00', - host_bridge=64 * b'\x00', - host_ip4_addr=4 * b'\x00', - host_ip6_addr=16 * b'\x00', - host_ip4_gw=4 * b'\x00', - host_ip6_gw=16 * b'\x00' + use_random_mac=False if mac else True, + mac_address=L2Util.mac_to_bin(mac) if mac else None, + host_mtu_set=False, + host_mac_addr_set=False, + host_ip4_prefix_set=False, + host_ip6_prefix_set=False, + host_ip4_gw_set=False, + host_ip6_gw_set=False, + host_namespace_set=False, + host_if_name_set=True, + host_bridge_set=False, + host_if_name=tap_name, ) err_msg = 'Failed to create tap interface {tap} on host {host}'.format( tap=tap_name, host=node['host']) @@ -120,14 +127,19 @@ class Tap(object): :returns: Processed tap interface dump. :rtype: dict """ - tap_dump['dev_name'] = tap_dump['dev_name'].rstrip('\x00') - tap_dump['host_if_name'] = tap_dump['host_if_name'].rstrip('\x00') - tap_dump['host_namespace'] = \ - tap_dump['host_namespace'].rstrip('\x00') - tap_dump['host_mac_addr'] = \ - L2Util.bin_to_mac(tap_dump['host_mac_addr']) - tap_dump['host_ip4_addr'] = ip_address(tap_dump['host_ip4_addr']) - tap_dump['host_ip6_addr'] = ip_address(tap_dump['host_ip6_addr']) + tap_dump['host_mac_addr'] = str(tap_dump['host_mac_addr']) + tap_dump['host_ip4_prefix'] = str(tap_dump['host_ip4_prefix']) + tap_dump['host_ip6_prefix'] = str(tap_dump['host_ip6_prefix']) + tap_dump['tap_flags'] = tap_dump['tap_flags'].value \ + if hasattr(tap_dump['tap_flags'], 'value') \ + else int(tap_dump['tap_flags']) + tap_dump['host_namespace'] = None \ + if tap_dump['host_namespace'] == '(nil)' \ + else tap_dump['host_namespace'] + tap_dump['host_bridge'] = None \ + if tap_dump['host_bridge'] == '(nil)' \ + else tap_dump['host_bridge'] + return tap_dump cmd = 'sw_interface_tap_v2_dump' @@ -140,7 +152,7 @@ class Tap(object): for dump in details: if name is None: data.append(process_tap_dump(dump)) - elif dump.get('host_if_name').rstrip('\x00') == name: + elif dump.get('host_if_name') == name: data = process_tap_dump(dump) break -- cgit 1.2.3-korg