openstack/project-config
Code Issues Proposed changes

Merge "Just use chmod instead of file for log permissions"

Browse Source
This commit is contained in:
Zuul
2018-06-18 13:51:56 +00:00
committed by Gerrit Code Review
parent 69dd523365 d45f691599
commit a74aef67c2
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
roles/test-upload-logs/tasks/main.yaml
View File

@@ -14,14 +14,14 @@
recurse: yes recurse: yes
mode: 0775 mode: 0775
# 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 - name: Ensure logs are readable before uploading
delegate_to: localhost delegate_to: localhost
file: command: "chmod -R u=rwX,g=rX,o=rX {{ zuul.executor.log_root }}/"
path: "{{ zuul.executor.log_root }}/" # ANSIBLE0007 chmod used in place of argument mode to file
mode: u=rwX,g=rX,o=rX tags:
state: directory - skip_ansible_lint
follow: false
recurse: yes
- name: Upload logs to log server - name: Upload logs to log server
synchronize: synchronize: