ci: build aarch64 wheel on GHA aarch64 runner (#2503)

Signed-off-by: mayeut <[email protected]>

Author: mayeut

.github/workflows/build.yml

@@ -27,7 +27,7 @@ jobs:
         include:
         - {os: ubuntu-latest, arch: x86_64}
         - {os: ubuntu-latest, arch: i686}
-        - {os: ubuntu-latest, arch: aarch64}
+        - {os: ubuntu-24.04-arm, arch: aarch64}
         - {os: macos-13, arch: x86_64}
         - {os: macos-14, arch: arm64}
         - {os: windows-2019, arch: AMD64}
@@ -48,18 +48,11 @@ jobs:
       with:
         python-version: 3.11
 
-    - name: Set up QEMU
-      uses: docker/setup-qemu-action@v3
-      if: matrix.arch == 'aarch64'
-
     - name: Create wheels + run tests
       uses: pypa/[email protected]
       env:
         CIBW_ARCHS: "${{ matrix.arch }}"
-        CIBW_ENABLE: "cpython-prerelease"
-        CIBW_TEST_EXTRAS: test
-        CIBW_TEST_COMMAND:
-          make -C {project} PYTHON="env python" PSUTIL_SCRIPTS_DIR="{project}/scripts" install-sysdeps install-pydeps-test install print-sysinfo test test-memleaks
+        CIBW_ENABLE: "${{ startsWith(github.ref, 'refs/tags/') && '' || 'cpython-prerelease' }}"
 
     - name: Upload wheels
       uses: actions/upload-artifact@v4

.gitignore

@@ -18,3 +18,4 @@ syntax: glob
 build/
 dist/
 wheelhouse/
+.tests/

Makefile

@@ -162,6 +162,13 @@ test-coverage:  ## Run test coverage.
 	$(PYTHON) -m coverage html
 	$(PYTHON) -m webbrowser -t htmlcov/index.html
 
+test-ci:
+	${MAKE} install-sysdeps
+	mkdir -p .tests
+	cd .tests/ && $(PYTHON) -c "from psutil.tests import print_sysinfo; print_sysinfo()"
+	cd .tests/ && $(PYTHON_ENV_VARS) PYTEST_ADDOPTS="-k 'not test_memleaks.py'" $(PYTHON) -m pytest $(PYTEST_ARGS) --pyargs psutil.tests
+	cd .tests/ && $(PYTHON_ENV_VARS) PYTEST_ADDOPTS="-k 'test_memleaks.py'"     $(PYTHON) -m pytest $(PYTEST_ARGS) --pyargs psutil.tests
+
 # ===================================================================
 # Linters
 # ===================================================================

psutil/tests/__init__.py

@@ -73,7 +73,7 @@
     "HAS_IONICE", "HAS_MEMORY_MAPS", "HAS_PROC_CPU_NUM", "HAS_RLIMIT",
     "HAS_SENSORS_BATTERY", "HAS_BATTERY", "HAS_SENSORS_FANS",
     "HAS_SENSORS_TEMPERATURES", "HAS_NET_CONNECTIONS_UNIX", "MACOS_11PLUS",
-    "MACOS_12PLUS", "COVERAGE", 'AARCH64', "QEMU_USER", "PYTEST_PARALLEL",
+    "MACOS_12PLUS", "COVERAGE", 'AARCH64', "PYTEST_PARALLEL",
     # subprocesses
     'pyrun', 'terminate', 'reap_children', 'spawn_testproc', 'spawn_zombie',
     'spawn_children_pair',
@@ -114,11 +114,6 @@
 CI_TESTING = GITHUB_ACTIONS
 COVERAGE = 'COVERAGE_RUN' in os.environ
 PYTEST_PARALLEL = "PYTEST_XDIST_WORKER" in os.environ  # `make test-parallel`
-if LINUX and GITHUB_ACTIONS:
-    with open('/proc/1/cmdline') as f:
-        QEMU_USER = "/bin/qemu-" in f.read()
-else:
-    QEMU_USER = False
 # are we a 64 bit process?
 IS_64BIT = sys.maxsize > 2**32
 AARCH64 = platform.machine() == "aarch64"

psutil/tests/test_contracts.py

@@ -27,7 +27,6 @@
 from psutil.tests import HAS_NET_IO_COUNTERS
 from psutil.tests import HAS_SENSORS_FANS
 from psutil.tests import HAS_SENSORS_TEMPERATURES
-from psutil.tests import QEMU_USER
 from psutil.tests import SKIP_SYSCONS
 from psutil.tests import PsutilTestCase
 from psutil.tests import create_sockets
@@ -268,7 +267,6 @@ def test_net_if_addrs(self):
                 assert isinstance(addr.netmask, (str, type(None)))
                 assert isinstance(addr.broadcast, (str, type(None)))
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_net_if_stats(self):
         # Duplicate of test_system.py. Keep it anyway.
         for ifname, info in psutil.net_if_stats().items():

psutil/tests/test_linux.py

@@ -12,6 +12,7 @@
 import errno
 import io
 import os
+import platform
 import re
 import shutil
 import socket
@@ -32,7 +33,6 @@
 from psutil.tests import HAS_RLIMIT
 from psutil.tests import PYPY
 from psutil.tests import PYTEST_PARALLEL
-from psutil.tests import QEMU_USER
 from psutil.tests import TOLERANCE_DISK_USAGE
 from psutil.tests import TOLERANCE_SYS_MEM
 from psutil.tests import PsutilTestCase
@@ -735,6 +735,9 @@ def test_against_lscpu(self):
                 core_ids.add(fields[1])
         assert psutil.cpu_count(logical=False) == len(core_ids)
 
+    @pytest.mark.skipif(
+        platform.machine() not in {"x86_64", "i686"}, reason="x86_64/i686 only"
+    )
     def test_method_2(self):
         meth_1 = psutil._pslinux.cpu_count_cores()
         with mock.patch('glob.glob', return_value=[]) as m:
@@ -754,6 +757,9 @@ def test_emulate_none(self):
 @pytest.mark.skipif(not LINUX, reason="LINUX only")
 class TestSystemCPUFrequency(PsutilTestCase):
     @pytest.mark.skipif(not HAS_CPU_FREQ, reason="not supported")
+    @pytest.mark.skipif(
+        AARCH64, reason="aarch64 does not always expose frequency"
+    )
     def test_emulate_use_second_file(self):
         # https://github.com/giampaolo/psutil/issues/981
         def path_exists_mock(path):
@@ -977,7 +983,6 @@ def test_ips(self):
 
 
 @pytest.mark.skipif(not LINUX, reason="LINUX only")
-@pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
 class TestSystemNetIfStats(PsutilTestCase):
     @pytest.mark.skipif(
         not shutil.which("ifconfig"), reason="ifconfig utility not available"
@@ -1546,7 +1551,7 @@ def test_issue_687(self):
         with ThreadTask():
             p = psutil.Process()
             threads = p.threads()
-            assert len(threads) == (3 if QEMU_USER else 2)
+            assert len(threads) == 2
             tid = sorted(threads, key=lambda x: x.id)[1].id
             assert p.pid != tid
             pt = psutil.Process(tid)
@@ -2227,7 +2232,6 @@ def test_name(self):
         value = self.read_status_file("Name:")
         assert self.proc.name() == value
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_status(self):
         value = self.read_status_file("State:")
         value = value[value.find('(') + 1 : value.rfind(')')]

psutil/tests/test_memleaks.py

@@ -39,7 +39,6 @@
 from psutil.tests import HAS_SENSORS_BATTERY
 from psutil.tests import HAS_SENSORS_FANS
 from psutil.tests import HAS_SENSORS_TEMPERATURES
-from psutil.tests import QEMU_USER
 from psutil.tests import TestMemoryLeak
 from psutil.tests import create_sockets
 from psutil.tests import get_testfn
@@ -396,7 +395,6 @@ def test_disk_usage(self):
         times = FEW_TIMES if POSIX else self.times
         self.execute(lambda: psutil.disk_usage('.'), times=times)
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_disk_partitions(self):
         self.execute(psutil.disk_partitions)
 
@@ -434,7 +432,6 @@ def test_net_if_addrs(self):
         tolerance = 80 * 1024 if WINDOWS else self.tolerance
         self.execute(psutil.net_if_addrs, tolerance=tolerance)
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_net_if_stats(self):
         self.execute(psutil.net_if_stats)
 

psutil/tests/test_misc.py

@@ -29,7 +29,6 @@
 from psutil._common import supports_ipv6
 from psutil._common import wrap_numbers
 from psutil.tests import HAS_NET_IO_COUNTERS
-from psutil.tests import QEMU_USER
 from psutil.tests import PsutilTestCase
 from psutil.tests import process_namespace
 from psutil.tests import pytest
@@ -269,9 +268,6 @@ def check(ret):
         for fun, name in ns.iter(ns.getters):
             if name in {"win_service_iter", "win_service_get"}:
                 continue
-            if QEMU_USER and name == "net_if_stats":
-                # OSError: [Errno 38] ioctl(SIOCETHTOOL) not implemented
-                continue
             with self.subTest(name=name):
                 try:
                     ret = fun()

psutil/tests/test_posix.py

@@ -26,7 +26,6 @@
 from psutil.tests import AARCH64
 from psutil.tests import HAS_NET_IO_COUNTERS
 from psutil.tests import PYTHON_EXE
-from psutil.tests import QEMU_USER
 from psutil.tests import PsutilTestCase
 from psutil.tests import pytest
 from psutil.tests import retry_on_failure
@@ -103,9 +102,6 @@ def ps_name(pid):
     if SUNOS:
         field = "comm"
     command = ps(field, pid).split()
-    if QEMU_USER:
-        assert "/bin/qemu-" in command[0]
-        return command[1]
     return command[0]
 
 

psutil/tests/test_process.py

@@ -49,7 +49,6 @@
 from psutil.tests import PYPY
 from psutil.tests import PYTHON_EXE
 from psutil.tests import PYTHON_EXE_ENV
-from psutil.tests import QEMU_USER
 from psutil.tests import PsutilTestCase
 from psutil.tests import ThreadTask
 from psutil.tests import call_until
@@ -257,7 +256,6 @@ def test_cpu_percent_numcpus_none(self):
             psutil.Process().cpu_percent()
             assert m.called
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_cpu_times(self):
         times = psutil.Process().cpu_times()
         assert times.user >= 0.0, times
@@ -270,7 +268,6 @@ def test_cpu_times(self):
         for name in times._fields:
             time.strftime("%H:%M:%S", time.localtime(getattr(times, name)))
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_cpu_times_2(self):
         user_time, kernel_time = psutil.Process().cpu_times()[:2]
         utime, ktime = os.times()[:2]
@@ -642,8 +639,6 @@ def test_memory_maps(self):
                 continue
             if BSD and nt.path == "pvclock":
                 continue
-            if QEMU_USER and "/bin/qemu-" in nt.path:
-                continue
             assert os.path.isabs(nt.path), nt.path
 
             if POSIX:
@@ -710,7 +705,6 @@ def test_is_running(self):
         assert not p.is_running()
         assert not p.is_running()
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_exe(self):
         p = self.spawn_psproc()
         exe = p.exe()
@@ -763,9 +757,6 @@ def test_cmdline(self):
                 if pyexe != PYTHON_EXE:
                     assert ' '.join(p.cmdline()[1:]) == ' '.join(cmdline[1:])
                     return
-            if QEMU_USER:
-                assert ' '.join(p.cmdline()[2:]) == ' '.join(cmdline)
-                return
             assert ' '.join(p.cmdline()) == ' '.join(cmdline)
 
     @pytest.mark.skipif(PYPY, reason="broken on PYPY")
@@ -783,8 +774,6 @@ def test_long_cmdline(self):
                 assert p.cmdline() == cmdline
             except psutil.ZombieProcess:
                 raise pytest.skip("OPENBSD: process turned into zombie")
-        elif QEMU_USER:
-            assert p.cmdline()[2:] == cmdline
         else:
             ret = p.cmdline()
             if NETBSD and ret == []:
@@ -798,8 +787,7 @@ def test_name(self):
         pyexe = os.path.basename(os.path.realpath(sys.executable)).lower()
         assert pyexe.startswith(name), (pyexe, name)
 
-    @pytest.mark.skipif(PYPY or QEMU_USER, reason="unreliable on PYPY")
-    @pytest.mark.skipif(QEMU_USER, reason="unreliable on QEMU user")
+    @pytest.mark.skipif(PYPY, reason="unreliable on PYPY")
     def test_long_name(self):
         pyexe = create_py_exe(self.get_testfn(suffix=string.digits * 2))
         cmdline = [
@@ -830,7 +818,6 @@ def test_long_name(self):
     # @pytest.mark.skipif(SUNOS, reason="broken on SUNOS")
     # @pytest.mark.skipif(AIX, reason="broken on AIX")
     # @pytest.mark.skipif(PYPY, reason="broken on PYPY")
-    # @pytest.mark.skipif(QEMU_USER, reason="broken on QEMU user")
     # def test_prog_w_funky_name(self):
     #     # Test that name(), exe() and cmdline() correctly handle programs
     #     # with funky chars such as spaces and ")", see:
@@ -938,7 +925,6 @@ def cleanup(init):
             except psutil.AccessDenied:
                 pass
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     def test_status(self):
         p = psutil.Process()
         assert p.status() == psutil.STATUS_RUNNING
@@ -1166,7 +1152,6 @@ def test_parent_multi(self):
         assert grandchild.parent() == child
         assert child.parent() == parent
 
-    @pytest.mark.skipif(QEMU_USER, reason="QEMU user not supported")
     @retry_on_failure()
     def test_parents(self):
         parent = psutil.Process()