From: Remi Gacogne <remi.gacogne@powerdns.com>
Date: Wed, 13 Jan 2021 08:37:13 +0000 (+0100)
Subject: dnsdist: Use only one TCP worker for the per-thread Lua regression test
X-Git-Tag: dnsdist-1.6.0-alpha1~24^2~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a8246eeeb91cc96963f280a81b7b953c4ed304bc;p=thirdparty%2Fpdns.git

dnsdist: Use only one TCP worker for the per-thread Lua regression test

Otherwise the distribution of queries to the backends is very hard
to predict since every thread uses its own counter. We used to start
only one worker thread at startup, and did not send enough queries
to get additional workers fired up.
---

diff --git a/regression-tests.dnsdist/test_Routing.py b/regression-tests.dnsdist/test_Routing.py
index 8aabad85fa..77c0c231d7 100644
--- a/regression-tests.dnsdist/test_Routing.py
+++ b/regression-tests.dnsdist/test_Routing.py
@@ -258,6 +258,9 @@ class TestRoutingLuaFFIPerThreadRoundRobinLB(DNSDistTest):
     _testServer2Port = 5351
     _config_params = ['_testServerPort', '_testServer2Port']
     _config_template = """
+    -- otherwise we start too many TCP workers, and as each thread
+    -- uses it own counter this makes the TCP queries distribution hard to predict
+    setMaxTCPClientThreads(1)
     setServerPolicyLuaFFIPerThread("luaffiroundrobin", [[
       local ffi = require("ffi")
       local C = ffi.C