From 33fb34665214bbbd0a4b3154169b21c2da01f69b Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Tue, 9 Jul 2019 12:17:09 +0200 Subject: PapiExecutor always verifies Do not support returning unverified replies anymore. Basically, ".get_replies().verify_replies()" is now just ".get_replies()". This allows fairly large simplifications both at call sites and in PapiExecutor.py + Rename get_dumps to get_details. + Introduce get_reply and get_sw_if_index. + Rename variables holding get_*() value, + e.g. get_stats() value is stored to variable named "stats". + Rename "item" of subsequent loop to hint the type instead. + Rename "details" function argument to "verbose". + Process reply details in place, instead of building new list. - Except hybrid blocks which can return both list or single item. - Except human readable text building blocks. + Rename most similar names to sw_if_index. - Except "vpp_sw_index" and some function names. + Use single run_cli_cmd from PapiExecutor. + Do not chain methods over multiple lines. + Small space gain is not worth readability loss. + Include minor code and docstrings improvement. + Add some TODOs. Change-Id: Ib2110a3d2101a74d5837baab3a58dc46aafc6ce3 Signed-off-by: Vratko Polak --- resources/libraries/python/VhostUser.py | 32 ++++++++++++-------------------- 1 file changed, 12 insertions(+), 20 deletions(-) (limited to 'resources/libraries/python/VhostUser.py') diff --git a/resources/libraries/python/VhostUser.py b/resources/libraries/python/VhostUser.py index 0c7a79e266..916d0829ee 100644 --- a/resources/libraries/python/VhostUser.py +++ b/resources/libraries/python/VhostUser.py @@ -33,21 +33,17 @@ class VhostUser(object): response. :rtype: list """ + cmd = "sw_interface_vhost_user_dump" with PapiExecutor(node) as papi_exec: - dump = papi_exec.add("sw_interface_vhost_user_dump").get_dump() + details = papi_exec.add(cmd).get_details() - key = "sw_interface_vhost_user_details" - data = list() - for item in dump.reply[0]["api_reply"]: - item[key]["interface_name"] = \ - item[key]["interface_name"].rstrip('\x00') - item[key]["sock_filename"] = \ - item[key]["sock_filename"].rstrip('\x00') - data.append(item) + for vhost in details: + vhost["interface_name"] = vhost["interface_name"].rstrip('\x00') + vhost["sock_filename"] = vhost["sock_filename"].rstrip('\x00') - logger.debug("VhostUser data:\n{data}".format(data=data)) + logger.debug("VhostUser details:\n{details}".format(details=details)) - return data + return details @staticmethod def vpp_create_vhost_user_interface(node, socket): @@ -67,25 +63,21 @@ class VhostUser(object): sock_filename=str(socket) ) with PapiExecutor(node) as papi_exec: - data = papi_exec.add(cmd, **args).get_replies(err_msg).\ - verify_reply(err_msg=err_msg) - - # Extract sw_if_idx: - sw_if_idx = data["sw_if_index"] + sw_if_index = papi_exec.add(cmd, **args).get_sw_if_index(err_msg) # Update the Topology: if_key = Topology.add_new_port(node, 'vhost') - Topology.update_interface_sw_if_index(node, if_key, sw_if_idx) + Topology.update_interface_sw_if_index(node, if_key, sw_if_index) - ifc_name = InterfaceUtil.vpp_get_interface_name(node, sw_if_idx) + ifc_name = InterfaceUtil.vpp_get_interface_name(node, sw_if_index) Topology.update_interface_name(node, if_key, ifc_name) - ifc_mac = InterfaceUtil.vpp_get_interface_mac(node, sw_if_idx) + ifc_mac = InterfaceUtil.vpp_get_interface_mac(node, sw_if_index) Topology.update_interface_mac_address(node, if_key, ifc_mac) Topology.update_interface_vhost_socket(node, if_key, socket) - return sw_if_idx + return sw_if_index @staticmethod def get_vhost_user_if_name_by_sock(node, socket): -- cgit 1.2.3-korg