public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: passt-dev@passt.top, Stefano Brivio <sbrivio@redhat.com>
Cc: jarichte@redhat.com, Cleber Rosa <crosa@redhat.com>,
	David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH v2 16/21] avocado/tasst: Helpers for test transferring data between sites
Date: Tue, 16 May 2023 12:01:30 +1000	[thread overview]
Message-ID: <20230516020135.1901256-17-david@gibson.dropbear.id.au> (raw)
In-Reply-To: <20230516020135.1901256-1-david@gibson.dropbear.id.au>

Many of our existing tests are based on using socat to transfer between
various locations connected via pasta or passt.  Add helpers to make
avocado tests performing similar transfers.  Add meta tests to verify those
work as expected when we don't have pasta or passt involved yet.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 Makefile                  |   2 +-
 avocado/tasst/__init__.py |   5 +
 avocado/tasst/transfer.py | 227 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 233 insertions(+), 1 deletion(-)
 create mode 100644 avocado/tasst/transfer.py

diff --git a/Makefile b/Makefile
index 9add0a4..8eaae0f 100644
--- a/Makefile
+++ b/Makefile
@@ -299,7 +299,7 @@ cppcheck: $(SRCS) $(HEADERS)
 AVOCADO = avocado
 
 avocado-assets:
-	$(MAKE) -C test nstool
+	$(MAKE) -C test nstool small.bin
 
 avocado-%: avocado-assets
 	PYTHONPATH=./avocado $(AVOCADO) run avocado --filter-by-tags=$*
diff --git a/avocado/tasst/__init__.py b/avocado/tasst/__init__.py
index bd7994c..790cf45 100644
--- a/avocado/tasst/__init__.py
+++ b/avocado/tasst/__init__.py
@@ -11,6 +11,11 @@
 import avocado
 
 
+class TasstSubData:
+    def __init__(self, **kwargs):
+        self.__dict__.update(kwargs)
+
+
 # Base class for avocado-based passt/pasta tests
 class Tasst(avocado.Test):
     # Fairly short default timeout
