8c6512107c
Newer ansbile-lint finds "when" or "become" statements that are at the end of blocks. Ordering these before the block seems like a very logical thing to do, as we read from top-to-bottom so it's good to see if the block will execute or not. This is a no-op, and just moves the places the newer linter found. Change-Id: If4d1dc4343ea2575c64510e1829c3fe02d6c273f
46 lines
1.5 KiB
YAML
46 lines
1.5 KiB
YAML
- name: Set zuul-log-path fact
|
|
include_role:
|
|
name: set-zuul-log-path-fact
|
|
when: zuul_log_path is not defined
|
|
|
|
# Always upload (true), never upload (false) or only on failure ('failure')
|
|
- name: Upload logs
|
|
when: zuul_site_upload_logs | default(true) | bool or
|
|
(zuul_site_upload_logs == 'failure' and not zuul_success | bool)
|
|
block:
|
|
# Use chmod instead of file because ansible 2.5 file with recurse and
|
|
# follow can't really handle symlinks to .
|
|
- name: Ensure logs are readable before uploading
|
|
delegate_to: localhost
|
|
command: "chmod -R u=rwX,g=rX,o=rX {{ zuul.executor.log_root }}/"
|
|
# ANSIBLE0007 chmod used in place of argument mode to file
|
|
tags:
|
|
- skip_ansible_lint
|
|
|
|
- name: Upload logs to swift
|
|
delegate_to: localhost
|
|
zuul_swift_upload:
|
|
cloud: "{{ zuul_log_cloud_config }}"
|
|
partition: "{{ zuul_log_partition }}"
|
|
container: "{{ zuul_log_container }}"
|
|
public: "{{ zuul_log_container_public }}"
|
|
prefix: "{{ zuul_log_path }}"
|
|
indexes: "{{ zuul_log_create_indexes }}"
|
|
files:
|
|
- "{{ zuul.executor.log_root }}/"
|
|
delete_after: "{{ zuul_log_delete_after | default(omit) }}"
|
|
register: upload_results
|
|
|
|
- name: Return log URL
|
|
when: upload_results is defined
|
|
block:
|
|
- name: Return log URL to Zuul
|
|
delegate_to: localhost
|
|
zuul_return:
|
|
data:
|
|
zuul:
|
|
log_url: "{{ upload_results.url }}/"
|
|
- name: Print upload failures
|
|
debug:
|
|
var: upload_results.upload_failures
|