fix: ci: Don't forward parent yaml variables to stress child pipelines

The global RUNNER_SCRIPT_TIMEOUT: 55m in the parent pipeline was being
forwarded to the stress and tsan:stress child pipelines, where forwarded
yaml variables outrank job-level variables. That caused stress jobs with
BIND_STRESS_TESTS_RUN_TIME >= 60 to be killed at 55 minutes, regardless
of the per-job RUNNER_SCRIPT_TIMEOUT set in the generated child config.

Set forward:yaml_variables: false on both trigger jobs; the generated
configs already declare every variable they need.

Assisted-by: Claude:claude-opus-4-7

Merge branch 'mnowak/fix-stress-test-script-timeout' into 'main'

See merge request isc-projects/bind9!12012
This commit is contained in:
Michal Nowak 2026-05-13 18:41:42 +02:00
commit d8e196a76a

View file

@ -1724,6 +1724,13 @@ unit:clang:tsan:
tsan:stress:
<<: *default_triggering_rules
stage: system
# In a downstream pipeline, yaml variables inherited from the parent outrank
# the child's job-level `variables:` block. Inherit only what the child
# actually needs (image lookup), so the parent's RUNNER_SCRIPT_TIMEOUT does
# not override the per-job values set in the generated child config.
inherit:
variables:
- CI_REGISTRY_IMAGE
variables:
PARENT_PIPELINE_ID: $CI_PIPELINE_ID
trigger:
@ -2477,6 +2484,13 @@ shotgun:doh-get:
stress-test-child-pipeline:
<<: *default_triggering_rules
stage: performance
# In a downstream pipeline, yaml variables inherited from the parent outrank
# the child's job-level `variables:` block. Inherit only what the child
# actually needs (image lookup), so the parent's RUNNER_SCRIPT_TIMEOUT does
# not override the per-job values set in the generated child config.
inherit:
variables:
- CI_REGISTRY_IMAGE
rules:
- if: '$CI_MERGE_REQUEST_DIFF_BASE_SHA != null'
changes: