From: Remi Gacogne <remi.gacogne@powerdns.com>
Date: Tue, 28 Mar 2023 15:22:27 +0000 (+0200)
Subject: dnsdist: Wait a bit longer for the process to exit before killing it
X-Git-Tag: rec-4.9.0-alpha1~23^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F12697%2Fhead;p=thirdparty%2Fpdns.git

dnsdist: Wait a bit longer for the process to exit before killing it

This prevents a long list of
```
kill... <Popen: returncode: None args: ['../pdns/dnsdistdist/dnsdist', '--supervised...>
```
when running with TSAN-enabled.
In my tests it did not seem to really slow the build: roughly 3%
slower but that might even be in the error margin for these tests.
---

diff --git a/regression-tests.dnsdist/dnsdisttests.py b/regression-tests.dnsdist/dnsdisttests.py
index 05de5e1bd6..dfc6ead966 100644
--- a/regression-tests.dnsdist/dnsdisttests.py
+++ b/regression-tests.dnsdist/dnsdisttests.py
@@ -174,7 +174,7 @@ class DNSDistTest(AssertEqualDNSMessageMixin, unittest.TestCase):
             return
         try:
             p.terminate()
-            for count in range(10):
+            for count in range(20):
                 x = p.poll()
                 if x is not None:
                     break