diff options
author | Peter Mikus <pmikus@cisco.com> | 2019-03-08 15:23:48 +0000 |
---|---|---|
committer | Peter Mikus <pmikus@cisco.com> | 2019-03-18 20:14:02 +0000 |
commit | 22417d56aa83eaced43c4ef4473e38a34fdb4dfa (patch) | |
tree | e6942f12138e8bf1478b64d7bd2fed9c7adf0169 /resources/libraries | |
parent | 6aaecf9f7fe3f0ad0d9bbad20e4853472d3e72df (diff) |
CSIT-1386 KernelVM - Part I
- Refactor QemuUtils
Full refactor of QemuUtils to prepare for thinner VMs.
Change-Id: I171ba11a9e6e9faec582bd58df6819598b0ed0b5
Signed-off-by: Peter Mikus <pmikus@cisco.com>
Diffstat (limited to 'resources/libraries')
7 files changed, 459 insertions, 687 deletions
diff --git a/resources/libraries/python/QemuUtils.py b/resources/libraries/python/QemuUtils.py index 6fad6ed8b7..74965ec580 100644 --- a/resources/libraries/python/QemuUtils.py +++ b/resources/libraries/python/QemuUtils.py @@ -15,157 +15,221 @@ from time import time, sleep import json -import re +from re import match # Disable due to pylint bug # pylint: disable=no-name-in-module,import-error from distutils.version import StrictVersion from robot.api import logger - -from resources.libraries.python.ssh import SSH, SSHTimeout +from resources.libraries.python.ssh import exec_cmd, exec_cmd_no_error from resources.libraries.python.Constants import Constants from resources.libraries.python.DUTSetup import DUTSetup from resources.libraries.python.topology import NodeType, Topology +__all__ = ["QemuOptions", "QemuUtils"] -class QemuUtils(object): - """QEMU utilities.""" - def __init__(self, qemu_id=1): - self._qemu_id = qemu_id - self._vhost_id = 0 - self._ssh = None - self._node = None - # Qemu Options - self._qemu_opt = {} - # Path to QEMU binary. Use x86_64 by default - self._qemu_opt['qemu_path'] = '/usr/bin/' - self._qemu_opt['qemu_bin'] = 'qemu-system-x86_64' - # QEMU Machine Protocol socket - self._qemu_opt['qmp_sock'] = '/tmp/qmp{0}.sock'.format(self._qemu_id) - # QEMU Guest Agent socket - self._qemu_opt['qga_sock'] = '/tmp/qga{0}.sock'.format(self._qemu_id) - # QEMU PID file - self._qemu_opt['pid_file'] = '/tmp/qemu{0}.pid'.format(self._qemu_id) - # Default 1 CPU. - self._qemu_opt['smp'] = '-smp 1,sockets=1,cores=1,threads=1' - # Daemonize the QEMU process after initialization. Default one - # management interface. - self._qemu_opt['options'] = '-cpu host -daemonize -enable-kvm ' \ - '-machine pc,accel=kvm,usb=off,mem-merge=off ' \ - '-net nic,macaddr=52:54:00:00:{0:02x}:ff -balloon none'\ - .format(self._qemu_id) - self._qemu_opt['ssh_fwd_port'] = 10021 + qemu_id - # Default serial console port - self._qemu_opt['serial_port'] = 4555 + qemu_id - # Default 512MB virtual RAM - self._qemu_opt['mem_size'] = 512 - # Default huge page mount point, required for Vhost-user interfaces. - self._qemu_opt['huge_mnt'] = '/mnt/huge' - # Default do not allocate huge pages. - self._qemu_opt['huge_allocate'] = False - # Default image for CSIT virl setup - self._qemu_opt['disk_image'] = '/var/lib/vm/vhost-nested.img' - # Virtio queue count - self._qemu_opt['queue_count'] = 1 - # Virtio queue size - self._qemu_opt['queue_size'] = None - # VM node info dict - self._vm_info = { - 'type': NodeType.VM, - 'port': self._qemu_opt['ssh_fwd_port'], - 'username': 'cisco', - 'password': 'cisco', - 'interfaces': {}, - } - # Qemu Sockets - self._socks = [self._qemu_opt.get('qmp_sock'), - self._qemu_opt.get('qga_sock')] +class QemuOptions(object): + """QEMU option class. - def qemu_set_path(self, path): - """Set binary path for QEMU. + The class can handle input parameters or parameters that acts as QEMU + command line parameters. The only variable is a list of dictionaries + where dictionaries can be added multiple times. This emulates the QEMU + behavior where one command line parameter can be used multiple times (1..N). + Example can be device or object (so it is not an issue to have one memory + block of 2G and and second memory block of 512M but from other numa). - :param path: Absolute path in filesystem. - :type path: str - """ - self._qemu_opt['qemu_path'] = path + Class does support get value or string representation that will return + space separated, dash prefixed string of key value pairs used for command + line. + """ - def qemu_set_queue_count(self, count): - """Set number of virtio queues. + # Use one instance of class per tests. + ROBOT_LIBRARY_SCOPE = 'TEST CASE' - :param count: Number of virtio queues. - :type count: int - """ - self._qemu_opt['queue_count'] = int(count) + def __init__(self): + self.variables = list() - def qemu_set_queue_size(self, size): - """Set RX/TX size of virtio queues. + def add(self, variable, value): + """Add parameter to the list. - :param size: Size of virtio queues. - :type size: int + :param variable: QEMU parameter. + :param value: Parameter value. + :type variable: str + :type value: str or int """ - self._qemu_opt['queue_size'] = int(size) + self.variables.append({str(variable): value}) - def qemu_set_smp(self, smp, cores, threads, sockets): - """Set SMP option for QEMU. + def get(self, variable): + """Get parameter of variable(s) from list that matches input value. - :param smp: Number of CPUs. - :param cores: Number of CPU cores on one socket. - :param threads: Number of threads on one CPU core. - :param sockets: Number of discrete sockets in the system. - :type smp: int - :type cores: int - :type threads: int - :type sockets: int + :param variable: QEMU parameter to get. + :type variable: str + :returns: List of values or value that matches input parameter. + :rtype: list or str """ - self._qemu_opt['smp'] = \ - ('-smp {smp},cores={cores},threads={threads},sockets={sockets}'. - format(smp=smp, cores=cores, threads=threads, sockets=sockets)) + selected = [d[variable] for d in self.variables if variable in d] + return selected if len(selected) > 1 else selected[0] - def qemu_set_ssh_fwd_port(self, fwd_port): - """Set host port for guest SSH forwarding. + def get_values(self): + """Get all values from dict items in list. - :param fwd_port: Port number on host for guest SSH forwarding. - :type fwd_port: int + :returns: List of all dictionary values. + :rtype: list """ - self._qemu_opt['ssh_fwd_port'] = fwd_port - self._vm_info['port'] = fwd_port + return [d.values()[0] for d in self.variables] - def qemu_set_serial_port(self, port): - """Set serial console port. + def __str__(self): + """Return space separated string of key value pairs. - :param port: Serial console port. - :type port: int + :returns: Space separated string of key value pairs. + :rtype: str """ - self._qemu_opt['serial_port'] = port + return " ".join(["-{k} {v}".format(k=d.keys()[0], v=d.values()[0]) + for d in self.variables]) - def qemu_set_mem_size(self, mem_size): - """Set virtual RAM size. +class QemuUtils(object): + """QEMU utilities.""" - :param mem_size: RAM size in Mega Bytes. - :type mem_size: int - """ - self._qemu_opt['mem_size'] = int(mem_size) + # Use one instance of class per tests. + ROBOT_LIBRARY_SCOPE = 'TEST CASE' - def qemu_set_huge_mnt(self, huge_mnt): - """Set hugefile mount point. + def __init__(self, node=None, qemu_id=1, smp=1, mem=512, vnf=None, + img='/var/lib/vm/vhost-nested.img', bin_path='/usr/bin'): + """Initialize QemuUtil class. - :param huge_mnt: System hugefile mount point. - :type huge_mnt: int + :param node: Node to run QEMU on. + :param qemu_id: QEMU identifier. + :param smp: Number of virtual SMP units (cores). + :param mem: Amount of memory. + :param vnf: Network function workload. + :param img: QEMU disk image or kernel image path. + :param bin_path: QEMU binary path. + :type node: dict + :type qemu_id: int + :type smp: int + :type mem: int + :type vnf: str + :type img: str + :type bin_path: str """ - self._qemu_opt['huge_mnt'] = huge_mnt + self._vhost_id = 0 + self._vm_info = { + 'type': NodeType.VM, + 'port': 10021 + qemu_id, + 'serial': 4555 + qemu_id, + 'username': 'cisco', + 'password': 'cisco', + 'interfaces': {}, + } + if node: + self.qemu_set_node(node) + # Input Options. + self._opt = QemuOptions() + self._opt.add('qemu_id', qemu_id) + self._opt.add('bin_path', bin_path) + self._opt.add('mem', int(mem)) + self._opt.add('smp', int(smp)) + self._opt.add('img', img) + self._opt.add('vnf', vnf) + # Temporary files. + self._temp = QemuOptions() + self._temp.add('pid', '/var/run/qemu_{id}.pid'.format(id=qemu_id)) + # Computed parameters for QEMU command line. + if '/var/lib/vm/' in img: + self._opt.add('vm_type', 'nestedvm') + self._temp.add('qmp', '/var/run/qmp_{id}.sock'.format(id=qemu_id)) + self._temp.add('qga', '/var/run/qga_{id}.sock'.format(id=qemu_id)) + else: + raise RuntimeError('QEMU: Unknown VM image option!') + self._params = QemuOptions() + self.add_params() + + def add_params(self): + """Set QEMU command line parameters.""" + self.add_default_params() + if self._opt.get('vm_type') == 'nestedvm': + self.add_nestedvm_params() + else: + raise RuntimeError('QEMU: Unsupported VM type!') + + def add_default_params(self): + """Set default QEMU command line parameters.""" + self._params.add('daemonize', '') + self._params.add('nodefaults', '') + self._params.add('name', 'vnf{qemu},debug-threads=on'. + format(qemu=self._opt.get('qemu_id'))) + self._params.add('no-user-config', '') + self._params.add('monitor', 'none') + self._params.add('display', 'none') + self._params.add('vga', 'none') + self._params.add('enable-kvm', '') + self._params.add('pidfile', '{pid}'. + format(pid=self._temp.get('pid'))) + self._params.add('cpu', 'host') + self._params.add('machine', 'pc,accel=kvm,usb=off,mem-merge=off') + self._params.add('smp', '{smp},sockets=1,cores={smp},threads=1'. + format(smp=self._opt.get('smp'))) + self._params.add('object', + 'memory-backend-file,id=mem,size={mem}M,' + 'mem-path=/mnt/huge,share=on'. + format(mem=self._opt.get('mem'))) + self._params.add('m', '{mem}M'. + format(mem=self._opt.get('mem'))) + self._params.add('numa', 'node,memdev=mem') + self._params.add('balloon', 'none') + + def add_nestedvm_params(self): + """Set NestedVM QEMU parameters.""" + self._params.add('net', 'nic,macaddr=52:54:00:00:{qemu:02x}:ff'. + format(qemu=self._opt.get('qemu_id'))) + self._params.add('net', 'user,hostfwd=tcp::{info[port]}-:22'. + format(info=self._vm_info)) + # TODO: Remove try except after fully migrated to Bionic or + # qemu_set_node is removed. + try: + locking = ',file.locking=off'\ + if self.qemu_version(version='2.10') else '' + except AttributeError: + locking = '' + self._params.add('drive', + 'file={img},format=raw,cache=none,if=virtio{locking}'. + format(img=self._opt.get('img'), locking=locking)) + self._params.add('qmp', 'unix:{qmp},server,nowait'. + format(qmp=self._temp.get('qmp'))) + self._params.add('chardev', 'socket,host=127.0.0.1,port={info[serial]},' + 'id=gnc0,server,nowait'.format(info=self._vm_info)) + self._params.add('device', 'isa-serial,chardev=gnc0') + self._params.add('chardev', + 'socket,path={qga},server,nowait,id=qga0'. + format(qga=self._temp.get('qga'))) + self._params.add('device', 'isa-serial,chardev=qga0') + + def qemu_set_node(self, node): + """Set node to run QEMU on. - def qemu_set_huge_allocate(self): - """Set flag to allocate more huge pages if needed.""" - self._qemu_opt['huge_allocate'] = True + :param node: Node to run QEMU on. + :type node: dict + """ + self._node = node + self._vm_info['host'] = node['host'] + if node['port'] != 22: + self._vm_info['host_port'] = node['port'] + self._vm_info['host_username'] = node['username'] + self._vm_info['host_password'] = node['password'] - def qemu_set_disk_image(self, disk_image): - """Set disk image. + def get_qemu_pids(self): + """Get QEMU CPU pids. - :param disk_image: Path of the disk image. - :type disk_image: str + :returns: List of QEMU CPU pids. + :rtype: list of str """ - self._qemu_opt['disk_image'] = disk_image + command = ("grep -rwl 'CPU' /proc/$(sudo cat {pid})/task/*/comm ". + format(pid=self._temp.get('pid'))) + command += (r"| xargs dirname | sed -e 's/\/.*\///g'") + + stdout, _ = exec_cmd_no_error(self._node, command) + return stdout.splitlines() def qemu_set_affinity(self, *host_cpus): """Set qemu affinity by getting thread PIDs via QMP and taskset to list @@ -174,18 +238,22 @@ class QemuUtils(object): :param host_cpus: List of CPU cores. :type host_cpus: list """ - qemu_cpus = self._qemu_qmp_exec('query-cpus')['return'] - - if len(qemu_cpus) != len(host_cpus): - raise ValueError('Host CPU count must match Qemu Thread count') - - for qemu_cpu, host_cpu in zip(qemu_cpus, host_cpus): - ret_code, _, _ = self._ssh.exec_command_sudo( - 'taskset -pc {host_cpu} {thread_id}'. - format(host_cpu=host_cpu, thread_id=qemu_cpu['thread_id'])) - if int(ret_code) != 0: - raise RuntimeError('Set affinity failed on {host}'. - format(host=self._node['host'])) + try: + qemu_cpus = self.get_qemu_pids() + + if len(qemu_cpus) != len(host_cpus): + raise ValueError('Host CPU count must match Qemu Thread count!') + + for qemu_cpu, host_cpu in zip(qemu_cpus, host_cpus): + command = ('taskset -pc {host_cpu} {thread}'. + format(host_cpu=host_cpu, thread=qemu_cpu)) + message = ('QEMU: Set affinity failed on {host}!'. + format(host=self._node['host'])) + exec_cmd_no_error(self._node, command, sudo=True, + message=message) + except (RuntimeError, ValueError): + self.qemu_kill_all() + raise def qemu_set_scheduler_policy(self): """Set scheduler policy to SCHED_RR with priority 1 for all Qemu CPU @@ -193,88 +261,65 @@ class QemuUtils(object): :raises RuntimeError: Set scheduler policy failed. """ - qemu_cpus = self._qemu_qmp_exec('query-cpus')['return'] - - for qemu_cpu in qemu_cpus: - ret_code, _, _ = self._ssh.exec_command_sudo( - 'chrt -r -p 1 {thread_id}'. - format(thread_id=qemu_cpu['thread_id'])) - if int(ret_code) != 0: - raise RuntimeError('Set SCHED_RR failed on {host}'. - format(host=self._node['host'])) - - def qemu_set_node(self, node): - """Set node to run QEMU on. - - :param node: Node to run QEMU on. - :type node: dict - """ - self._node = node - self._ssh = SSH() - self._ssh.connect(node) - self._vm_info['host'] = node['host'] - if node['port'] != 22: - self._vm_info['host_port'] = node['port'] - self._vm_info['host_username'] = node['username'] - self._vm_info['host_password'] = node['password'] - arch = Topology.get_node_arch(node) - self._qemu_opt['qemu_bin'] = 'qemu-system-{arch}'.format(arch=arch) + try: + qemu_cpus = self.get_qemu_pids() + + for qemu_cpu in qemu_cpus: + command = ('chrt -r -p 1 {thread}'. + format(thread=qemu_cpu)) + message = ('QEMU: Set SCHED_RR failed on {host}'. + format(host=self._node['host'])) + exec_cmd_no_error(self._node, command, sudo=True, + message=message) + except (RuntimeError, ValueError): + self.qemu_kill_all() + raise - def qemu_add_vhost_user_if(self, socket, server=True, mac=None, - jumbo_frames=False): + def qemu_add_vhost_user_if(self, socket, server=True, jumbo_frames=False, + queue_size=None, queues=1): """Add Vhost-user interface. :param socket: Path of the unix socket. :param server: If True the socket shall be a listening socket. - :param mac: Vhost-user interface MAC address (optional, otherwise is - used auto-generated MAC 52:54:00:00:xx:yy). :param jumbo_frames: Set True if jumbo frames are used in the test. + :param queue_size: Vring queue size. + :param queues: Number of queues. :type socket: str :type server: bool - :type mac: str :type jumbo_frames: bool + :type queue_size: int + :type queues: int """ self._vhost_id += 1 - # Create unix socket character device. - chardev = (' -chardev socket,id=char{vhost_id},path={socket}{server}'. - format(vhost_id=self._vhost_id, - socket=socket, - server=',server' if server is True else '')) - self._qemu_opt['options'] += chardev - # Create Vhost-user network backend. - netdev = (' -netdev vhost-user,id=vhost{vhost_id},' - 'chardev=char{vhost_id},queues={queue_count}'. - format(vhost_id=self._vhost_id, - queue_count=self._qemu_opt.get('queue_count'))) - self._qemu_opt['options'] += netdev - # If MAC is not specified use auto-generated MAC address based on - # template 52:54:00:00:<qemu_id>:<vhost_id>, e.g. vhost1 MAC of QEMU - # with ID 1 is 52:54:00:00:01:01 - mac = ('52:54:00:00:{qemu_id:02x}:{vhost_id:02x}'. - format(qemu_id=self._qemu_id, vhost_id=self._vhost_id))\ - if mac is None else mac - + self._params.add('chardev', + 'socket,id=char{vhost},path={socket}{server}'. + format(vhost=self._vhost_id, socket=socket, + server=',server' if server is True else '')) + self._params.add('netdev', + 'vhost-user,id=vhost{vhost},' + 'chardev=char{vhost},queues={queues}'. + format(vhost=self._vhost_id, queues=queues)) + mac = ('52:54:00:00:{qemu:02x}:{vhost:02x}'. + format(qemu=self._opt.get('qemu_id'), vhost=self._vhost_id)) queue_size = ('rx_queue_size={queue_size},tx_queue_size={queue_size}'. - format(queue_size=self._qemu_opt.get('queue_size')))\ - if self._qemu_opt.get('queue_size') else '' - vector_size = ('vectors={vectors}'. - format(vectors=2*self._qemu_opt.get('queue_count')+2))\ - if self._qemu_opt.get('queue_count') else '' - - # Create Virtio network device. - device = (' -device virtio-net-pci,netdev=vhost{vhost_id},mac={mac},' - 'mq=on,{vector_size},csum=off,gso=off,guest_tso4=off,' - 'guest_tso6=off,guest_ecn=off,mrg_rxbuf={mbuf},{queue_size}'. - format(vhost_id=self._vhost_id, mac=mac, - mbuf='on,host_mtu=9200' if jumbo_frames else 'off', - queue_size=queue_size, vector_size=vector_size)) - self._qemu_opt['options'] += device - # Add interface MAC and socket to the node dict + format(queue_size=queue_size)) if queue_size else '' + mbuf = 'on,host_mtu=9200' + self._params.add('device', + 'virtio-net-pci,netdev=vhost{vhost},' + 'mac={mac},mq=on,vectors={vectors},csum=off,gso=off,' + 'guest_tso4=off,guest_tso6=off,guest_ecn=off,' + 'mrg_rxbuf={mbuf},{queue_size}'. + format(vhost=self._vhost_id, mac=mac, + mbuf=mbuf if jumbo_frames else 'off', + queue_size=queue_size, + vectors=(2 * queues + 2))) + + # Add interface MAC and socket to the node dict. if_data = {'mac_address': mac, 'socket': socket} - if_name = 'vhost{vhost_id}'.format(vhost_id=self._vhost_id) + if_name = 'vhost{vhost}'.format(vhost=self._vhost_id) self._vm_info['interfaces'][if_name] = if_data - # Add socket to the socket list - self._socks.append(socket) + # Add socket to temporary file list. + self._temp.add(if_name, socket) def _qemu_qmp_exec(self, cmd): """Execute QMP command. @@ -288,14 +333,15 @@ class QemuUtils(object): response will contain the "error" keyword instead of "return". """ # To enter command mode, the qmp_capabilities command must be issued. - ret_code, stdout, _ = self._ssh.exec_command( - 'echo "{{ \\"execute\\": \\"qmp_capabilities\\" }}' - '{{ \\"execute\\": \\"{cmd}\\" }}" | ' - 'sudo -S socat - UNIX-CONNECT:{qmp_sock}'. - format(cmd=cmd, qmp_sock=self._qemu_opt.get('qmp_sock'))) - if int(ret_code) != 0: - raise RuntimeError('QMP execute "{cmd}" failed on {host}'. - format(cmd=cmd, host=self._node['host'])) + command = ('echo "{{ \\"execute\\": \\"qmp_capabilities\\" }}' + '{{ \\"execute\\": \\"{cmd}\\" }}" | ' + 'sudo -S socat - UNIX-CONNECT:{qmp}'. + format(cmd=cmd, qmp=self._temp.get('qmp'))) + message = ('QMP execute "{cmd}" failed on {host}'. + format(cmd=cmd, host=self._node['host'])) + stdout, _ = exec_cmd_no_error(self._node, command, sudo=False, + message=message) + # Skip capabilities negotiation messages. out_list = stdout.splitlines() if len(out_list) < 3: @@ -305,16 +351,14 @@ class QemuUtils(object): def _qemu_qga_flush(self): """Flush the QGA parser state.""" - ret_code, stdout, _ = self._ssh.exec_command( - '(printf "\xFF"; sleep 1) | ' - 'sudo -S socat - UNIX-CONNECT:{qga_sock}'. - format(qga_sock=self._qemu_opt.get('qga_sock'))) - if int(ret_code) != 0: - raise RuntimeError('QGA flush failed on {host}'. - format(host=self._node['host'])) - if not stdout: - return {} - return json.loads(stdout.split('\n', 1)[0]) + command = ('(printf "\xFF"; sleep 1) | ' + 'sudo -S socat - UNIX-CONNECT:{qga}'. + format(qga=self._temp.get('qga'))) + message = ('QGA flush failed on {host}'.format(host=self._node['host'])) + stdout, _ = exec_cmd_no_error(self._node, command, sudo=False, + message=message) + + return json.loads(stdout.split('\n', 1)[0]) if stdout else dict() def _qemu_qga_exec(self, cmd): """Execute QGA command. @@ -324,16 +368,15 @@ class QemuUtils(object): :param cmd: QGA command to execute. :type cmd: str """ - ret_code, stdout, _ = self._ssh.exec_command( - '(echo "{{ \\"execute\\": \\"{cmd}\\" }}"; sleep 1) | ' - 'sudo -S socat - UNIX-CONNECT:{qga_sock}'. - format(cmd=cmd, qga_sock=self._qemu_opt.get('qga_sock'))) - if int(ret_code) != 0: - raise RuntimeError('QGA execute "{cmd}" failed on {host}'. - format(cmd=cmd, host=self._node['host'])) - if not stdout: - return {} - return json.loads(stdout.split('\n', 1)[0]) + command = ('(echo "{{ \\"execute\\": \\"{cmd}\\" }}"; sleep 1) | ' + 'sudo -S socat - UNIX-CONNECT:{qga}'. + format(cmd=cmd, qga=self._temp.get('qga'))) + message = ('QGA execute "{cmd}" failed on {host}'. + format(cmd=cmd, host=self._node['host'])) + stdout, _ = exec_cmd_no_error(self._node, command, sudo=False, + message=message) + + return json.loads(stdout.split('\n', 1)[0]) if stdout else dict() def _wait_until_vm_boot(self, timeout=60): """Wait until QEMU VM is booted. @@ -347,9 +390,8 @@ class QemuUtils(object): start = time() while True: if time() - start > timeout: - raise RuntimeError('timeout, VM {disk} not booted on {host}'. - format(disk=self._qemu_opt['disk_image'], - host=self._node['host'])) + raise RuntimeError('timeout, VM not booted on {host}'. + format(host=self._node['host'])) out = None try: out = self._qemu_qga_flush() @@ -363,23 +405,21 @@ class QemuUtils(object): break while True: if time() - start > timeout: - raise RuntimeError('timeout, VM with {disk} not booted ' - 'on {host}'. - format(disk=self._qemu_opt['disk_image'], - host=self._node['host'])) + raise RuntimeError('timeout, VM not booted on {host}'. + format(host=self._node['host'])) out = None try: out = self._qemu_qga_exec('guest-ping') except ValueError: logger.trace('QGA guest-ping unexpected output {out}'. format(out=out)) - # Empty output - VM not booted yet + # Empty output - VM not booted yet. if not out: sleep(5) - # Non-error return - VM booted + # Non-error return - VM booted. elif out.get('return') is not None: break - # Skip error and wait + # Skip error and wait. elif out.get('error') is not None: sleep(5) else: @@ -388,29 +428,25 @@ class QemuUtils(object): logger.trace('QGA guest-ping unexpected output {out}'. format(out=out)) - logger.trace('VM with {disk_image} booted on {host}'. - format(disk_image=self._qemu_opt['disk_image'], - host=self._node['host'])) + logger.trace('VM booted on {host}'.format(host=self._node['host'])) def _update_vm_interfaces(self): """Update interface names in VM node dict.""" # Send guest-network-get-interfaces command via QGA, output example: # {"return": [{"name": "eth0", "hardware-address": "52:54:00:00:04:01"}, - # {"name": "eth1", "hardware-address": "52:54:00:00:04:02"}]} + # {"name": "eth1", "hardware-address": "52:54:00:00:04:02"}]}. out = self._qemu_qga_exec('guest-network-get-interfaces') interfaces = out.get('return') mac_name = {} if not interfaces: - raise RuntimeError('Get VM {disk_image} interface list failed ' - 'on {host}'. - format(disk_image=self._qemu_opt['disk_image'], - host=self._node['host'])) - # Create MAC-name dict + raise RuntimeError('Get VM interface list failed on {host}'. + format(host=self._node['host'])) + # Create MAC-name dict. for interface in interfaces: if 'hardware-address' not in interface: continue mac_name[interface['hardware-address']] = interface['name'] - # Match interface by MAC and save interface name + # Match interface by MAC and save interface name. for interface in self._vm_info['interfaces'].values(): mac = interface.get('mac_address') if_name = mac_name.get(mac) @@ -423,119 +459,38 @@ class QemuUtils(object): def qemu_start(self): """Start QEMU and wait until VM boot. - .. note:: First set at least node to run QEMU on. - :returns: VM node info. :rtype: dict """ - # Qemu binary path - bin_path = ('{qemu_path}{qemu_bin}'. - format(qemu_path=self._qemu_opt.get('qemu_path'), - qemu_bin=self._qemu_opt.get('qemu_bin'))) - - # Memory and huge pages - mem = ('-object memory-backend-file,id=mem,size={mem_size}M,' - 'mem-path={path},share=on -m {mem_size} -numa node,memdev=mem'. - format(mem_size=self._qemu_opt.get('mem_size'), - path=self._qemu_opt.get('huge_mnt'))) - - # Drive option - drive = ('-drive file={disk_image},format=raw,cache=none,if=virtio' - '{locking}'. - format(disk_image=self._qemu_opt.get('disk_image'), - locking=',file.locking=off'\ - if self._qemu_version_is_greater('2.10') else '')) - - # SSH forwarding - ssh = ('-net user,hostfwd=tcp::{ssh_fwd_port}-:22'. - format(ssh_fwd_port=self._qemu_opt.get('ssh_fwd_port'))) - # Setup QMP via unix socket - qmp = ('-qmp unix:{qmp_sock},server,nowait'. - format(qmp_sock=self._qemu_opt.get('qmp_sock'))) - # Setup QGA via chardev (unix socket) and isa-serial channel - qga = ('-chardev socket,path={qga_sock},server,nowait,id=qga0 ' - '-device isa-serial,chardev=qga0'. - format(qga_sock=self._qemu_opt.get('qga_sock'))) - # Setup serial console - serial = ('-chardev socket,host=127.0.0.1,port={serial_port},id=gnc0,' - 'server,nowait -device isa-serial,chardev=gnc0'. - format(serial_port=self._qemu_opt.get('serial_port'))) - - # Graphic setup - graphic = '-monitor none -display none -vga none' - - # PID file - pid = ('-pidfile {pid_file}'. - format(pid_file=self._qemu_opt.get('pid_file'))) - - # By default check only if hugepages are available. - # If 'huge_allocate' is set to true try to allocate as well. - DUTSetup.check_huge_page(self._node, self._qemu_opt.get('huge_mnt'), - self._qemu_opt.get('mem_size'), - allocate=self._qemu_opt.get('huge_allocate')) - - # Run QEMU - cmd = ('{bin_path} {smp} {mem} {ssh} {options} {drive} {qmp} {serial} ' - '{qga} {graphic} {pid}'. - format(bin_path=bin_path, smp=self._qemu_opt.get('smp'), - mem=mem, ssh=ssh, options=self._qemu_opt.get('options'), - drive=drive, qmp=qmp, serial=serial, qga=qga, - graphic=graphic, pid=pid)) + DUTSetup.check_huge_page(self._node, '/mnt/huge', self._opt.get('mem')) + + command = ('{bin_path}/qemu-system-{arch} {params}'. + format(bin_path=self._opt.get('bin_path'), + arch=Topology.get_node_arch(self._node), + params=self._params)) + message = ('QEMU: Start failed on {host}!'. + format(host=self._node['host'])) + try: - ret_code, _, _ = self._ssh.exec_command_sudo(cmd, timeout=300) - if int(ret_code) != 0: - raise RuntimeError('QEMU start failed on {host}'. - format(host=self._node['host'])) - # Wait until VM boot + exec_cmd_no_error(self._node, command, timeout=300, sudo=True, + message=message) self._wait_until_vm_boot() - except (RuntimeError, SSHTimeout): + # Update interface names in VM node dict. + self._update_vm_interfaces() + except RuntimeError: self.qemu_kill_all() - self.qemu_clear_socks() raise - logger.trace('QEMU started successfully.') - # Update interface names in VM node dict - self._update_vm_interfaces() - # Return VM node dict return self._vm_info - def qemu_quit(self): - """Quit the QEMU emulator.""" - out = self._qemu_qmp_exec('quit') - err = out.get('error') - if err is not None: - raise RuntimeError('QEMU quit failed on {host}: {error}'. - format(host=self._node['host'], - error=json.dumps(err))) - - def qemu_system_powerdown(self): - """Power down the system (if supported).""" - out = self._qemu_qmp_exec('system_powerdown') - err = out.get('error') - if err is not None: - raise RuntimeError( - 'QEMU system powerdown failed on {host}: {error}'. - format(host=self._node['host'], error=json.dumps(err))) - - def qemu_system_reset(self): - """Reset the system.""" - out = self._qemu_qmp_exec('system_reset') - err = out.get('error') - if err is not None: - raise RuntimeError( - 'QEMU system reset failed on {host}: {error}'. - format(host=self._node['host'], error=json.dumps(err))) - def qemu_kill(self): """Kill qemu process.""" - # Note: in QEMU start phase there are 3 QEMU processes because we - # daemonize QEMU - self._ssh.exec_command_sudo('chmod +r {pid}'. - format(pid=self._qemu_opt.get('pid_file'))) - self._ssh.exec_command_sudo('kill -SIGKILL $(cat {pid})'. - format(pid=self._qemu_opt.get('pid_file'))) - # Delete PID file - self._ssh.exec_command_sudo('rm -f {pid}'. - format(pid=self._qemu_opt.get('pid_file'))) + exec_cmd(self._node, 'chmod +r {pid}'. + format(pid=self._temp.get('pid')), sudo=True) + exec_cmd(self._node, 'kill -SIGKILL $(cat {pid})'. + format(pid=self._temp.get('pid')), sudo=True) + + for value in self._temp.get_values(): + exec_cmd(self._node, 'rm -f {value}'.format(value=value), sudo=True) def qemu_kill_all(self, node=None): """Kill all qemu processes on DUT node if specified. @@ -545,86 +500,31 @@ class QemuUtils(object): """ if node: self.qemu_set_node(node) - self._ssh.exec_command_sudo('pkill -SIGKILL qemu') - - def qemu_clear_socks(self): - """Remove all sockets created by QEMU.""" - # If serial console port still open kill process - self._ssh.exec_command_sudo('fuser -k {serial_port}/tcp'. - format(serial_port=\ - self._qemu_opt.get('serial_port'))) - # Delete all created sockets - for socket in self._socks: - self._ssh.exec_command_sudo('rm -f {socket}'. - format(socket=socket)) - - def qemu_system_status(self): - """Return current VM status. - - VM should be in following status: - - - debug: QEMU running on a debugger - - finish-migrate: paused to finish the migration process - - inmigrate: waiting for an incoming migration - - internal-error: internal error has occurred - - io-error: the last IOP has failed - - paused: paused - - postmigrate: paused following a successful migrate - - prelaunch: QEMU was started with -S and guest has not started - - restore-vm: paused to restore VM state - - running: actively running - - save-vm: paused to save the VM state - - shutdown: shut down (and -no-shutdown is in use) - - suspended: suspended (ACPI S3) - - watchdog: watchdog action has been triggered - - guest-panicked: panicked as a result of guest OS panic - - :returns: VM status. - :rtype: str - """ - out = self._qemu_qmp_exec('query-status') - ret = out.get('return') - if ret is not None: - return ret.get('status') - else: - err = out.get('error') - raise RuntimeError('QEMU query-status failed on {host}: {error}'. - format(host=self._node['host'], - error=json.dumps(err))) + exec_cmd(self._node, 'pkill -SIGKILL qemu', sudo=True) - def qemu_version(self): - """Return Qemu version. + for value in self._temp.get_values(): + exec_cmd(self._node, 'rm -f {value}'.format(value=value), sudo=True) - :returns: Qemu version. - :rtype: str - """ - # Qemu binary path - bin_path = ('{qemu_path}{qemu_bin}'. - format(qemu_path=self._qemu_opt.get('qemu_path'), - qemu_bin=self._qemu_opt.get('qemu_bin'))) + def qemu_version(self, version=None): + """Return Qemu version or compare if version is higher than parameter. + :param version: Version to compare. + :type version: str + :returns: Qemu version or Boolean if version is higher than parameter. + :rtype: str or bool + """ + command = ('{bin_path}/qemu-system-{arch} --version'. + format(bin_path=self._opt.get('bin_path'), + arch=Topology.get_node_arch(self._node))) try: - ret_code, stdout, _ = self._ssh.exec_command_sudo( - '{bin_path} --version'. - format(bin_path=bin_path)) - if int(ret_code) != 0: - raise RuntimeError('Failed to get QEMU version on {host}'. - format(host=self._node['host'])) - - return re.match(r'QEMU emulator version ([\d.]*)', stdout).group(1) - except (RuntimeError, SSHTimeout): + stdout, _ = exec_cmd_no_error(self._node, command, sudo=True) + ver = match(r'QEMU emulator version ([\d.]*)', stdout).group(1) + return StrictVersion(ver) > StrictVersion(version) \ + if version else ver + except RuntimeError: self.qemu_kill_all() - self.qemu_clear_socks() raise - def _qemu_version_is_greater(self, version): - """Compare Qemu versions. - - :returns: True if installed Qemu version is greater. - :rtype: bool - """ - return StrictVersion(self.qemu_version()) > StrictVersion(version) - @staticmethod def build_qemu(node, force_install=False, apply_patch=False): """Build QEMU from sources. @@ -637,9 +537,6 @@ class QemuUtils(object): :type apply_patch: bool :raises RuntimeError: If building QEMU failed. """ - ssh = SSH() - ssh.connect(node) - directory = (' --directory={install_dir}{patch}'. format(install_dir=Constants.QEMU_INSTALL_DIR, patch='-patch' if apply_patch else '-base')) @@ -647,18 +544,16 @@ class QemuUtils(object): format(install_version=Constants.QEMU_INSTALL_VERSION)) force = ' --force' if force_install else '' patch = ' --patch' if apply_patch else '' - arch = Topology.get_node_arch(node) target_list = (' --target-list={arch}-softmmu'. - format(arch=arch)) - - ret_code, _, _ = ssh.exec_command( - "sudo -E sh -c '{fw_dir}/{lib_sh}/qemu_build.sh{version}{directory}" - "{force}{patch}{target_list}'". - format(fw_dir=Constants.REMOTE_FW_DIR, - lib_sh=Constants.RESOURCES_LIB_SH, - version=version, directory=directory, force=force, - patch=patch, target_list=target_list), 1000) - - if int(ret_code) != 0: - raise RuntimeError('QEMU build failed on {host}'. - format(host=node['host'])) + format(arch=Topology.get_node_arch(node))) + + command = ("sudo -E sh -c " + "'{fw_dir}/{lib_sh}/qemu_build.sh{version}{directory}" + "{force}{patch}{target_list}'". + format(fw_dir=Constants.REMOTE_FW_DIR, + lib_sh=Constants.RESOURCES_LIB_SH, + version=version, directory=directory, force=force, + patch=patch, target_list=target_list)) + message = ('QEMU: Build failed on {host}!'.format(host=node['host'])) + exec_cmd_no_error(node, command, sudo=False, message=message, + timeout=1000) diff --git a/resources/libraries/robot/performance/performance_configuration.robot b/resources/libraries/robot/performance/performance_configuration.robot index cf287c2af6..d4819ff9c4 100644 --- a/resources/libraries/robot/performance/performance_configuration.robot +++ b/resources/libraries/robot/performance/performance_configuration.robot @@ -32,7 +32,6 @@ | Resource | resources/libraries/robot/l2/l2_patch.robot | Resource | resources/libraries/robot/ip/ip4.robot | Resource | resources/libraries/robot/ip/ip6.robot -| Resource | resources/libraries/robot/vm/qemu.robot | Resource | resources/libraries/robot/l2/tagging.robot | Resource | resources/libraries/robot/overlay/srv6.robot | Documentation | Performance suite keywords - configuration. @@ -2321,40 +2320,32 @@ | | ... | \| DUT1_VM2 \| qemu_id=${2} \| | | ... | | [Arguments] | ${dut} | ${sock1} | ${sock2} | ${vm_name} | ${nf_cpus} -| | ... | ${qemu_id}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} +| | ... | ${qemu_id}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${1024} | | ... | ${use_tuned_cfs}=${False} | ${auto_scale}=${True} | | ... +| | ${apply_patch}= | Set Variable | ${False} +| | ${perf_qemu_path}= | Set Variable If | ${apply_patch} +| | ... | ${perf_qemu_path}-patch/bin +| | ... | ${perf_qemu_path}-base/bin | | ${nf_cpus_count}= | Get Length | ${nf_cpus} | | ${rxq}= | Run Keyword If | ${auto_scale} == ${True} | | ... | Set Variable | ${rxq_count_int} | | ... | ELSE | Set Variable | ${1} -| | Import Library | resources.libraries.python.QemuUtils | qemu_id=${qemu_id} +| | Import Library | resources.libraries.python.QemuUtils | ${nodes['${dut}']} +| | ... | qemu_id=${qemu_id} | smp=${nf_cpus_count} | mem=${2048} +| | ... | img=${perf_vm_image} | bin_path=${perf_qemu_path} | | ... | WITH NAME | ${vm_name} -| | Run keyword | ${vm_name}.Qemu Set Node | ${nodes['${dut}']} -| | ${serial_port}= | Evaluate | ${qemu_id} + ${4555} -| | Run keyword | ${vm_name}.Qemu Set Serial Port | ${serial_port} -| | ${ssh_fwd_port}= | Evaluate | ${qemu_id} + ${10021} -| | Run keyword | ${vm_name}.Qemu Set Ssh Fwd Port | ${ssh_fwd_port} -| | Run keyword | ${vm_name}.Qemu Set Queue Count | ${rxq_count_int} -| | Run keyword | ${vm_name}.Qemu Set Queue Size | ${perf_qemu_qsz} -| | Run keyword | ${vm_name}.Qemu Add Vhost User If | ${sock1} -| | ... | jumbo_frames=${jumbo} -| | Run keyword | ${vm_name}.Qemu Add Vhost User If | ${sock2} -| | ... | jumbo_frames=${jumbo} -| | ${apply_patch}= | Set Variable | ${False} -| | ${perf_qemu_path}= | Set Variable If | ${apply_patch} -| | ... | ${perf_qemu_path}-patch/bin/ -| | ... | ${perf_qemu_path}-base/bin/ +| | Run Keyword | ${vm_name}.Qemu Add Vhost User If | ${sock1} +| | ... | jumbo_frames=${jumbo} | queues=${rxq_count_int} +| | ... | queue_size=${perf_qemu_qsz} +| | Run Keyword | ${vm_name}.Qemu Add Vhost User If | ${sock2} +| | ... | jumbo_frames=${jumbo} | queues=${rxq_count_int} +| | ... | queue_size=${perf_qemu_qsz} | | Run Keyword If | ${qemu_build} | ${vm_name}.Build QEMU | ${nodes['${dut}']} | | ... | apply_patch=${apply_patch} -| | Run keyword | ${vm_name}.Qemu Set Path | ${perf_qemu_path} -| | Run keyword | ${vm_name}.Qemu Set Smp | ${nf_cpus_count} | ${nf_cpus_count} -| | ... | 1 | 1 -| | Run keyword | ${vm_name}.Qemu Set Mem Size | 2048 -| | Run keyword | ${vm_name}.Qemu Set Disk Image | ${perf_vm_image} -| | ${vm}= | Run keyword | ${vm_name}.Qemu Start -| | Run keyword | ${vm_name}.Qemu Set Affinity | @{nf_cpus} -| | Run keyword If | ${use_tuned_cfs} | ${vm_name}.Qemu Set Scheduler Policy +| | ${vm}= | Run Keyword | ${vm_name}.Qemu Start +| | Run Keyword | ${vm_name}.Qemu Set Affinity | @{nf_cpus} +| | Run Keyword If | ${use_tuned_cfs} | ${vm_name}.Qemu Set Scheduler Policy | | ${max_pkt_len}= | Set Variable If | ${jumbo} | 9200 | ${EMPTY} | | ${testpmd_cpus}= | Cpu list per node str | ${nodes['${dut}']} | ${0} | | ... | cpu_cnt=${nf_cpus_count} @@ -2382,10 +2373,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure guest VMs with dpdk-testpmd connected via \ -| | ... | vhost-user on node \| DUT1 \| 1 \| False \| 256 \| +| | ... | vhost-user on node \| DUT1 \| 1 \| False \| 1024 \| | | ... | | [Arguments] | ${dut} | ${vm_count}=${1} | ${jumbo}=${False} | -| | ... | ${perf_qemu_qsz}=${256} | ${use_tuned_cfs}=${False} +| | ... | ${perf_qemu_qsz}=${1024} | ${use_tuned_cfs}=${False} | | ... | | :FOR | ${number} | IN RANGE | 1 | ${vm_count}+1 | | | ${nf_cpus}= | Create network function CPU list | ${dut} @@ -2395,10 +2386,11 @@ | | | ${sock2}= | Set Variable | /tmp/sock-${number}-2 | | | ${vm}= | | | ... | Configure guest VM with dpdk-testpmd connected via vhost-user -| | | ... | ${dut} | ${sock1} | ${sock2} | ${dut}_VM${number} +| | | ... | ${dut} | ${sock1} | ${sock2} | ${TEST NAME}${dut}_VM${number} | | | ... | ${nf_cpus} | qemu_id=${number} | jumbo=${jumbo} | | | ... | perf_qemu_qsz=${perf_qemu_qsz} | use_tuned_cfs=${use_tuned_cfs} -| | | Set To Dictionary | ${${dut}_vm_refs} | ${dut}_VM${number} | ${vm} +| | | Set To Dictionary | ${${dut}_vm_refs} | ${TEST NAME}${dut}_VM${number} +| | | ... | ${vm} | Configure guest VMs with dpdk-testpmd connected via vhost-user | | [Documentation] @@ -2417,10 +2409,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure guest VMs with dpdk-testpmd connected via vhost-user\ -| | ... | \| 1 \| False \| 256 \| +| | ... | \| 1 \| False \| 1024 \| | | ... -| | [Arguments] | ${vm_count}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} -| | ... | ${use_tuned_cfs}=${False} +| | [Arguments] | ${vm_count}=${1} | ${jumbo}=${False} +| | ... | ${perf_qemu_qsz}=${1024} | ${use_tuned_cfs}=${False} | | ... | | ${duts}= | Get Matches | ${nodes} | DUT* | | :FOR | ${dut} | IN | @{duts} @@ -2467,37 +2459,29 @@ | | ... | | [Arguments] | ${dut} | ${sock1} | ${sock2} | ${vm_name} | | ... | ${eth0_mac} | ${eth1_mac} | ${nf_cpus} | ${qemu_id}=${1} -| | ... | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} +| | ... | ${jumbo}=${False} | ${perf_qemu_qsz}=${1024} | | ... | ${use_tuned_cfs}=${False} | ${auto_scale}=${True} | | ... +| | ${apply_patch}= | Set Variable | ${False} +| | ${perf_qemu_path}= | Set Variable If | ${apply_patch} +| | ... | ${perf_qemu_path}-patch/bin +| | ... | ${perf_qemu_path}-base/bin | | ${nf_cpus_count}= | Get Length | ${nf_cpus} | | ${rxq}= | Run Keyword If | ${auto_scale} == ${True} | | ... | Set Variable | ${rxq_count_int} | | ... | ELSE | Set Variable | ${1} -| | Import Library | resources.libraries.python.QemuUtils | qemu_id=${qemu_id} +| | Import Library | resources.libraries.python.QemuUtils | ${nodes['${dut}']} +| | ... | qemu_id=${qemu_id} | smp=${nf_cpus_count} | mem=${2048} +| | ... | img=${perf_vm_image} | bin_path=${perf_qemu_path} | | ... | WITH NAME | ${vm_name} -| | Run keyword | ${vm_name}.Qemu Set Node | ${nodes['${dut}']} -| | ${serial_port}= | Evaluate | ${qemu_id} + ${4555} -| | Run keyword | ${vm_name}.Qemu Set Serial Port | ${serial_port} -| | ${ssh_fwd_port}= | Evaluate | ${qemu_id} + ${10021} -| | Run keyword | ${vm_name}.Qemu Set Ssh Fwd Port | ${ssh_fwd_port} -| | Run keyword | ${vm_name}.Qemu Set Queue Count | ${rxq_count_int} -| | Run keyword | ${vm_name}.Qemu Set Queue Size | ${perf_qemu_qsz} | | Run keyword | ${vm_name}.Qemu Add Vhost User If | ${sock1} -| | ... | jumbo_frames=${jumbo} +| | ... | jumbo_frames=${jumbo} | queues=${rxq_count_int} +| | ... | queue_size=${perf_qemu_qsz} | | Run keyword | ${vm_name}.Qemu Add Vhost User If | ${sock2} -| | ... | jumbo_frames=${jumbo} -| | ${apply_patch}= | Set Variable | ${False} -| | ${perf_qemu_path}= | Set Variable If | ${apply_patch} -| | ... | ${perf_qemu_path}-patch/bin/ -| | ... | ${perf_qemu_path}-base/bin/ +| | ... | jumbo_frames=${jumbo} | queues=${rxq_count_int} +| | ... | queue_size=${perf_qemu_qsz} | | Run Keyword If | ${qemu_build} | ${vm_name}.Build QEMU | ${nodes['${dut}']} | | ... | apply_patch=${False} -| | Run keyword | ${vm_name}.Qemu Set Path | ${perf_qemu_path} -| | Run keyword | ${vm_name}.Qemu Set Smp | ${nf_cpus_count} | ${nf_cpus_count} -| | ... | 1 | 1 -| | Run keyword | ${vm_name}.Qemu Set Mem Size | 2048 -| | Run keyword | ${vm_name}.Qemu Set Disk Image | ${perf_vm_image} | | ${vm}= | Run keyword | ${vm_name}.Qemu Start | | Run keyword | ${vm_name}.Qemu Set Affinity | @{nf_cpus} | | Run keyword If | ${use_tuned_cfs} | ${vm_name}.Qemu Set Scheduler Policy @@ -2532,10 +2516,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure guest VMs with dpdk-testpmd-mac connected via \ -| | ... | vhost-user on node \| DUT1 \| 1 \| False \| 256 \| +| | ... | vhost-user on node \| DUT1 \| 1 \| False \| 1024 \| | | ... | | [Arguments] | ${dut} | ${vm_count}=${1} | ${jumbo}=${False} | -| | ... | ${perf_qemu_qsz}=${256} | ${use_tuned_cfs}=${False} +| | ... | ${perf_qemu_qsz}=${1024} | ${use_tuned_cfs}=${False} | | ... | | :FOR | ${number} | IN RANGE | 1 | ${vm_count}+1 | | | ${nf_cpus}= | Create network function CPU list | ${dut} @@ -2545,12 +2529,13 @@ | | | ${sock2}= | Set Variable | /tmp/sock-${number}-2 | | | ${vm}= | | | ... | Configure guest VM with dpdk-testpmd-mac connected via vhost-user -| | | ... | ${dut} | ${sock1} | ${sock2} | ${dut}_VM${number} +| | | ... | ${dut} | ${sock1} | ${sock2} | ${TEST NAME}${dut}_VM${number} | | | ... | ${${dut}-vhost-${number}-if1_mac} | | | ... | ${${dut}-vhost-${number}-if2_mac} | nf_cpus=${nf_cpus} | | | ... | qemu_id=${number} | jumbo=${jumbo} | perf_qemu_qsz=${perf_qemu_qsz} | | | ... | use_tuned_cfs=${use_tuned_cfs} -| | | Set To Dictionary | ${${dut}_vm_refs} | ${dut}_VM${number} | ${vm} +| | | Set To Dictionary | ${${dut}_vm_refs} | ${TEST NAME}${dut}_VM${number} +| | | ... | ${vm} | Configure guest VMs with dpdk-testpmd-mac connected via vhost-user | | [Documentation] @@ -2569,10 +2554,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure guest VMs with dpdk-testpmd-mac connected via vhost-user\ -| | ... | \| 1 \| False \| 256 \| +| | ... | \| 1 \| False \| 1024 \| | | ... -| | [Arguments] | ${vm_count}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} -| | ... | ${use_tuned_cfs}=${False} +| | [Arguments] | ${vm_count}=${1} | ${jumbo}=${False} +| | ... | ${perf_qemu_qsz}=${1024} | ${use_tuned_cfs}=${False} | | ... | | ${duts}= | Get Matches | ${nodes} | DUT* | | :FOR | ${dut} | IN | @{duts} @@ -2604,10 +2589,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure chain of NFs with dpdk-testpmd-mac connected via \ -| | ... | vhost-user on node \| DUT1 \| 1 \| 1 \| 1 \| False \| 256 \| +| | ... | vhost-user on node \| DUT1 \| 1 \| 1 \| 1 \| False \| 1024 \| | | ... | | [Arguments] | ${dut} | ${nf_chains}=${1} | ${nf_chain}=${1} -| | ... | ${nf_nodes}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} +| | ... | ${nf_nodes}=${1} | ${jumbo}=${False} | ${perf_qemu_qsz}=${1024} | | ... | ${use_tuned_cfs}=${False} | ${auto_scale}=${False} | | ... | | ${tg_if1_mac}= | Get Interface MAC | ${tg} | ${tg_if1} @@ -2661,10 +2646,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure chain of NFs with dpdk-testpmd-mac connected via\ -| | ... | vhost-user \| 1 \| 1 \| 1 \| False \| 256 \| +| | ... | vhost-user \| 1 \| 1 \| 1 \| False \| 1024 \| | | ... | | [Arguments] | ${nf_chains}=${1} | ${nf_chain}=${1} | ${nf_nodes}=${1} -| | ... | ${jumbo}=${False} | ${perf_qemu_qsz}=${256} +| | ... | ${jumbo}=${False} | ${perf_qemu_qsz}=${1024} | | ... | ${use_tuned_cfs}=${False} | ${auto_scale}=${False} | | ... | | ${duts}= | Get Matches | ${nodes} | DUT* @@ -2695,10 +2680,10 @@ | | ... | *Example:* | | ... | | ... | \| Configure chains of VMs with dpdk-testpmd-mac connected via \ -| | ... | vhost-user \| 1 \| 1 \| False \| 256 \| +| | ... | vhost-user \| 1 \| 1 \| False \| 1024 \| | | ... | | [Arguments] | ${nf_chains}=${1} | ${nf_nodes}=${1} | ${jumbo}=${False} -| | ... | ${perf_qemu_qsz}=${256} | ${use_tuned_cfs}=${False} +| | ... | ${perf_qemu_qsz}=${1024} | ${use_tuned_cfs}=${False} | | ... | ${auto_scale}=${False} | | ... | | :FOR | ${nf_chain} | IN RANGE | 1 | ${nf_chains}+1 diff --git a/resources/libraries/robot/performance/performance_setup.robot b/resources/libraries/robot/performance/performance_setup.robot index 98f6310510..98adf6df9d 100644 --- a/resources/libraries/robot/performance/performance_setup.robot +++ b/resources/libraries/robot/performance/performance_setup.robot @@ -221,8 +221,6 @@ | | | ${vm}= | Get From Dictionary | ${dut_vm_refs} | ${vm_name} | | | ${index}= | Evaluate | ${index} + 1 | | | Dpdk Testpmd Stop | ${vm} -| | | Run Keyword | ${vm_name}.Qemu Set Node | ${dut_node} -| | | Run Keyword | ${vm_name}.Qemu Clear Socks | | | Run Keyword If | '${index}' == '${vms_number}' | ${vm_name}.Qemu Kill All | Tear down guest VM @@ -245,8 +243,6 @@ | | :FOR | ${vm_name} | IN | @{dut_vm_refs} | | | ${vm}= | Get From Dictionary | ${dut_vm_refs} | ${vm_name} | | | ${index}= | Evaluate | ${index} + 1 -| | | Run Keyword | ${vm_name}.Qemu Set Node | ${dut_node} -| | | Run Keyword | ${vm_name}.Qemu Clear Socks | | | Run Keyword If | '${index}' == '${vms_number}' | ${vm_name}.Qemu Kill All # Suite setups diff --git a/resources/libraries/robot/performance/performance_utils.robot b/resources/libraries/robot/performance/performance_utils.robot index 480f7e1f33..6ce706a514 100644 --- a/resources/libraries/robot/performance/performance_utils.robot +++ b/resources/libraries/robot/performance/performance_utils.robot @@ -32,7 +32,6 @@ | Resource | resources/libraries/robot/l2/l2_xconnect.robot | Resource | resources/libraries/robot/ip/ip4.robot | Resource | resources/libraries/robot/ip/ip6.robot -| Resource | resources/libraries/robot/vm/qemu.robot | Resource | resources/libraries/robot/l2/tagging.robot | Documentation | Performance suite keywords - utilities to find and verify NDR | ... | and PDR. diff --git a/resources/libraries/robot/shared/default.robot b/resources/libraries/robot/shared/default.robot index 91fde22038..6d99b959d3 100644 --- a/resources/libraries/robot/shared/default.robot +++ b/resources/libraries/robot/shared/default.robot @@ -33,7 +33,6 @@ | Library | resources.libraries.python.topology.Topology | ... | Resource | resources/libraries/robot/shared/container.robot -| Resource | resources/libraries/robot/vm/double_qemu_setup.robot | Resource | resources/libraries/robot/vm/qemu.robot *** Keywords *** @@ -518,22 +517,13 @@ | | [Documentation] | Common test teardown for functional tests with LISP and\ | | ... | QEMU. | | ... -| | ... | *Arguments:* -| | ... | - vm_node - VM to stop. Type: string -| | ... -| | ... | *Example:* -| | ... -| | ... | \| Tear down LISP functional test with QEMU \| ${vm_node} \| -| | ... -| | [Arguments] | ${vm_node} -| | ... | | Remove All Added Ports On All DUTs From Topology | ${nodes} | | Show Packet Trace on All DUTs | ${nodes} | | Show PAPI History On All DUTs | ${nodes} | | Show Vpp Settings | ${nodes['DUT1']} | | Show Vpp Settings | ${nodes['DUT2']} | | Vpp Show Errors On All DUTs | ${nodes} -| | Stop and clear QEMU | ${nodes['DUT1']} | ${vm_node} +| | Stop and clear QEMU | ${nodes['DUT1']} | | Verify VPP PID in Teardown | Set up TAP functional test @@ -609,21 +599,17 @@ | | ... | | ... | *Arguments:* | | ... | - dut1_node - Node Nr.1 where to clean qemu. Type: dictionary -| | ... | - qemu_node1 - VM Nr.1 node info dictionary. Type: string | | ... | - dut2_node - Node Nr.2 where to clean qemu. Type: dictionary -| | ... | - qemu_node2 - VM Nr.2 node info dictionary. Type: string | | ... | | ... | *Example:* | | ... -| | ... | \| Tear down FDS functional test \| ${dut1_node} \| ${qemu_node1}\ -| | ... | \| ${dut2_node} \| ${qemu_node2} \| +| | ... | \| Tear down FDS functional test \| ${dut1_node} \| ${dut2_node} \| | | ... -| | [Arguments] | ${dut1_node} | ${qemu_node1} | ${dut2_node} -| | ... | ${qemu_node2} +| | [Arguments] | ${dut1_node} | ${dut2_node} | | ... | | Tear down functional test -| | Tear down QEMU | ${dut1_node} | ${qemu_node1} | qemu_node1 -| | Tear down QEMU | ${dut2_node} | ${qemu_node2} | qemu_node2 +| | Tear down QEMU | ${dut1_node} | qemu_node1 +| | Tear down QEMU | ${dut2_node} | qemu_node2 | Tear down functional test with container | | [Documentation] diff --git a/resources/libraries/robot/vm/double_qemu_setup.robot b/resources/libraries/robot/vm/double_qemu_setup.robot deleted file mode 100644 index 377e222804..0000000000 --- a/resources/libraries/robot/vm/double_qemu_setup.robot +++ /dev/null @@ -1,95 +0,0 @@ -# Copyright (c) 2016 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: -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. -*** Settings *** -| Library | resources.libraries.python.IPUtil - -*** Keywords *** -| Configure QEMU vhost and run it -| | [Documentation] | Setup Qemu with 4 vhost-user interfaces and 4 namespaces. -| | ... | Each call will be different object instance. -| | ... -| | ... | *Arguments:* -| | ... | - dut_node - Node where to setup qemu. Type: dict -| | ... | - sock1 - Socket path for first Vhost-User interface. Type: string -| | ... | - sock2 - Socket path for second Vhost-User interface. Type: string -| | ... | - sock3 - Socket path for third Vhost-User interface. Type: string -| | ... | - sock4 - Socket path for forth Vhost-User interface. Type: string -| | ... | - ip1 - IP address for namespace 1. Type: string -| | ... | - ip2 - IP address for namespace 2. Type: string -| | ... | - ip3 - IP address for namespace 3. Type: string -| | ... | - ip4 - IP address for namespace 4. Type: string -| | ... | - prefix_length - IP prefix length. Type: int -| | ... | - qemu_name - Qemu instance name by which the object will be accessed. -| | ... | Type: string -| | ... | - mac_ID - MAC address ID used to differentiate qemu instances and -| | ... | namespaces assigned to them. Type: string -| | ... -| | ... | *Example:* -| | ... -| | ... | \| Setup QEMU Vhost And Run\| {nodes['DUT1']} \| /tmp/sock1 \ -| | ... | \| /tmp/sock2 \| /tmp/sock3 \| /tmp/sock4 \| 16.0.0.1 \| 16.0.0.2 \ -| | ... | \| 16.0.0.3 \| 16.0.0.4 \| 24 \| qemu_instance_1 \| 06 \| -| | ... -| | [Arguments] | ${dut_node} | ${sock1} | ${sock2} | ${sock3} | ${sock4} -| | ... | ${ip1} | ${ip2} | ${ip3} | ${ip4} | ${prefix_length} -| | ... | ${qemu_name} | ${mac_ID} -| | Import Library | resources.libraries.python.QemuUtils \ -| | ... | WITH NAME | ${qemu_name} -| | ${qemu_add_vhost}= | Replace Variables | ${qemu_name}.Qemu Add Vhost User If -| | ${qemu_set_node}= | Replace Variables | ${qemu_name}.Qemu Set Node -| | ${qemu_start}= | Replace Variables | ${qemu_name}.Qemu Start -| | Run keyword | ${qemu_add_vhost} | ${sock1} | mac=52:54:00:00:${mac_ID}:01 -| | Run keyword | ${qemu_add_vhost} | ${sock2} | mac=52:54:00:00:${mac_ID}:02 -| | Run keyword | ${qemu_add_vhost} | ${sock3} | mac=52:54:00:00:${mac_ID}:03 -| | Run keyword | ${qemu_add_vhost} | ${sock4} | mac=52:54:00:00:${mac_ID}:04 -| | Run keyword | ${qemu_set_node} | ${dut_node} -| | ${vm}= | Run keyword | ${qemu_start} -| | ${vhost1}= | Get Vhost User If Name By Sock | ${vm} | ${sock1} -| | ${vhost2}= | Get Vhost User If Name By Sock | ${vm} | ${sock2} -| | ${vhost3}= | Get Vhost User If Name By Sock | ${vm} | ${sock3} -| | ${vhost4}= | Get Vhost User If Name By Sock | ${vm} | ${sock4} -| | Set Interface State | ${vm} | ${vhost1} | up | if_type=name -| | Set Interface State | ${vm} | ${vhost2} | up | if_type=name -| | Set Interface State | ${vm} | ${vhost3} | up | if_type=name -| | Set Interface State | ${vm} | ${vhost4} | up | if_type=name -| | Setup Network Namespace -| | ... | ${vm} | nmspace1 | ${vhost1} | ${ip1} | ${prefix_length} -| | Setup Network Namespace -| | ... | ${vm} | nmspace2 | ${vhost2} | ${ip2} | ${prefix_length} -| | Setup Network Namespace -| | ... | ${vm} | nmspace3 | ${vhost3} | ${ip3} | ${prefix_length} -| | Setup Network Namespace -| | ... | ${vm} | nmspace4 | ${vhost4} | ${ip4} | ${prefix_length} -| | Set Test Variable | ${${qemu_name}} | ${vm} - -| Tear down QEMU -| | [Documentation] | Stop specific qemu instance -| | ... | running on ${dut_node}, ${vm} is VM node info dictionary -| | ... | returned by qemu_start or None. -| | ... | *Arguments:* -| | ... | - dut_node - Node where to clean qemu. Type: dict -| | ... | - vm - VM node info dictionary. Type: string -| | ... | - qemu_name - Qemu instance by name. Type: string -| | ... -| | ... | *Example:* -| | ... -| | ... | \| Tear down QEMU \| ${node['DUT1']} \| ${vm} \| qemu_node_1 \| -| | ... -| | [Arguments] | ${dut_node} | ${vm} | ${qemu_name} -| | ${set_node}= | Replace Variables | ${qemu_name}.Qemu Set Node -| | ${kill}= | Replace Variables | ${qemu_name}.Qemu Kill -| | ${clear_socks}= | Replace Variables | ${qemu_name}.Qemu Clear Socks -| | Run Keyword | ${set_node} | ${dut_node} -| | Run Keyword | ${kill} -| | Run Keyword | ${clear_socks} -| | Run Keyword If | ${vm} is not None | Disconnect | ${vm} diff --git a/resources/libraries/robot/vm/qemu.robot b/resources/libraries/robot/vm/qemu.robot index 097f3cb1a8..528b7fbe8c 100644 --- a/resources/libraries/robot/vm/qemu.robot +++ b/resources/libraries/robot/vm/qemu.robot @@ -1,4 +1,4 @@ -# Copyright (c) 2016 Cisco and/or its affiliates. +# Copyright (c) 2019 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: @@ -11,89 +11,95 @@ # See the License for the specific language governing permissions and # limitations under the License. *** Settings *** +| Library | resources.libraries.python.IPUtil | Library | resources.libraries.python.QemuUtils -| Library | resources.libraries.python.ssh.SSH *** Keywords *** - -| QEMU build list should exist -| | [Documentation] | Return TRUE if variable QEMU_BUILD exist, otherwise FALSE -| | ${ret} | ${tmp}= | Run Keyword And Ignore Error -| | ... | Variable Should Exist | @{QEMU_BUILD} -| | Return From Keyword If | "${ret}" == "PASS" | ${TRUE} -| | Return From Keyword | ${FALSE} - -| Is QEMU ready on node -| | [Documentation] | Check if QEMU was built on the node before -| | [Arguments] | ${node} -| | ${ret}= | QEMU build list should exist -| | Return From Keyword If | ${ret} == ${FALSE} | ${FALSE} -| | ${ret} | ${tmp}= | Run Keyword And Ignore Error -| | ... | Should Contain | ${QEMU_BUILD} | ${node['host']} -| | Return From Keyword If | "${ret}" == "PASS" | ${TRUE} -| | Return From Keyword | ${FALSE} - -| Add node to QEMU build list -| | [Documentation] | Add node to the list of nodes with builded QEMU (global -| | ... | variable QEMU_BUILD) -| | [Arguments] | ${node} -| | ${ret}= | QEMU build list should exist -| | Run Keyword If | ${ret} == ${TRUE} -| | ... | Append To List | ${QEMU_BUILD} | ${node['host']} -| | ... | ELSE | Set Global Variable | @{QEMU_BUILD} | ${node['host']} - -| Build QEMU on node -| | [Documentation] | Build QEMU from sources on the Node. Nodes with successful -| | ... | QEMU build are stored in global variable list QEMU_BUILD +| Configure QEMU vhost and run it +| | [Documentation] +| | ... | Setup Qemu with 4 vhost-user interfaces and 4 namespaces. +| | ... | Each call will be different object instance. | | ... | | ... | *Arguments:* -| | ... | - node - Node on which to build qemu. Type: dictionary -| | ... | - force_install - If True, then remove previous build. Type: bool -| | ... | - apply_patch - If True, then apply patches from qemu_patches dir. -| | ... | Type: bool +| | ... | - dut_node - Node where to setup qemu. Type: dict +| | ... | - sock1 - Socket path for first Vhost-User interface. Type: string +| | ... | - sock2 - Socket path for second Vhost-User interface. Type: string +| | ... | - sock3 - Socket path for third Vhost-User interface. Type: string +| | ... | - sock4 - Socket path for forth Vhost-User interface. Type: string +| | ... | - ip1 - IP address for namespace 1. Type: string +| | ... | - ip2 - IP address for namespace 2. Type: string +| | ... | - ip3 - IP address for namespace 3. Type: string +| | ... | - ip4 - IP address for namespace 4. Type: string +| | ... | - prefix_length - IP prefix length. Type: int +| | ... | - qemu_name - Qemu instance name by which the object will be accessed. +| | ... | Type: string +| | ... | - mac_ID - MAC address ID used to differentiate qemu instances and +| | ... | namespaces assigned to them. Type: string | | ... | | ... | *Example:* | | ... -| | ... | \| Build QEMU on node \| ${node['DUT1']} \| False \| False \| +| | ... | \| Setup QEMU Vhost And Run\| {nodes['DUT1']} \| /tmp/sock1 \ +| | ... | \| /tmp/sock2 \| /tmp/sock3 \| /tmp/sock4 \| 16.0.0.1 \| 16.0.0.2 \ +| | ... | \| 16.0.0.3 \| 16.0.0.4 \| 24 \| qemu_instance_1 \| 06 \| | | ... -| | [Arguments] | ${node} | ${force_install}=${False} | ${apply_patch}=${False} -| | ${ready}= | Is QEMU ready on node | ${node} -| | Return From Keyword If | ${ready} == ${TRUE} -| | Build QEMU | ${node} -| | Add node to QEMU build list | ${node} +| | [Arguments] | ${dut_node} | ${sock1} | ${sock2} | ${sock3} | ${sock4} +| | ... | ${ip1} | ${ip2} | ${ip3} | ${ip4} | ${prefix_length} +| | ... | ${qemu_name} | ${mac_ID}=${None} +| | ... +| | Import Library | resources.libraries.python.QemuUtils \ +| | ... | node=${dut_node} | WITH NAME | ${qemu_name} +| | Run keyword | ${qemu_name}.Qemu Add Vhost User If | ${sock1} +| | Run keyword | ${qemu_name}.Qemu Add Vhost User If | ${sock2} +| | Run keyword | ${qemu_name}.Qemu Add Vhost User If | ${sock3} +| | Run keyword | ${qemu_name}.Qemu Add Vhost User If | ${sock4} +| | ${vm}= | Run keyword | ${qemu_name}.Qemu Start +| | ${vhost1}= | Get Vhost User If Name By Sock | ${vm} | ${sock1} +| | ${vhost2}= | Get Vhost User If Name By Sock | ${vm} | ${sock2} +| | ${vhost3}= | Get Vhost User If Name By Sock | ${vm} | ${sock3} +| | ${vhost4}= | Get Vhost User If Name By Sock | ${vm} | ${sock4} +| | Set Interface State | ${vm} | ${vhost1} | up | if_type=name +| | Set Interface State | ${vm} | ${vhost2} | up | if_type=name +| | Set Interface State | ${vm} | ${vhost3} | up | if_type=name +| | Set Interface State | ${vm} | ${vhost4} | up | if_type=name +| | Setup Network Namespace +| | ... | ${vm} | nmspace1 | ${vhost1} | ${ip1} | ${prefix_length} +| | Setup Network Namespace +| | ... | ${vm} | nmspace2 | ${vhost2} | ${ip2} | ${prefix_length} +| | Setup Network Namespace +| | ... | ${vm} | nmspace3 | ${vhost3} | ${ip3} | ${prefix_length} +| | Setup Network Namespace +| | ... | ${vm} | nmspace4 | ${vhost4} | ${ip4} | ${prefix_length} +| | Set Test Variable | ${${qemu_name}} | ${vm} -| Build QEMU on all DUTs -| | [Documentation] | Build QEMU from sources on all DUTs. Nodes with successful -| | ... | QEMU build are stored in global variable list QEMU_BUILD +| Tear down QEMU +| | [Documentation] +| | ... | Stop specific qemu instance running on ${dut_node}. | | ... | | ... | *Arguments:* -| | ... | - force_install - If True, then remove previous build. Type: bool -| | ... | - apply_patch - If True, then apply patches from qemu_patches dir. -| | ... | Type: bool +| | ... | - dut_node - Node where to clean qemu. Type: dict +| | ... | - qemu_name - Qemu instance by name. Type: string | | ... | | ... | *Example:* | | ... -| | ... | \| Build QEMU on all DUTs \| False \| False \| +| | ... | \| Tear down QEMU \| ${node['DUT1']} \| qemu_node_1 \| +| | ... +| | [Arguments] | ${dut_node} | ${qemu_name} | | ... -| | [Arguments] | ${force_install}=${False} | ${apply_patch}=${False} -| | ${duts}= | Get Matches | ${nodes} | DUT* -| | :FOR | ${dut} | IN | @{duts} -| | | Build QEMU on node | ${nodes['${dut}']} | ${force_install} | -| | | ... | ${apply_patch} +| | Run Keyword | ${qemu_name}.Qemu Set Node | ${dut_node} +| | Run Keyword | ${qemu_name}.Qemu Kill | Stop and clear QEMU -| | [Documentation] | Stop QEMU, clear used sockets and close SSH connection -| | ... | running on ${dut}, ${vm} is VM node info dictionary -| | ... | returned by qemu_start or None. -| | [Arguments] | ${dut} | ${vm} +| | [Documentation] +| | ... | Stop QEMU, clear used sockets running on ${dut}. +| | ... +| | ... | *Arguments:* +| | ... | - dut_node - Node where to clean qemu. Type: dict +| | ... +| | ... | *Example:* +| | ... +| | ... | \| Tear down QEMU \| ${node['DUT1']} \| +| | ... +| | [Arguments] | ${dut} +| | ... | | Qemu Set Node | ${dut} | | Qemu Kill -| | Qemu Clear Socks -| | Run Keyword If | ${vm} is not None | Disconnect | ${vm} - -| Kill Qemu on all DUTs -| | [Documentation] | Kill QEMU processes on all DUTs. -| | ${duts}= | Get Matches | ${nodes} | DUT* -| | :FOR | ${dut} | IN | @{duts} -| | | Qemu Set Node | ${nodes['${dut}']} -| | | Qemu Kill |