]> 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:32:17 +0000 (18:32 +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.

(cherry picked from commit 3fd7e7c81f81e21adebde2930fe511fe335f825e)

.gitlab-ci.yml

index 7c40dc7d1abe56866aa44ce20a24ad8dda9dc26e..baa987e033b13adb6c3b02e572b299d8a7b78d75 100644 (file)
@@ -1427,12 +1427,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