windmill
/
windmill
Code Issues Proposed changes

Use recursive: false for log collection 

Don't recursive copy subdirectories when collecting logs, as we don't
know what is inside them. This means we need to be more speciic of
which directories we want to copy.

Change-Id: I50691a2536d1f4127eddd133f6d99157085ed76c
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2018-03-28 12:28:03 -04:00
parent 549cb217b7
commit ca7de58905
No known key found for this signature in database
GPG Key ID: 611A80832067AF38
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tests/collect-logs.yaml
View File

@ -41,6 +41,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -75,6 +76,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -82,8 +84,8 @@
with_items:
- /etc/nodepool/builder-logging.conf
- /etc/nodepool/nodepool.yaml
- /var/log/nodepool/builds/*.log
- /var/log/nodepool/builder-debug.log
- /var/log/nodepool/builds
- /var/log/nodepool/nodepool-builder.log
- hosts: nl01
@ -112,6 +114,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -148,6 +151,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -185,6 +189,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname}}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -222,6 +227,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -259,6 +265,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"
@ -296,6 +303,7 @@
synchronize:
dest: "{{ zuul.executor.log_root }}/logs/{{ inventory_hostname }}"
mode: pull
recursive: false
rsync_opts:
- "--relative"
src: "{{ item }}"