Skip to content

Commit 1d6a9c7

Browse files
committed
[reproducer] Pass extra_variable_files to deployment args for PreMetal
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 append each extra_variable_files entry as `-e @<file>` to cifmw_deploy_architecture_args and cifmw_post_deploy_args before the zuul_vars.yaml merge, preserving the correct precedence: base < extra_variable_files < zuul_vars.
1 parent 21dbd39 commit 1d6a9c7

1 file changed

Lines changed: 13 additions & 0 deletions

File tree

roles/reproducer/tasks/premetal.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,19 @@
3030
loop_control:
3131
loop_var: secret_file
3232

33+
- name: Pass extra variable files to deployment scripts
34+
when: extra_variable_files is defined
35+
ansible.builtin.set_fact:
36+
cifmw_deploy_architecture_args: >
37+
{{ cifmw_deploy_architecture_args | default('') }}
38+
-e @{{ extra_variable_file }}
39+
cifmw_post_deploy_args: >
40+
{{ cifmw_post_deploy_args | default('') }}
41+
-e @{{ extra_variable_file }}
42+
loop: "{{ extra_variable_files }}"
43+
loop_control:
44+
loop_var: extra_variable_file
45+
3346
- name: Print final cifmw_deploy_architecture_args
3447
ansible.builtin.debug:
3548
msg: >

0 commit comments

Comments
 (0)