gh-124213: Skip tests failing with --suppress-sync=true when inside systemd-nspawn container

Lib/test/support/__init__.py

@@ -61,6 +61,7 @@
     "without_optimizer",
     "force_not_colorized",
     "BrokenIter",
+    "in_systemd_nspawn", "in_systemd_nspawn_sync_suppressed",
     ]
 
 
@@ -2873,3 +2874,40 @@ def __iter__(self):
         if self.iter_raises:
             1/0
         return self
+
+
+def in_systemd_nspawn() -> bool:
+    """
+    Test whether the test suite is running inside of systemd-nspawn container
+    """
+
+    try:
+        with open("/run/systemd/container", "rb") as fp:
+            return fp.read().rstrip() == b"systemd-nspawn"
+    except FileNotFoundError:
+        return False
+
+
+def in_systemd_nspawn_sync_suppressed() -> bool:
+    """
+    Test whether the test suite is runing in systemd-nspawn with ``--suppress-sync=true``
+
+    Return True if the test suite is being run inside a systemd-nspawn
+    container and ``--suppress-sync=true`` option is detected to be used,
+    False otherwise.  This can be used to skip tests that rely
+    on ``fsync()`` calls and similar not being intercepted.
+    """
+
+    if hasattr(os, "O_SYNC") and in_systemd_nspawn():
+        import errno
+
+        # If systemd-nspawn is used, O_SYNC flag will immediately trigger
+        # EINVAL.  Otherwise, ENOENT will be given instead.
+        try:
+            with os.open("", os.O_RDONLY | os.O_SYNC):
+                pass
+        except OSError as err:
+            if err.errno == errno.EINVAL:
+                return True
+
+    return False

Lib/test/test_mmap.py

@@ -1,5 +1,6 @@
 from test.support import (
     requires, _2G, _4G, gc_collect, cpython_only, is_emscripten, is_apple,
+    in_systemd_nspawn_sync_suppressed,
 )
 from test.support.import_helper import import_module
 from test.support.os_helper import TESTFN, unlink
@@ -839,7 +840,7 @@ def test_flush_return_value(self):
         mm.write(b'python')
         result = mm.flush()
         self.assertIsNone(result)
-        if sys.platform.startswith(('linux', 'android')):
+        if sys.platform.startswith(('linux', 'android')) and not in_systemd_nspawn_sync_suppressed():
             # 'offset' must be a multiple of mmap.PAGESIZE on Linux.
             # See bpo-34754 for details.
             self.assertRaises(OSError, mm.flush, 1, len(b'python'))

Lib/test/test_os.py

@@ -2351,9 +2351,13 @@ def test_chmod(self):
 
 @unittest.skipIf(support.is_wasi, "Cannot create invalid FD on WASI.")
 class TestInvalidFD(unittest.TestCase):
-    singles = ["fchdir", "dup", "fdatasync", "fstat",
-               "fstatvfs", "fsync", "tcgetpgrp", "ttyname"]
-    singles_fildes = {"fchdir", "fdatasync", "fsync"}
+    singles = ["fchdir", "dup", "fstat", "fstatvfs", "tcgetpgrp", "ttyname"]
+    singles_fildes = {"fchdir"}
+    # systemd-nspawn --suppress-sync=true does not verify fd passed
+    # fdatasync() and fsync(), and always returns success
+    if not support.in_systemd_nspawn_sync_suppressed():
+        singles += ["fdatasync", "fsync"]
+        singles_fildes |= {"fdatasync", "fsync"}
     #singles.append("close")
     #We omit close because it doesn't raise an exception on some platforms
     def get_single(f):

Misc/NEWS.d/next/Tests/2024-09-18-18-39-21.gh-issue-124213.AQq_xg.rst

@@ -0,0 +1,3 @@
+Detect whether the test suite is running inside a systemd-nspawn container
+with ``--suppress-sync=true`` option, and skip the ``test_os``
+and ``test_mmap`` tests that are failing in this scenario.