X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=resources%2Flibraries%2Fpython%2FSetupFramework.py;h=26a07c3a3869e5695c034821dbaad3c2afcb846e;hp=b3df489685376bb1bf78dbb4ed428b6a7ec8b1cd;hb=1e5030b987404fb1cf1dfabe31a150092ff84a3d;hpb=8f77a1ac982b07802f0fb209f589708c27f3e9c5 diff --git a/resources/libraries/python/SetupFramework.py b/resources/libraries/python/SetupFramework.py index b3df489685..26a07c3a38 100644 --- a/resources/libraries/python/SetupFramework.py +++ b/resources/libraries/python/SetupFramework.py @@ -1,4 +1,4 @@ -# Copyright (c) 2016 Cisco and/or its affiliates. +# Copyright (c) 2021 Cisco and/or its affiliates. # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at: @@ -16,39 +16,49 @@ nodes. All tasks required to be run before the actual tests are started is supposed to end up here. """ -import shlex -from subprocess import Popen, PIPE, call -from multiprocessing import Pool +from os import environ, remove from tempfile import NamedTemporaryFile -from os.path import basename +import threading +import traceback from robot.api import logger -from robot.libraries.BuiltIn import BuiltIn -from resources.libraries.python.ssh import SSH -from resources.libraries.python.constants import Constants as con +from resources.libraries.python.Constants import Constants as con +from resources.libraries.python.ssh import exec_cmd_no_error, scp_node +from resources.libraries.python.LocalExecution import run from resources.libraries.python.topology import NodeType -__all__ = ["SetupFramework"] +__all__ = [u"SetupFramework"] + def pack_framework_dir(): - """Pack the testing WS into temp file, return its name.""" + """Pack the testing WS into temp file, return its name. - tmpfile = NamedTemporaryFile(suffix=".tgz", prefix="openvpp-testing-") - file_name = tmpfile.name - tmpfile.close() + :returns: Tarball file name. + :rtype: str + :raises Exception: When failed to pack testing framework. + """ - proc = Popen( - shlex.split("tar --exclude-vcs -zcf {0} .".format(file_name)), - stdout=PIPE, stderr=PIPE) - (stdout, stderr) = proc.communicate() + try: + directory = environ[u"TMPDIR"] + except KeyError: + directory = None - logger.debug(stdout) - logger.debug(stderr) + if directory is not None: + tmpfile = NamedTemporaryFile( + suffix=u".tgz", prefix=u"csit-testing-", dir=f"{directory}" + ) + else: + tmpfile = NamedTemporaryFile(suffix=u".tgz", prefix=u"csit-testing-") + file_name = tmpfile.name + tmpfile.close() - return_code = proc.wait() - if 0 != return_code: - raise Exception("Could not pack testing framework.") + run( + [ + u"tar", u"--sparse", u"--exclude-vcs", u"--exclude=output*.xml", + u"--exclude=./tmp", u"-zcf", file_name, u"." + ], msg=u"Could not pack testing framework" + ) return file_name @@ -56,87 +66,196 @@ def pack_framework_dir(): def copy_tarball_to_node(tarball, node): """Copy tarball file from local host to remote node. - :param tarball: path to tarball to upload - :param node: dictionary created from topology - :type tarball: string + :param tarball: Path to tarball to upload. + :param node: Dictionary created from topology. + :type tarball: str :type node: dict - :return: nothing + :returns: nothing """ - logger.console('Copying tarball to {0}'.format(node['host'])) - ssh = SSH() - ssh.connect(node) - - ssh.scp(tarball, "/tmp/") + logger.console( + f"Copying tarball to {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} starts." + ) + scp_node(node, tarball, u"/tmp/") + logger.console( + f"Copying tarball to {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} done." + ) def extract_tarball_at_node(tarball, node): """Extract tarball at given node. - Extracts tarball using tar on given node to specific CSIT loocation. + Extracts tarball using tar on given node to specific CSIT location. - :param tarball: path to tarball to upload - :param node: dictionary created from topology - :type tarball: string + :param tarball: Path to tarball to upload. + :param node: Dictionary created from topology. + :type tarball: str :type node: dict - :return: nothing + :returns: nothing + :raises RuntimeError: When failed to unpack tarball. """ - logger.console('Extracting tarball to {0} on {1}'.format( - con.REMOTE_FW_DIR, node['host'])) - ssh = SSH() - ssh.connect(node) - - cmd = 'sudo rm -rf {1}; mkdir {1} ; tar -zxf {0} -C {1}; ' \ - 'rm -f {0}'.format(tarball, con.REMOTE_FW_DIR) - (ret_code, _, stderr) = ssh.exec_command(cmd, timeout=30) - if 0 != ret_code: - logger.error('Unpack error: {0}'.format(stderr)) - raise Exception('Failed to unpack {0} at node {1}'.format( - tarball, node['host'])) + logger.console( + f"Extracting tarball to {con.REMOTE_FW_DIR} on {node[u'type']} " + f"host {node[u'host']}, port {node[u'port']} starts." + ) + cmd = f"sudo rm -rf {con.REMOTE_FW_DIR}; mkdir {con.REMOTE_FW_DIR}; " \ + f"tar -zxf {tarball} -C {con.REMOTE_FW_DIR}; rm -f {tarball}" + exec_cmd_no_error( + node, cmd, + message=f"Failed to extract {tarball} at node {node[u'type']} " + f"host {node[u'host']}, port {node[u'port']}", + timeout=90, include_reason=True + ) + logger.console( + f"Extracting tarball to {con.REMOTE_FW_DIR} on {node[u'type']} " + f"host {node[u'host']}, port {node[u'port']} done." + ) def create_env_directory_at_node(node): - """Create fresh virtualenv to a directory, install pip requirements.""" - logger.console('Extracting virtualenv, installing requirements.txt ' - 'on {0}'.format(node['host'])) - ssh = SSH() - ssh.connect(node) - (ret_code, stdout, stderr) = ssh.exec_command( - 'cd {0} && rm -rf env && virtualenv env && . env/bin/activate && ' - 'pip install -r requirements.txt'.format(con.REMOTE_FW_DIR), - timeout=100) - if 0 != ret_code: - logger.error('Virtualenv creation error: {0}'.format(stdout + stderr)) - raise Exception('Virtualenv setup failed') - else: - logger.console('Virtualenv created on {0}'.format(node['host'])) - -def setup_node(args): - """Run all set-up methods for a node. + """Create fresh virtualenv to a directory, install pip requirements. - This method is used as map_async parameter. It receives tuple with all - parameters as passed to map_async function. + Return stdout and stderr of the command, + so we see which installs are behaving weird (e.g. attempting download). - :param args: all parameters needed to setup one node - :type args: tuple - :return: nothing + :param node: Node to create virtualenv on. + :type node: dict + :returns: Stdout and stderr. + :rtype: str, str + :raises RuntimeError: When failed to setup virtualenv. + """ + logger.console( + f"Virtualenv setup including requirements.txt on {node[u'type']} " + f"host {node[u'host']}, port {node[u'port']} starts." + ) + cmd = f"cd {con.REMOTE_FW_DIR} && rm -rf env && virtualenv " \ + f"-p $(which python3) --system-site-packages --never-download env " \ + f"&& source env/bin/activate && ANSIBLE_SKIP_CONFLICT_CHECK=1 " \ + f"pip3 install -r requirements.txt" + stdout, stderr = exec_cmd_no_error( + node, cmd, timeout=100, include_reason=True, + message=f"Failed install at node {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']}" + ) + logger.console( + f"Virtualenv setup on {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} done." + ) + return stdout, stderr + + +def setup_node(node, tarball, remote_tarball, results=None, logs=None): + """Copy a tarball to a node and extract it. + + :param node: A node where the tarball will be copied and extracted. + :param tarball: Local path of tarball to be copied. + :param remote_tarball: Remote path of the tarball. + :param results: A list where to store the result of node setup, optional. + :param logs: A list where to store anything that should be logged. + :type node: dict + :type tarball: str + :type remote_tarball: str + :type results: list + :type logs: list + :returns: True - success, False - error + :rtype: bool """ - tarball, remote_tarball, node = args - copy_tarball_to_node(tarball, node) - extract_tarball_at_node(remote_tarball, node) - if node['type'] == NodeType.TG: - create_env_directory_at_node(node) - logger.console('Setup of node {0} done'.format(node['host'])) + try: + copy_tarball_to_node(tarball, node) + extract_tarball_at_node(remote_tarball, node) + if node[u"type"] == NodeType.TG: + stdout, stderr = create_env_directory_at_node(node) + if isinstance(logs, list): + logs.append(f"{node[u'host']} Env stdout: {stdout}") + logs.append(f"{node[u'host']} Env stderr: {stderr}") + except Exception: + # any exception must result in result = False + # since this runs in a thread and can't be caught anywhere else + err_msg = f"Node {node[u'type']} host {node[u'host']}, " \ + f"port {node[u'port']} setup failed." + logger.console(err_msg) + if isinstance(logs, list): + logs.append(f"{err_msg} Exception: {traceback.format_exc()}") + result = False + else: + logger.console( + f"Setup of node {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} done." + ) + result = True + + if isinstance(results, list): + results.append(result) + return result + def delete_local_tarball(tarball): """Delete local tarball to prevent disk pollution. - :param tarball: path to tarball to upload - :type tarball: string - :return: nothing + :param tarball: Path of local tarball to delete. + :type tarball: str + :returns: nothing + """ + remove(tarball) + + +def delete_framework_dir(node): + """Delete framework directory in /tmp/ on given node. + + :param node: Node to delete framework directory on. + :type node: dict """ - call(shlex.split('sh -c "rm {0} > /dev/null 2>&1"'.format(tarball))) + logger.console( + f"Deleting framework directory on {node[u'type']} host {node[u'host']}," + f" port {node[u'port']} starts." + ) + exec_cmd_no_error( + node, f"sudo rm -rf {con.REMOTE_FW_DIR}", + message=f"Framework delete failed at node {node[u'type']} " + f"host {node[u'host']}, port {node[u'port']}", + timeout=100, include_reason=True + ) + logger.console( + f"Deleting framework directory on {node[u'type']} host {node[u'host']}," + f" port {node[u'port']} done." + ) + + +def cleanup_node(node, results=None, logs=None): + """Delete a tarball from a node. + + :param node: A node where the tarball will be delete. + :param results: A list where to store the result of node cleanup, optional. + :param logs: A list where to store anything that should be logged. + :type node: dict + :type results: list + :type logs: list + :returns: True - success, False - error + :rtype: bool + """ + try: + delete_framework_dir(node) + except Exception: + err_msg = f"Cleanup of node {node[u'type']} host {node[u'host']}, " \ + f"port {node[u'port']} failed." + logger.console(err_msg) + if isinstance(logs, list): + logs.append(f"{err_msg} Exception: {traceback.format_exc()}") + result = False + else: + logger.console( + f"Cleanup of node {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} done." + ) + result = True -class SetupFramework(object): # pylint: disable=too-few-public-methods + if isinstance(results, list): + results.append(result) + return result + + +class SetupFramework: """Setup suite run on topology nodes. Many VAT/CLI based tests need the scripts at remote hosts before executing @@ -144,36 +263,90 @@ class SetupFramework(object): # pylint: disable=too-few-public-methods to all nodes in topology under /tmp/ """ - def __init__(self): - pass - @staticmethod def setup_framework(nodes): - """Pack the whole directory and extract in temp on each node.""" + """Pack the whole directory and extract in temp on each node. + + :param nodes: Topology nodes. + :type nodes: dict + :raises RuntimeError: If setup framework failed. + """ tarball = pack_framework_dir() - msg = 'Framework packed to {0}'.format(tarball) + msg = f"Framework packed to {tarball}" logger.console(msg) logger.trace(msg) - remote_tarball = "/tmp/{0}".format(basename(tarball)) + remote_tarball = f"{tarball}" + + results = list() + logs = list() + threads = list() - # Turn off loggining since we use multiprocessing - log_level = BuiltIn().set_log_level('NONE') - params = ((tarball, remote_tarball, node) for node in nodes.values()) - pool = Pool(processes=len(nodes)) - result = pool.map_async(setup_node, params) - pool.close() - pool.join() + for node in nodes.values(): + args = node, tarball, remote_tarball, results, logs + thread = threading.Thread(target=setup_node, args=args) + thread.start() + threads.append(thread) logger.info( - 'Executed node setups in parallel, waiting for processes to end') - result.wait() + u"Executing node setups in parallel, waiting for threads to end." + ) + + for thread in threads: + thread.join() - logger.info('Results: {0}'.format(result.get())) + logger.info(f"Results: {results}") + + for log in logs: + logger.trace(log) - # Turn on loggining - BuiltIn().set_log_level(log_level) - logger.trace('Test framework copied to all topology nodes') delete_local_tarball(tarball) - logger.console('All nodes are ready') + if all(results): + logger.console(u"All nodes are ready.") + for node in nodes.values(): + logger.info( + f"Setup of node {node[u'type']} host {node[u'host']}, " + f"port {node[u'port']} done." + ) + else: + raise RuntimeError(u"Failed to setup framework.") + + +class CleanupFramework: + """Clean up suite run on topology nodes.""" + + @staticmethod + def cleanup_framework(nodes): + """Perform cleanup on each node. + + :param nodes: Topology nodes. + :type nodes: dict + :raises RuntimeError: If cleanup framework failed. + """ + + results = list() + logs = list() + threads = list() + + for node in nodes.values(): + thread = threading.Thread(target=cleanup_node, + args=(node, results, logs)) + thread.start() + threads.append(thread) + + logger.info( + u"Executing node cleanups in parallel, waiting for threads to end." + ) + + for thread in threads: + thread.join() + + logger.info(f"Results: {results}") + + for log in logs: + logger.trace(log) + if all(results): + logger.console(u"All nodes cleaned up.") + else: + raise RuntimeError(u"Failed to cleaned up framework.")