zuul/zuul-jobs
Code Issues Proposed changes

Fix some item usage where var is zj_* now

Browse Source 
In the fetch-sphinx-tarball role our handling of pdfs didn't fully
migrate the loop control var 'item' renaming. Update to replace now
incorrect 'item' with correct 'zj_' prefixed var names.

Change-Id: Ia5cb68ccbc410d88349e9dd6e3fc2551d5eff5e2
This commit is contained in:
Clark Boylan 2020-05-01 12:08:49 -07:00
parent 10cd19fdae
commit 1075767b18
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
roles/fetch-sphinx-tarball/tasks/pdf.yaml
View File

@ -39,14 +39,14 @@
- name: Fetch PDF files - name: Fetch PDF files
synchronize: synchronize:
dest: "{{ zuul.executor.log_root }}/pdf/{{ item.item }}" dest: "{{ zuul.executor.log_root }}/pdf/{{ zj_pdf.item }}"
mode: pull mode: pull
src: "{{ item.stat.path }}" src: "{{ zj_pdf.stat.path }}"
verify_host: true verify_host: true
with_items: "{{ pdf_file_stat.results }}" with_items: "{{ pdf_file_stat.results }}"
loop_control: loop_control:
loop_var: zj_pdf loop_var: zj_pdf
when: item.stat.exists when: zj_pdf.stat.exists
- name: Return PDF artifact to Zuul - name: Return PDF artifact to Zuul
@ -61,4 +61,4 @@
with_items: "{{ pdf_file_stat.results }}" with_items: "{{ pdf_file_stat.results }}"
loop_control: loop_control:
loop_var: zj_pdf loop_var: zj_pdf
when: item.stat.exists when: zj_pdf.stat.exists