]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Drop parallel build from stress tests
authorMichal Nowak <mnowak@isc.org>
Thu, 9 Mar 2023 10:10:53 +0000 (11:10 +0100)
committerMichal Nowak <mnowak@isc.org>
Thu, 9 Mar 2023 17:24:03 +0000 (18:24 +0100)
BUILD_PARALLEL_JOBS environmental variable is set to 6, which does not
align well with 4 and 8 CPU core systems dedicated to CI "stress" tests.
When multiple parallel jobs run on the host, they compete for resources
with an undesirable result: 6 compiler processes of one job may starve
named, resulting in lower-than-expected throughput and minutes-long
query response latency spikes.

Better drop the build parallelism of BIND-under-test. About 1-2 minutes
are added to the 60-65 minutes long job duration.

.gitlab-ci.yml

index 9ba462df3803f08b2183e05aa76d59f7fa86e187..5e90f4a5f0ef349298cdf796506addda3d696c00 100644 (file)
@@ -1362,12 +1362,16 @@ respdiff-long-third-party:
 
 # "Stress" tests
 
+# Parallel build in the "make" step is avoided since multiple jobs can be
+# executed concurrently on the same runner. This may present problems when one
+# job runs a performance-sensitive task of replying to queries while another
+# takes all cores to build BIND.
 .stress: &stress_job
   stage: performance
   script:
     - *configure
     - *setup_interfaces
-    - make -j${BUILD_PARALLEL_JOBS:-1} -k all V=1
+    - make -k all V=1
     - make DESTDIR="${INSTALL_PATH}" install
     - git clone --depth 1 https://gitlab-ci-token:${CI_JOB_TOKEN}@gitlab.isc.org/isc-private/bind-qa.git
     - cd bind-qa/bind9/stress