diff --git a/avocado/tasst/transfer.py b/avocado/tasst/transfer.py
new file mode 100644
index 0000000..e891012
--- /dev/null
+++ b/avocado/tasst/transfer.py
@@ -0,0 +1,227 @@
+#! /usr/bin/python3
+
+# SPDX-License-Identifier: GPL-2.0-or-later
+#
+# tasst - Test A Simple Socket Transport
+#         library of test helpers for passt & pasta
+#
+# tasst/transfer.py - Helpers for testing data transfers
+#
+# Copyright Red Hat
+# Author: David Gibson <david@gibson.dropbear.id.au>
+
+import ipaddress
+import time
+
+import avocado
+
+from tasst import Tasst, TasstSubData
+from tasst.site import Site
+from tasst.nstool import UnshareSite
+
+
+# HACK: how long to wait for the server to be ready and listening (s)
+SERVER_READY_DELAY = 0.05  # 1/20th of a second
+
+
+# socat needs IPv6 addresses in square brackets
+def socat_fmt(ip):
+    if isinstance(ip, ipaddress.IPv6Address):
+        return '[{}]'.format(ip)
+    elif isinstance(ip, ipaddress.IPv4Address):
+        return '{}'.format(ip)
+    else:
+        raise TypeError
+
+
+class BaseTransferTasst(Tasst):
+    def socat_upload(self, datafile, cs, ss, connect, listen):
+        server = ss.bg('socat -u {} STDOUT'.format(listen), verbose=False)
+        time.sleep(SERVER_READY_DELAY)
+        cs.fg('socat -u OPEN:{} {}'.format(datafile, connect))
+        res = server.run()
+        self.assertEquals(res.exit_status, 0)
+        srcdata = cs.output('cat {}'.format(datafile), verbose=False)
+        self.assertEquals(srcdata, res.stdout)
+
+    def socat_download(self, datafile, cs, ss, connect, listen):
+        server = ss.bg('socat -u OPEN:{} {}'.format(datafile, listen))
+        time.sleep(SERVER_READY_DELAY)
+        dstdata = cs.output('socat -u {} STDOUT'.format(connect), verbose=False)
+        res = server.run()
+        self.assertEquals(res.exit_status, 0)
+        srcdata = ss.output('cat {}'.format(datafile), verbose=False)
+        self.assertEquals(srcdata, dstdata)
+
+    def _tcp_socat(self, datafile, connectip, connectport,
+                   listenip, listenport, fromip):
+        v6 = isinstance(connectip, ipaddress.IPv6Address)
+        if listenport is None:
+            listenport = connectport
+        if v6:
+            connect = 'TCP6:[{}]:{},ipv6only'.format(connectip, connectport)
+            listen = 'TCP6-LISTEN:{},ipv6only'.format(listenport)
+        else:
+            connect = 'TCP4:{}:{}'.format(connectip, connectport)
+            listen = 'TCP4-LISTEN:{}'.format(listenport)
+        if listenip is not None:
+            listen += ',bind=' + socat_fmt(listenip)
+        if fromip is not None:
+            connect += ',bind=' + socat_fmt(fromip)
+        return (connect, listen)
+
+    def tcp_upload(self, datafile, cs, ss, connectip, connectport,
+                   listenip=None, listenport=None, fromip=None):
+        connect, listen = self._tcp_socat(datafile, connectip, connectport,
+                                          listenip, listenport, fromip)
+        self.socat_upload(datafile, cs, ss, connect, listen)
+
+    def tcp_download(self, datafile, cs, ss, connectip, connectport,
+                     listenip=None, listenport=None, fromip=None):
+        connect, listen = self._tcp_socat(datafile, connectip, connectport,
+                                          listenip, listenport, fromip)
+        self.socat_download(datafile, cs, ss, connect, listen)
+
+    def udp_transfer(self, datafile, cs, ss, connectip, connectport,
+                     listenip=None, listenport=None, fromip=None):
+        v6 = isinstance(connectip, ipaddress.IPv6Address)
+        if listenport is None:
+            listenport = connectport
+        if v6:
+            connect = 'UDP6:[{}]:{},ipv6only,shut-null'.format(connectip, connectport)
+            listen = 'UDP6-LISTEN:{},ipv6only,null-eof'.format(listenport)
+            if listenip is not None:
+                assert isinstance(listenip, ipaddress.IPv6Address)
+                listen += ',bind=[{}]'.format(listenip)
+        else:
+            connect = 'UDP4:{}:{},shut-null'.format(connectip, connectport)
+            listen = 'UDP4-LISTEN:{},null-eof'.format(listenport)
+            if listenip is not None:
+                assert isinstance(listenip, ipaddress.IPv4Address)
+                listen += ',bind={}'.format(listenip)
+
+        self.socat_upload(datafile, cs, ss, connect, listen)
+
+    def subsetup(self, datafile, cs, ss, ip4, ip6, port,
+                 listen_ip4=None, listen_ip6=None, listenport=None,
+                 from_ip4=None, from_ip6=None):
+        cs.require_cmds('socat', 'cat')
+        ss.require_cmds('socat', 'cat')
+
+        Tasst.subsetup(self, BaseTransferTasst,
+                       TasstSubData(datafile=datafile, cs=cs, ss=ss,
+                                    ip4=ip4, ip6=ip6, port=port,
+                                    listen_ip4=listen_ip4, listen_ip6=listen_ip6,
+                                    listenport=listenport,
+                                    from_ip4=from_ip4, from_ip6=from_ip6))
+
+
+class TcpUploadTasst(BaseTransferTasst):
+    """
+    :avocado: disable
+    """
+
+    def test_tcp4_upload(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.tcp_upload(sub.datafile, sub.cs, sub.ss, sub.ip4, sub.port,
+                        listenip=sub.listen_ip4, listenport=sub.listenport,
+                        fromip=sub.from_ip4)
+
+    def test_tcp6_upload(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.tcp_upload(sub.datafile, sub.cs, sub.ss, sub.ip6, sub.port,
+                        listenip=sub.listen_ip6, listenport=sub.listenport,
+                        fromip=sub.from_ip6)
+
+
+class MetaTcpUploadTasst(TcpUploadTasst):
+    """Ugly workaround for
+    https://github.com/avocado-framework/avocado/issues/5680.
+    Explicitly apply the "meta" tag to the tests in TransferTasst.
+
+    :avocado: disable
+    :avocado: tags=meta
+
+    """
+
+    def test_tcp4_upload(self):
+        super().test_tcp4_upload()
+
+    def test_tcp6_upload(self):
+        super().test_tcp6_upload()
+
+
+class UdpTransferTasst(BaseTransferTasst):
+    """
+    :avocado: disable
+    """
+
+    def test_udp4_transfer(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.udp_transfer(sub.datafile, sub.cs, sub.ss, sub.ip4, sub.port,
+                          listenip=sub.listen_ip4, listenport=sub.listenport,
+                          fromip=sub.from_ip4)
+
+    def test_udp6_transfer(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.udp_transfer(sub.datafile, sub.cs, sub.ss, sub.ip6, sub.port,
+                          listenip=sub.listen_ip6, listenport=sub.listenport,
+                          fromip=sub.from_ip6)
+
+
+class MetaUdpTransferTasst(UdpTransferTasst):
+    """Ugly workaround for
+    https://github.com/avocado-framework/avocado/issues/5680.
+    Explicitly apply the "meta" tag to the tests in TransferTasst.
+
+    :avocado: disable
+    :avocado: tags=meta
+
+    """
+
+    def test_udp4_transfer(self):
+        super().test_udp4_transfer()
+
+    def test_udp6_transfer(self):
+        super().test_udp6_transfer()
+
+
+LOOPBACK4 = ipaddress.ip_address('127.0.0.1')
+LOOPBACK6 = ipaddress.ip_address('::1')
+
+
+class BaseLocalTransferTasst(MetaTcpUploadTasst, MetaUdpTransferTasst):
+    """Test the transfer helpers
+    :avocado: disable
+    """
+
+    PORT = 10000
+
+    def setUp(self):
+        super().setUp()
+        self.ns = UnshareSite(type(self).__name__ + '.netns', '-Un')
+        self.ns.ifup('lo')
+
+    def tearDown(self):
+        self.ns.close()
+        super().tearDown()
+
+
+class LocalTransferTasst(BaseLocalTransferTasst):
+    """Test the transfer helpers
+    """
+
+    def setUp(self):
+        super().setUp()
+        BaseTransferTasst.subsetup(self, 'test/small.bin', self.ns, self.ns,
+                                   LOOPBACK4, LOOPBACK6, self.PORT)
+        
+
+class LocalTransferTasstOptions(BaseLocalTransferTasst):
+    def setUp(self):
+        super().setUp()
+        BaseTransferTasst.subsetup(self, 'test/small.bin', self.ns, self.ns,
+                                   LOOPBACK4, LOOPBACK6, self.PORT,
+                                   listen_ip4=LOOPBACK4, listen_ip6=LOOPBACK6,
+                                   listenport=self.PORT,
+                                   from_ip4=LOOPBACK4, from_ip6=LOOPBACK6)
-- 
@@ -0,0 +1,227 @@
+#! /usr/bin/python3
+
+# SPDX-License-Identifier: GPL-2.0-or-later
+#
+# tasst - Test A Simple Socket Transport
+#         library of test helpers for passt & pasta
+#
+# tasst/transfer.py - Helpers for testing data transfers
+#
+# Copyright Red Hat
+# Author: David Gibson <david@gibson.dropbear.id.au>
+
+import ipaddress
+import time
+
+import avocado
+
+from tasst import Tasst, TasstSubData
+from tasst.site import Site
+from tasst.nstool import UnshareSite
+
+
+# HACK: how long to wait for the server to be ready and listening (s)
+SERVER_READY_DELAY = 0.05  # 1/20th of a second
+
+
+# socat needs IPv6 addresses in square brackets
+def socat_fmt(ip):
+    if isinstance(ip, ipaddress.IPv6Address):
+        return '[{}]'.format(ip)
+    elif isinstance(ip, ipaddress.IPv4Address):
+        return '{}'.format(ip)
+    else:
+        raise TypeError
+
+
+class BaseTransferTasst(Tasst):
+    def socat_upload(self, datafile, cs, ss, connect, listen):
+        server = ss.bg('socat -u {} STDOUT'.format(listen), verbose=False)
+        time.sleep(SERVER_READY_DELAY)
+        cs.fg('socat -u OPEN:{} {}'.format(datafile, connect))
+        res = server.run()
+        self.assertEquals(res.exit_status, 0)
+        srcdata = cs.output('cat {}'.format(datafile), verbose=False)
+        self.assertEquals(srcdata, res.stdout)
+
+    def socat_download(self, datafile, cs, ss, connect, listen):
+        server = ss.bg('socat -u OPEN:{} {}'.format(datafile, listen))
+        time.sleep(SERVER_READY_DELAY)
+        dstdata = cs.output('socat -u {} STDOUT'.format(connect), verbose=False)
+        res = server.run()
+        self.assertEquals(res.exit_status, 0)
+        srcdata = ss.output('cat {}'.format(datafile), verbose=False)
+        self.assertEquals(srcdata, dstdata)
+
+    def _tcp_socat(self, datafile, connectip, connectport,
+                   listenip, listenport, fromip):
+        v6 = isinstance(connectip, ipaddress.IPv6Address)
+        if listenport is None:
+            listenport = connectport
+        if v6:
+            connect = 'TCP6:[{}]:{},ipv6only'.format(connectip, connectport)
+            listen = 'TCP6-LISTEN:{},ipv6only'.format(listenport)
+        else:
+            connect = 'TCP4:{}:{}'.format(connectip, connectport)
+            listen = 'TCP4-LISTEN:{}'.format(listenport)
+        if listenip is not None:
+            listen += ',bind=' + socat_fmt(listenip)
+        if fromip is not None:
+            connect += ',bind=' + socat_fmt(fromip)
+        return (connect, listen)
+
+    def tcp_upload(self, datafile, cs, ss, connectip, connectport,
+                   listenip=None, listenport=None, fromip=None):
+        connect, listen = self._tcp_socat(datafile, connectip, connectport,
+                                          listenip, listenport, fromip)
+        self.socat_upload(datafile, cs, ss, connect, listen)
+
+    def tcp_download(self, datafile, cs, ss, connectip, connectport,
+                     listenip=None, listenport=None, fromip=None):
+        connect, listen = self._tcp_socat(datafile, connectip, connectport,
+                                          listenip, listenport, fromip)
+        self.socat_download(datafile, cs, ss, connect, listen)
+
+    def udp_transfer(self, datafile, cs, ss, connectip, connectport,
+                     listenip=None, listenport=None, fromip=None):
+        v6 = isinstance(connectip, ipaddress.IPv6Address)
+        if listenport is None:
+            listenport = connectport
+        if v6:
+            connect = 'UDP6:[{}]:{},ipv6only,shut-null'.format(connectip, connectport)
+            listen = 'UDP6-LISTEN:{},ipv6only,null-eof'.format(listenport)
+            if listenip is not None:
+                assert isinstance(listenip, ipaddress.IPv6Address)
+                listen += ',bind=[{}]'.format(listenip)
+        else:
+            connect = 'UDP4:{}:{},shut-null'.format(connectip, connectport)
+            listen = 'UDP4-LISTEN:{},null-eof'.format(listenport)
+            if listenip is not None:
+                assert isinstance(listenip, ipaddress.IPv4Address)
+                listen += ',bind={}'.format(listenip)
+
+        self.socat_upload(datafile, cs, ss, connect, listen)
+
+    def subsetup(self, datafile, cs, ss, ip4, ip6, port,
+                 listen_ip4=None, listen_ip6=None, listenport=None,
+                 from_ip4=None, from_ip6=None):
+        cs.require_cmds('socat', 'cat')
+        ss.require_cmds('socat', 'cat')
+
+        Tasst.subsetup(self, BaseTransferTasst,
+                       TasstSubData(datafile=datafile, cs=cs, ss=ss,
+                                    ip4=ip4, ip6=ip6, port=port,
+                                    listen_ip4=listen_ip4, listen_ip6=listen_ip6,
+                                    listenport=listenport,
+                                    from_ip4=from_ip4, from_ip6=from_ip6))
+
+
+class TcpUploadTasst(BaseTransferTasst):
+    """
+    :avocado: disable
+    """
+
+    def test_tcp4_upload(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.tcp_upload(sub.datafile, sub.cs, sub.ss, sub.ip4, sub.port,
+                        listenip=sub.listen_ip4, listenport=sub.listenport,
+                        fromip=sub.from_ip4)
+
+    def test_tcp6_upload(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.tcp_upload(sub.datafile, sub.cs, sub.ss, sub.ip6, sub.port,
+                        listenip=sub.listen_ip6, listenport=sub.listenport,
+                        fromip=sub.from_ip6)
+
+
+class MetaTcpUploadTasst(TcpUploadTasst):
+    """Ugly workaround for
+    https://github.com/avocado-framework/avocado/issues/5680.
+    Explicitly apply the "meta" tag to the tests in TransferTasst.
+
+    :avocado: disable
+    :avocado: tags=meta
+
+    """
+
+    def test_tcp4_upload(self):
+        super().test_tcp4_upload()
+
+    def test_tcp6_upload(self):
+        super().test_tcp6_upload()
+
+
+class UdpTransferTasst(BaseTransferTasst):
+    """
+    :avocado: disable
+    """
+
+    def test_udp4_transfer(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.udp_transfer(sub.datafile, sub.cs, sub.ss, sub.ip4, sub.port,
+                          listenip=sub.listen_ip4, listenport=sub.listenport,
+                          fromip=sub.from_ip4)
+
+    def test_udp6_transfer(self):
+        sub = self.get_subsetup(BaseTransferTasst)
+        self.udp_transfer(sub.datafile, sub.cs, sub.ss, sub.ip6, sub.port,
+                          listenip=sub.listen_ip6, listenport=sub.listenport,
+                          fromip=sub.from_ip6)
+
+
+class MetaUdpTransferTasst(UdpTransferTasst):
+    """Ugly workaround for
+    https://github.com/avocado-framework/avocado/issues/5680.
+    Explicitly apply the "meta" tag to the tests in TransferTasst.
+
+    :avocado: disable
+    :avocado: tags=meta
+
+    """
+
+    def test_udp4_transfer(self):
+        super().test_udp4_transfer()
+
+    def test_udp6_transfer(self):
+        super().test_udp6_transfer()
+
+
+LOOPBACK4 = ipaddress.ip_address('127.0.0.1')
+LOOPBACK6 = ipaddress.ip_address('::1')
+
+
+class BaseLocalTransferTasst(MetaTcpUploadTasst, MetaUdpTransferTasst):
+    """Test the transfer helpers
+    :avocado: disable
+    """
+
+    PORT = 10000
+
+    def setUp(self):
+        super().setUp()
+        self.ns = UnshareSite(type(self).__name__ + '.netns', '-Un')
+        self.ns.ifup('lo')
+
+    def tearDown(self):
+        self.ns.close()
+        super().tearDown()
+
+
+class LocalTransferTasst(BaseLocalTransferTasst):
+    """Test the transfer helpers
+    """
+
+    def setUp(self):
+        super().setUp()
+        BaseTransferTasst.subsetup(self, 'test/small.bin', self.ns, self.ns,
+                                   LOOPBACK4, LOOPBACK6, self.PORT)
+        
+
+class LocalTransferTasstOptions(BaseLocalTransferTasst):
+    def setUp(self):
+        super().setUp()
+        BaseTransferTasst.subsetup(self, 'test/small.bin', self.ns, self.ns,
+                                   LOOPBACK4, LOOPBACK6, self.PORT,
+                                   listen_ip4=LOOPBACK4, listen_ip6=LOOPBACK6,
+                                   listenport=self.PORT,
+                                   from_ip4=LOOPBACK4, from_ip6=LOOPBACK6)
-- 
2.40.1


  parent reply	other threads:[~2023-05-16  2:01 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-16  2:01 [PATCH v2 00/21] RFCv2: Proof-of-concept conversion of some tests to Avocado framework David Gibson
2023-05-16  2:01 ` [PATCH v2 01/21] test/nstool: Provide useful error if given a path that's too long David Gibson
2023-05-16  2:01 ` [PATCH v2 02/21] test/nstool: Fix fd leak in accept() loop David Gibson
2023-05-16  2:01 ` [PATCH v2 03/21] avocado: Make a duplicate copy of testsuite for comparison purposes David Gibson
2023-05-16  2:01 ` [PATCH v2 04/21] avocado: Don't double download assets for test/ and oldtest/ David Gibson
2023-05-16  2:01 ` [PATCH v2 05/21] avocado: Move static checkers to avocado David Gibson
2023-05-16  2:01 ` [PATCH v2 06/21] avocado/tasst: Helper functions for executing commands in different places David Gibson
2023-05-16  2:01 ` [PATCH v2 07/21] avocado: Convert build tests to avocado David Gibson
2023-05-16  2:01 ` [PATCH v2 08/21] avocado/tasst: Add helpers for running background commands on sites David Gibson
2023-05-16  2:01 ` [PATCH v2 09/21] avocado/tasst: Add helper to get network interface names for a site David Gibson
2023-05-16  2:01 ` [PATCH v2 10/21] avocado/tasst: Add helpers to run commands with nstool David Gibson
2023-05-16  2:01 ` [PATCH v2 11/21] avocado/tasst: Add ifup and network address helpers to Site David Gibson
2023-05-16  2:01 ` [PATCH v2 12/21] avocado/tasst: Helper for creating veth devices between namespaces David Gibson
2023-05-16  2:01 ` [PATCH v2 13/21] avocado/tasst: Add helper for getting MTU of a network interface David Gibson
2023-05-16  2:01 ` [PATCH v2 14/21] avocado/tasst: Add helper to wait for IP address to appear David Gibson
2023-05-16  2:01 ` [PATCH v2 15/21] avocado/tasst: Add helpers for getting a site's routes David Gibson
2023-05-16  2:01 ` David Gibson [this message]
2023-05-16  2:01 ` [PATCH v2 17/21] avocado/tasst: IP address allocation helpers David Gibson
2023-05-16  2:01 ` [PATCH v2 18/21] avocado/tasst: Helpers for testing NDP behaviour David Gibson
2023-05-16  2:01 ` [PATCH v2 19/21] avocado/tasst: Helpers for testing DHCP & DHCPv6 behaviour David Gibson
2023-05-16  2:01 ` [PATCH v2 20/21] avocado/tasst: Helpers to construct a simple network environment for tests David Gibson
2023-05-16  2:01 ` [PATCH v2 21/21] avocado: Convert basic pasta tests David Gibson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230516020135.1901256-17-david@gibson.dropbear.id.au \
    --to=david@gibson.dropbear.id.au \
    --cc=crosa@redhat.com \
    --cc=jarichte@redhat.com \
    --cc=passt-dev@passt.top \
    --cc=sbrivio@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).