From 1b8f0711f2610339a9191483d7ea1de6ec356e4f Mon Sep 17 00:00:00 2001
From: Tom Krizek <tkrizek@isc.org>
Date: Tue, 30 May 2023 14:11:14 +0200
Subject: [PATCH] Fix base_port calculation in pytest runner

The selected base port should be in the range <port_min, port_max), the
formula was incorrect.

Credit for discovering this fault goes to Ondrej Sury.

(cherry picked from commit e8ea6b610b775f3afe34b76883385b9ab53cf462)
---
 bin/tests/system/conftest.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/tests/system/conftest.py b/bin/tests/system/conftest.py
index efcc2f51ff..9294198ba1 100644
--- a/bin/tests/system/conftest.py
+++ b/bin/tests/system/conftest.py
@@ -312,7 +312,7 @@ else:
         # worker threads, multiple tests may have same port values assigned. If
         # these tests are then executed simultaneously, the test results will
         # be misleading.
-        base_port = int(time.time() // 3600) % (port_max - port_min)
+        base_port = int(time.time() // 3600) % (port_max - port_min) + port_min
 
         return {mod: base_port + i * PORTS_PER_TEST for i, mod in enumerate(modules)}