[reproducer] Pass extra_variable_files to deployment args for PreMetal#4010
[reproducer] Pass extra_variable_files to deployment args for PreMetal#4010srac0 wants to merge 1 commit into
Conversation
srac0
force-pushed
the
extra-var-files-pre-metal
branch
from
09a0b28 to
1d6a9c7
Compare
|
New changes are detected. LGTM label has been removed. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
srac0
changed the title
srac0
force-pushed
the
extra-var-files-pre-metal
branch
from
1d6a9c7 to
4a1192c
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 44m 38s |
srac0
force-pushed
the
extra-var-files-pre-metal
branch
2 times, most recently
from
5578c4e to
5979158
Compare
PreMetal provisions hosts from the main branch, generating reproducer-variables.yml with base scenario vars. During the actual Zuul job, overwrite_zuul_vars.yml only merges zuul_vars.yaml into reproducer-variables.yml, ignoring extra_variable_files entirely. Additionally, the slurp + copy-content pattern for transferring reproducer-variables.yml triggers Jinja2 evaluation on the file content, breaking when the bootstrap bakes raw templates into nested structures. Signed-off-by: Samuel Rac <srac@redhat.com>
srac0
force-pushed
the
extra-var-files-pre-metal
branch
from
5979158 to
5eea3b0
Compare
2 participants
PreMetal provisions hosts from the main branch, generating reproducer-variables.yml with base scenario vars. During the actual Zuul job, overwrite_zuul_vars.yml only merges zuul_vars.yaml into reproducer-variables.yml, ignoring extra_variable_files entirely.
This means any scenario override passed via extra_variable_files is silently lost on the controller. The base values from PreMetal provisioning win, causing jobs to run the wrong test plan.
Iteratively merge each extra_variable_files entry into reproducer-variables.yml before the zuul_vars.yaml merge, preserving the correct precedence: base < extra_variable_files < zuul_vars.