Add check to deploy config if log exists

Role will now test if a log exist (and hence the application exists
    on the host before writing a logrotate config to the server.

README.md

@@ -1,2 +1,9 @@
 # ansible-logrotate-plusplus
 Ansible role which installs and configures logrotate
+It can test if paths exist before writing a logrotate config to the server.
+Load the roles default vars with custom paths and per path parameters, and run the playbook across a dynamic
+infrastructure and only write logerotate rules to the appropriate system with the correct paths present.
+
+This project was based of https://github.com/arillso/ansible.logrotate 1.5.2
+(https://github.com/arillso/ansible.logrotate/commit/038649f7933c21ba9f1f2c8363bfb4d49aaf46f2)
+

defaults/main.yml

@@ -36,8 +36,7 @@ logrotate_btmp:
 # An example of multiple log rotate applications with available settings:
 # logrotate_applications:
 #   - name: name-your-log-rotate-application
-#     definitions:
+#     logs:
-#       - logs:
 #       - /var/log/apt/term.log
 #       - /var/log/apt/history.log
 #     options:
@@ -46,4 +45,41 @@ logrotate_btmp:
 #       - missingok
 #       - notifempty
 #       - compress
-logrotate_applications: []
+logrotate_applications:
+  - name: nextcloud-snap-apache-php_errors
+    logs:
+      - /var/snap/nextcloud/current/apache/logs/*.log
+    options:
+      - daily
+      - compress
+      - rotate 6
+      - missingok
+      - copytruncate
+      - delaycompress
+
+  - name: nextcloud-snap-apache-error_log"
+    logs:
+      - /var/snap/nextcloud/current/apache/logs/error_log
+    options:
+      - daily
+      - compress
+      - rotate 6
+      - missingok
+      - copytruncate
+      - delaycompress
+
+  - name: custom
+    logs:
+      -  /var/log/custom/*_log
+    options:
+      - daily
+      - compress
+      - rotate 6
+
+  - name : dpkg
+    logs:
+      - /var/log/dpkg.log
+    options:
+      - daily
+      - compress
+      - rotate 6

tasks/create-logrotate-application-configuration-files.yml

@@ -0,0 +1,22 @@
+---
+- name: 'check if there exist log files for {{ item.name }}'
+  shell:
+    cmd: "ls -l {{ item.logs|join(' ') }}"
+  changed_when: false
+  register: _available_logs
+  check_mode: false
+  failed_when: false
+  tags:
+    - configuration
+
+- name: 'create logrotate configuration file for {{ item.name }}'
+  become: true
+  template:
+    src: 'etc/logrotate.d/application.j2'
+    dest: '/etc/logrotate.d/{{ item.name }}'
+    owner: root
+    group: root
+    mode: 0644
+  when: _available_logs.stdout_lines|length() > 0
+  tags:
+    - configuration

tasks/main.yml

@@ -44,15 +44,10 @@
     - configuration
 
 - name: 'create logrotate application configuration files'
-  become: true
+  include_tasks: create-logrotate-application-configuration-files.yml
-  template:
+  loop: '{{ logrotate_applications }}'
-    src: 'etc/logrotate.d/application.j2'
+  loop_control:
-    dest: '/etc/logrotate.d/{{ item.name }}'
+    label: "{{ item.name }}"
-    owner: root
-    group: root
-    mode: 0644
-  with_items:
-    - '{{ logrotate_applications }}'
   tags:
     - configuration
 

templates/etc/logrotate.d/application.j2

@@ -1,37 +1,35 @@
 {{ ansible_managed | comment }}
 
-{% for definition in item.definitions %}
+{{ item.logs | join(" ") }} {
-{{ definition.logs | join(" ") }} {
+{% for option in item.options %}
-{% for option in definition.options %}
     {{ option }}
 {% endfor %}
-{% if definition.postrotate|default([]) %}
+{% if item.postrotate|default([]) %}
     postrotate
-{% for line in definition.postrotate %}
+{% for line in item.postrotate %}
         {{ line }}
 {% endfor %}
     endscript
 {% endif %}
-{% if definition.preremove|default([]) %}
+{% if item.preremove|default([]) %}
     preremove
-{% for line in definition.preremove %}
+{% for line in item.preremove %}
         {{ line }}
 {% endfor %}
     endscript
 {% endif %}
-{% if definition.lastaction|default([]) %}
+{% if item.lastaction|default([]) %}
     lastaction
-{% for line in definition.lastaction %}
+{% for line in item.lastaction %}
         {{ line }}
 {% endfor %}
     endscript
 {% endif %}
-{% if definition.firstaction|default([]) %}
+{% if item.firstaction|default([]) %}
     firstaction
-{% for line in definition.firstaction %}
+{% for line in item.firstaction %}
         {{ line }}
 {% endfor %}
     endscript
 {% endif %}
 }
-{% endfor %}