Merge pull request #443 from zhan9san/feature/move-naive-assertion-to-common-role

refactor: move naive assertion to common role

roles/_common/meta/argument_specs.yml

@@ -81,6 +81,9 @@ argument_specs:
                   {{ ('python-apt' if ansible_python_version is version('3', '<') else 'python3-apt') }}
                   {% else %}\
                   {% endif %}"
+      _common_web_listen_address:
+        description: "Address on which to listen"
+        default: ""
   selinux:
     short_description: "Internal only - common selinux configuration tasks"
     description: "Internal only - selinux requirements"

roles/_common/tasks/preflight.yml

@@ -74,3 +74,14 @@
     - "{{ ansible_parent_role_names | first | regex_replace(ansible_collection_name ~ '.', '') }}_configure"
     - install
     - "{{ ansible_parent_role_names | first | regex_replace(ansible_collection_name ~ '.', '') }}_install"
+
+- name: Naive assertion of proper listen address
+  ansible.builtin.assert:
+    that:
+      - >-
+        [_common_web_listen_address] |
+          flatten |
+          reject('match', '.+:\\d+$') |
+          list |
+          length == 0
+  when: (_common_web_listen_address)

roles/_common/vars/main.yml

@@ -18,6 +18,7 @@ _common_binary_unarchive_opts: ""
 _common_tls_server_config: {}
 _common_http_server_config: {}
 _common_basic_auth_users: {}
+_common_web_listen_address: ""
 # Variables that should not be overwritten
 __common_binary_basename: "{{ _common_binary_url | urlsplit('path') | basename }}"
 __common_github_api_headers: "{{ {'GITHUB_TOKEN': lookup('ansible.builtin.env', 'GITHUB_TOKEN')} if (lookup('ansible.builtin.env', 'GITHUB_TOKEN')) else {} }}"

