zuul/zuul-jobs
Code Issues Proposed changes

Actually set zuul_log_path

Browse Source 
We renamed log_path to zuul_log_path everywhere but where we set the
fact. Make sure the fact setting uses the name we expect everywhere
else.

Change-Id: Ib4616ab4634ae7c5a5e93ea4e49bf218e125a9cb
This commit is contained in:
Clark Boylan 2017-09-10 11:03:48 -07:00
parent 00e673c36b
commit 166236eab3
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
roles/set-zuul-log-path-fact/tasks/main.yaml
View File

@ -1,9 +1,9 @@
- name: Set log path for a change - name: Set log path for a change
when: zuul.change is defined when: zuul.change is defined
set_fact: set_fact:
log_path: "{{ zuul.change[-2:] }}/{{ zuul.change }}/{{ zuul.patchset }}/{{ zuul.pipeline }}/{{ zuul.job }}/{{ zuul.build[:7] }}" zuul_log_path: "{{ zuul.change[-2:] }}/{{ zuul.change }}/{{ zuul.patchset }}/{{ zuul.pipeline }}/{{ zuul.job }}/{{ zuul.build[:7] }}"
- name: Set log path for a ref update - name: Set log path for a ref update
when: zuul.newrev is defined when: zuul.newrev is defined
set_fact: set_fact:
log_path: "{{ zuul.newrev[-2:] }}/{{ zuul.newrev }}/{{ zuul.pipeline }}/{{ zuul.job }}/{{ zuul.build[:7] }}" zuul_log_path: "{{ zuul.newrev[-2:] }}/{{ zuul.newrev }}/{{ zuul.pipeline }}/{{ zuul.job }}/{{ zuul.build[:7] }}"