Merge pull request #10638 from pytest-dev/backport-10607-to-7.2.x

nicoddemus · web-flow · commit 211d08e9bca5 · 2023-01-06T09:33:55.000-03:00
[7.2.x] Mitigate directory creation race condition
diff --git a/changelog/10607.bugfix.rst b/changelog/10607.bugfix.rst
@@ -0,0 +1 @@
+Fix a race condition when creating junitxml reports, which could occur when multiple instances of pytest execute in parallel.
diff --git a/src/_pytest/junitxml.py b/src/_pytest/junitxml.py
@@ -645,8 +645,8 @@ def pytest_sessionstart(self) -> None:
 
     def pytest_sessionfinish(self) -> None:
         dirname = os.path.dirname(os.path.abspath(self.logfile))
-        if not os.path.isdir(dirname):
-            os.makedirs(dirname)
+        # exist_ok avoids filesystem race conditions between checking path existence and requesting creation
+        os.makedirs(dirname, exist_ok=True)
 
         with open(self.logfile, "w", encoding="utf-8") as logfile:
             suite_stop_time = timing.time()

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Fix a race condition when creating junitxml reports, which could occur when multiple instances of pytest execute in parallel.`