zuul/zuul-jobs
Code Issues Proposed changes

upload-artifactory: fix variable misspelling

Browse Source 
Also adds a check that makes sure the instance the artifact
is to be uploaded to is actually defined.

Change-Id: Ie80fa9869e49566dc39e815c10146d45724f5744
This commit is contained in:
Albin Vass 2020-05-28 15:37:22 +02:00
parent 8d90e3d4cb
commit 2477bb6d4d
2 changed files with 15 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
roles/upload-artifactory/tasks/upload.yaml
View File

@ -1,3 +1,8 @@
- name: Fail if artifactory instance is not defined
fail:
msg: "Instance {{ zj_artifact.instance }} is not defined."
when: zj_artifact.instance not in upload_artifactory_instances
- name: Make sure artifact exists on the executor
stat:
path: "{{ _undocumented_test_work_dir_ | default(zuul.executor.work_root) }}\
@ -25,36 +30,36 @@
request_header: "{{ request_header | combine(zj_artifact.headers|default({})) }}"
- name: Add api key to header
when: "'api_key' in upload_artifactory_instance[zj_artifact.instance]"
when: "'api_key' in upload_artifactory_instances[zj_artifact.instance]"
no_log: true
set_fact:
request_header: "{{ request_header |
combine({'X-JFrog-Art-Api': upload_artifactory_instance[zj_artifact.instance].api_key}) }}"
combine({'X-JFrog-Art-Api': upload_artifactory_instances[zj_artifact.instance].api_key}) }}"
- name: Set artifactory password
no_log: true
set_fact:
_artifactory_password: "{{ upload_artifactory_instance[zj_artifact.instance].password }}"
_artifactory_password: "{{ upload_artifactory_instances[zj_artifact.instance].password }}"
when:
- "'api_key' not in upload_artifactory_instance[zj_artifact.instance]"
- "'password' in upload_artifactory_instance[zj_artifact.instance]"
- "'api_key' not in upload_artifactory_instances[zj_artifact.instance]"
- "'password' in upload_artifactory_instances[zj_artifact.instance]"
- name: Set artifactory url fact
set_fact:
_artifactory_url: "{{ upload_artifactory_instance[zj_artifact.instance].transport | default('https') }}://\
{{ upload_artifactory_instance[zj_artifact.instance].fqdn }}\
_artifactory_url: "{{ upload_artifactory_instances[zj_artifact.instance].transport | default('https') }}://\
{{ upload_artifactory_instances[zj_artifact.instance].fqdn }}\
/artifactory/{{ zj_artifact.dest }}"
- name: Upload artifact
uri:
user: "{{ upload_artifactory_instance[zj_artifact.instance].user }}"
user: "{{ upload_artifactory_instances[zj_artifact.instance].user }}"
password: "{{ _artifactory_password | default(omit) }}"
url: "{{ _artifactory_url }}"
src: "{{ _undocumented_test_work_dir_ | default(zuul.executor.work_root) }}/artifacts/{{ zj_artifact.src }}"
headers: "{{ request_header }}"
status_code: 201
method: PUT
force_basic_auth: "{{ upload_artifactory_instance[zj_artifact.instance].force_basic_auth |
force_basic_auth: "{{ upload_artifactory_instances[zj_artifact.instance].force_basic_auth |
default(false) | bool }}"
remote_src: yes # To not unecessarily copy artifact to a tempfile
delegate_to: "{{ _undocumented_test_worker_node_ | default('localhost') }}"

2
test-playbooks/artifactory/run.yaml
View File

@ -47,7 +47,7 @@
- name: Set artifactory instances fact
set_fact:
cacheable: true
upload_artifactory_instance:
upload_artifactory_instances:
localhost_password:
fqdn: localhost:8081
transport: http