CHANGELOG.md

@@ -1,17 +1,3 @@
-## v1.2.0
-
-### Fix
-* Replace flatten jinja filter with **list**.
-* Call module with FQCN.
-
-### Enhancements
-* Use comment jinja filter with ansible_managed to allow multi-lines comment.
-
-## v1.1.2
-
-### Enhancements
-* Use to_nice_json to manage packages list.
-
 ## v1.1.1
 
 ### Fix

README.md

@@ -82,7 +82,7 @@ Jérémy Gardais
 * [IPR][ipr website] (Institut de Physique de Rennes)
 
 [gogs to github hook]: https://stackoverflow.com/a/21998477
-[xymon_server source]: https://git.ipr.univ-rennes.fr/cellinfo/ansible.xymon_server
+[xymon_server source]: https://git.ipr.univ-rennes1.fr/cellinfo/ansible.xymon_server
 [xymon_server github]: https://github.com/ipr-cnrs/xymon_server
 [wtfpl website]: http://www.wtfpl.net/about/
 [ipr website]: https://ipr.univ-rennes1.fr/

defaults/main.yml

@@ -11,7 +11,7 @@
 #
 # List of base packages to install.
 xymon_server__base_packages:
-  - xymon
+  - 'xymon'
 
                                                                    # ]]]
 # .. envvar:: xymon_server__deploy_state [[[

meta/main.yml

@@ -7,7 +7,7 @@ galaxy_info:
   description: "Manage Xymon server configuration"
   license: WTFPL
   company: IPR
-  issue_tracker_url: https://git.ipr.univ-rennes.fr/cellinfo/ansible.xymon_server/issues
+  issue_tracker_url: https://git.ipr.univ-rennes1.fr/cellinfo/ansible.xymon_server/issues
   min_ansible_version: 2.4
   platforms:
   - name: Debian

tasks/main.yml

@@ -5,17 +5,19 @@
 
 # Server Manage required system packages [[[1
 - name: Ensure required packages are in there desired state
-  ansible.builtin.package:
-    name: '{{ xymon_server__base_packages | list }}'
+  package:
+    name: '{{ item }}'
     state: '{{ "present" if (xymon_server__deploy_state == "present") else "absent" }}'
     install_recommends: False
+  with_flattened:
+    - '{{ xymon_server__base_packages }}'
   register: server_pkg_result
   until: server_pkg_result is success
 
 # Server Manage /etc configuration files [[[1
 
 - name: Ensure /etc directory structure exists
-  ansible.builtin.file:
+  file:
     path: '/etc/xymon/{{ item.path }}'
     state: directory
   with_filetree: '{{ xymon_server__etc_src }}'
@@ -23,7 +25,7 @@
   notify: ['restart xymon server service']
 
 - name: Generate Xymon configuration files
-  ansible.builtin.template:
+  template:
     src: '{{ item.src }}'
     dest: "/etc/xymon/{{ item.path | replace('.j2','') }}"
     mode: '{{ item.mode }}'
@@ -32,7 +34,7 @@
   notify: ['restart xymon server service']
 
 - name: Generate Xymon configuration symlinks
-  ansible.builtin.template:
+  template:
     src: '{{ item.src }}'
     dest: "/etc/xymon/{{ item.path }}"
     mode: '{{ item.mode }}'
@@ -45,7 +47,7 @@
 # Manage service [[[1
 
 - name: Manage Xymon server service
-  ansible.builtin.service:
+  service:
     name: '{{ xymon_server__service_name }}'
     state: '{{ "started" if (xymon_server__deploy_state == "present" and
                             (xymon_server__service_manage | d(True) | bool))

templates/etc/xymon/hosts.cfg.j2

@@ -1,4 +1,4 @@
-# {{ ansible_managed | comment }}
+# {{ ansible_managed }}
 #
 # Master configuration file for Xymon
 #
@@ -20,6 +20,6 @@
 
 #page common Common
 
-#group-except sslcert|bbd|xymond|xymongen|xymonnet|xymonproxy <font size="+1">Test</font>
-#include hosts.d/common.cfg
+#group-except sslcert|bbd|xymond|xymongen|xymonnet|xymonproxy <font size="+1">Test</font>⏎
+#include hosts.d/common.cfg⏎
 

templates/etc/xymon/hosts.d/common.cfg.j2

@@ -1,4 +1,4 @@
-# {{ ansible_managed | comment }}
+# {{ ansible_managed }}
 
 #192.168.0.1      www.{{ ansible_domain }}                  # apt libs SSH apache https://www.{{ ansible_domain }}
 #192.168.0.2      pxe.{{ ansible_domain }}                  # apt libs SSH DESCR:PXE CLIENT:deploy