roles/alertmanager/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ alertmanager_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           alertmanager_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [alertmanager_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Discover latest version
   ansible.builtin.set_fact:
     alertmanager_version: "{{ (lookup('url', 'https://api.github.com/repos/{{ _alertmanager_repo }}/releases/latest', headers=_github_api_headers,

roles/bind_exporter/tasks/preflight.yml

@@ -3,16 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
-
+  vars:
-- name: Naive assertion of proper listen address
+    _common_web_listen_address: "{{ bind_exporter_web_listen_address }}"
-  ansible.builtin.assert:
-    that:
-      - >-
-        [bind_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
 
 - name: Assert that TLS config is correct
   when: bind_exporter_tls_server_config | length > 0

roles/blackbox_exporter/tasks/preflight.yml

@@ -5,6 +5,7 @@
     tasks_from: preflight.yml
   vars:
     _common_dependencies: "{{ _blackbox_exporter_dependencies }}"
+    _common_web_listen_address: "{{ blackbox_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -17,16 +18,6 @@
           blackbox_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [blackbox_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Discover latest version
   ansible.builtin.set_fact:
     blackbox_exporter_version: "{{ (lookup('url', 'https://api.github.com/repos/{{ _blackbox_exporter_repo }}/releases/latest', headers=_github_api_headers,

roles/chrony_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ chrony_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           chrony_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [chrony_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert collectors are not both disabled and enabled at the same time
   ansible.builtin.assert:
     that:

roles/fail2ban_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ fail2ban_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         fail2ban_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [fail2ban_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Discover latest version
   ansible.builtin.set_fact:
     fail2ban_exporter_version: "{{ (lookup('url', 'https://gitlab.com/api/v4/projects/{{ _fail2ban_exporter_repo }}/releases',

roles/influxdb_exporter/tasks/preflight.yml

@@ -3,6 +3,9 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ influxdb_exporter_web_listen_address }}"
+
   tags:
     - influxdb_exporter
     - preflight
@@ -17,20 +20,6 @@
     - preflight
     - influxdb_exporter_preflight
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [influxdb_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-  tags:
-    - influxdb_exporter
-    - preflight
-    - influxdb_exporter_preflight
-
 - name: Discover latest version
   ansible.builtin.set_fact:
     influxdb_exporter_version: "{{ (lookup('url', 'https://api.github.com/repos/{{ _influxdb_exporter_repo }}/releases/latest', headers=_influxdb_exporter_github_api_headers,

roles/ipmi_exporter/tasks/preflight.yml

@@ -5,6 +5,7 @@
     tasks_from: preflight.yml
   vars:
     _common_dependencies: "{{ _ipmi_exporter_dependencies }}"
+    _common_web_listen_address: "{{ ipmi_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -12,16 +13,6 @@
       - >-
         ipmi_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [ipmi_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: ipmi_exporter_tls_server_config | length > 0
   block:

roles/memcached_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ memcached_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         memcached_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [memcached_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: memcached_exporter_tls_server_config | length > 0
   block:

roles/mongodb_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ mongodb_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         mongodb_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [mongodb_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: mongodb_exporter_tls_server_config | length > 0
   block:

roles/mysqld_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ mysqld_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           mysqld_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [mysqld_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert collectors are not both disabled and enabled at the same time
   ansible.builtin.assert:
     that:

roles/nginx_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ nginx_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         nginx_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [nginx_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: nginx_exporter_tls_server_config | length > 0
   block:

roles/node_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ node_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           node_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [node_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert collectors are not both disabled and enabled at the same time
   ansible.builtin.assert:
     that:

roles/postgres_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ postgres_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           postgres_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [postgres_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert collectors are not both disabled and enabled at the same time
   ansible.builtin.assert:
     that:

roles/process_exporter/tasks/preflight.yml

@@ -3,11 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
-
+  vars:
-- name: Naive assertion of proper listen address
+    _common_web_listen_address: "{{ process_exporter_web_listen_address }}"
-  ansible.builtin.assert:
-    that:
-      - "':' in process_exporter_web_listen_address"
 
 - name: Discover latest version
   ansible.builtin.set_fact:

roles/prometheus/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ prometheus_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         prometheus_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [prometheus_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert no duplicate config flags
   ansible.builtin.assert:
     that:

roles/pushgateway/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ pushgateway_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           pushgateway_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [pushgateway_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: pushgateway_tls_server_config | length > 0
   block:

roles/redis_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ redis_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         redis_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [redis_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when:
     - redis_exporter_tls_server_key_file | length > 0

roles/smartctl_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ smartctl_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         smartctl_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [smartctl_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: smartctl_exporter_tls_server_config | length > 0
   block:

roles/smokeping_prober/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ smokeping_prober_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           smokeping_prober_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [smokeping_prober_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: smokeping_prober_tls_server_config | length > 0
   block:

roles/snmp_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ snmp_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -15,16 +17,6 @@
           snmp_exporter_web_listen_address | type_debug == "list"
         )
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [snmp_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Discover latest version
   ansible.builtin.set_fact:
     snmp_exporter_version: "{{ (lookup('url', 'https://api.github.com/repos/{{ _snmp_exporter_repo }}/releases/latest', headers=_github_api_headers,

roles/systemd_exporter/tasks/preflight.yml

@@ -3,6 +3,8 @@
   ansible.builtin.include_role:
     name: prometheus.prometheus._common
     tasks_from: preflight.yml
+  vars:
+    _common_web_listen_address: "{{ systemd_exporter_web_listen_address }}"
 
 - name: Assert that used version supports listen address type
   ansible.builtin.assert:
@@ -10,16 +12,6 @@
       - >-
         systemd_exporter_web_listen_address is string
 
-- name: Naive assertion of proper listen address
-  ansible.builtin.assert:
-    that:
-      - >-
-        [systemd_exporter_web_listen_address] |
-          flatten |
-          reject('match', '.+:\\d+$') |
-          list |
-          length == 0
-
 - name: Assert that TLS config is correct
   when: systemd_exporter_tls_server_config | length > 0
   block: