zuul/zuul-jobs
Code Issues Proposed changes

Remove extra task from download-artifacts

Browse Source 
This task is no longer necessary and should have been removed in
6523b6a530 as part of the work to
download multiple artifacts.

Also fix a syntax error in a template string.

Change-Id: I6a1d067a22a55843ec3320204bb673eaa0bcab4d
This commit is contained in:
James E. Blair
2019-06-10 09:10:01 -07:00
parent de66d231d9
commit aff442e896
2 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
roles/download-artifact/tasks/inner.yaml
View File

@@ -2,7 +2,7 @@
uri: uri:
url: "{{ artifact.url }}" url: "{{ artifact.url }}"
dest: "{{ download_artifact_directory }}" dest: "{{ download_artifact_directory }}"
loop: "{{ (build.artifacts }}" loop: "{{ build.artifacts }}"
loop_control: loop_control:
loop_var: artifact loop_var: artifact
when: "artifact.name == download_artifact_name or ((download_artifact_name | type_debug) == 'list' and artifact.name in download_artifact_name)" when: "artifact.name == download_artifact_name or ((download_artifact_name | type_debug) == 'list' and artifact.name in download_artifact_name)"

3
roles/download-artifact/tasks/main.yaml
View File

@@ -2,9 +2,6 @@
uri: uri:
url: "{{ download_artifact_api }}/builds?{{ download_artifact_query }}" url: "{{ download_artifact_api }}/builds?{{ download_artifact_query }}"
register: build register: build
- name: Extract artifact URL
set_fact:
archive_url: "{{ (build.json[0].artifacts | selectattr('name', 'equalto', download_artifact_name) | list)[0].url }}"
- name: Process build - name: Process build
loop: "{{ build.json }}" loop: "{{ build.json }}"
loop_control: loop_control: