Merge pull request 'refresh-2024' (#1) from refresh-2024 into master

Reviewed-on: #1

CHANGELOG.md

@@ -1,3 +1,11 @@
+## 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
 

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-rennes1.fr/cellinfo/ansible.xymon_server
+[xymon_server source]: https://git.ipr.univ-rennes.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-rennes1.fr/cellinfo/ansible.xymon_server/issues
+  issue_tracker_url: https://git.ipr.univ-rennes.fr/cellinfo/ansible.xymon_server/issues
   min_ansible_version: 2.4
   platforms:
   - name: Debian

tasks/main.yml

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