From aff442e89612af8bb735c9bbc244accdfdc01186 Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Mon, 10 Jun 2019 09:10:01 -0700 Subject: [PATCH] Remove extra task from download-artifacts This task is no longer necessary and should have been removed in 6523b6a5306b25ce39c33c6069c6125fae8e65c8 as part of the work to download multiple artifacts. Also fix a syntax error in a template string. Change-Id: I6a1d067a22a55843ec3320204bb673eaa0bcab4d --- roles/download-artifact/tasks/inner.yaml | 2 +- roles/download-artifact/tasks/main.yaml | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/roles/download-artifact/tasks/inner.yaml b/roles/download-artifact/tasks/inner.yaml index 3db238850..a42cf3868 100644 --- a/roles/download-artifact/tasks/inner.yaml +++ b/roles/download-artifact/tasks/inner.yaml @@ -2,7 +2,7 @@ uri: url: "{{ artifact.url }}" dest: "{{ download_artifact_directory }}" - loop: "{{ (build.artifacts }}" + loop: "{{ build.artifacts }}" loop_control: loop_var: artifact when: "artifact.name == download_artifact_name or ((download_artifact_name | type_debug) == 'list' and artifact.name in download_artifact_name)" diff --git a/roles/download-artifact/tasks/main.yaml b/roles/download-artifact/tasks/main.yaml index 5daf4af29..6f27e4dd8 100644 --- a/roles/download-artifact/tasks/main.yaml +++ b/roles/download-artifact/tasks/main.yaml @@ -2,9 +2,6 @@ uri: url: "{{ download_artifact_api }}/builds?{{ download_artifact_query }}" 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 loop: "{{ build.json }}" loop_control: