fix: using skip install on a subset of a group of hosts

roles/alertmanager/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: "Get checksum list"
+  ansible.builtin.set_fact:
+    __alertmanager_checksums: "{{ lookup('url', alertmanager_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __alertmanager_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __alertmanager_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __alertmanager_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/alertmanager/tasks/preflight.yml

@@ -57,21 +57,8 @@
   when:
     - alertmanager_binary_local_dir | length == 0
     - not alertmanager_skip_install
-  block:
-    - name: "Get checksum list"
-      ansible.builtin.set_fact:
-        __alertmanager_checksums: "{{ lookup('url', alertmanager_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __alertmanager_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __alertmanager_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __alertmanager_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
-
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml
 
 - name: Fail when extra config flags are duplicating ansible variables
   ansible.builtin.fail:

roles/blackbox_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __blackbox_exporter_checksums: "{{ lookup('url', blackbox_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __blackbox_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __blackbox_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __blackbox_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/blackbox_exporter/tasks/preflight.yml

@@ -57,17 +57,5 @@
   when:
     - blackbox_exporter_binary_local_dir | length == 0
     - not blackbox_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __blackbox_exporter_checksums: "{{ lookup('url', blackbox_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __blackbox_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __blackbox_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __blackbox_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/chrony_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __chrony_exporter_checksums: "{{ lookup('url', chrony_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __chrony_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __chrony_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __chrony_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/chrony_exporter/tasks/preflight.yml

@@ -105,17 +105,5 @@
   when:
     - chrony_exporter_binary_local_dir | length == 0
     - not chrony_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __chrony_exporter_checksums: "{{ lookup('url', chrony_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __chrony_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __chrony_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __chrony_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/mysqld_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __mysqld_exporter_checksums: "{{ lookup('url', mysqld_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __mysqld_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __mysqld_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __mysqld_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/mysqld_exporter/tasks/preflight.yml

@@ -105,17 +105,5 @@
   when:
     - mysqld_exporter_binary_local_dir | length == 0
     - not mysqld_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __mysqld_exporter_checksums: "{{ lookup('url', mysqld_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __mysqld_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __mysqld_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __mysqld_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/node_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __node_exporter_checksums: "{{ lookup('url', node_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __node_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __node_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __node_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/node_exporter/tasks/preflight.yml

@@ -105,17 +105,5 @@
   when:
     - node_exporter_binary_local_dir | length == 0
     - not node_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __node_exporter_checksums: "{{ lookup('url', node_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __node_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __node_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __node_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/prometheus/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: "Get checksum list"
+  ansible.builtin.set_fact:
+    __prometheus_checksums: "{{ lookup('url', prometheus_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __prometheus_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __prometheus_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __prometheus_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/prometheus/tasks/preflight.yml

@@ -105,17 +105,5 @@
   when:
     - prometheus_binary_local_dir | length == 0
     - not prometheus_skip_install
-  block:
-    - name: "Get checksum list"
-      ansible.builtin.set_fact:
-        __prometheus_checksums: "{{ lookup('url', prometheus_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __prometheus_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __prometheus_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __prometheus_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/pushgateway/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __pushgateway_checksums: "{{ lookup('url', pushgateway_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __pushgateway_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __pushgateway_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __pushgateway_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/pushgateway/tasks/preflight.yml

@@ -99,17 +99,5 @@
   when:
     - pushgateway_binary_local_dir | length == 0
     - not pushgateway_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __pushgateway_checksums: "{{ lookup('url', pushgateway_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __pushgateway_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __pushgateway_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __pushgateway_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/smartctl_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __smartctl_exporter_checksums: "{{ lookup('url', smartctl_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __smartctl_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __smartctl_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __smartctl_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/smartctl_exporter/tasks/preflight.yml

@@ -94,17 +94,5 @@
   when:
     - smartctl_exporter_binary_local_dir | length == 0
     - not smartctl_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __smartctl_exporter_checksums: "{{ lookup('url', smartctl_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __smartctl_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __smartctl_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __smartctl_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/smokeping_prober/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __smokeping_prober_checksums: "{{ lookup('url', smokeping_prober_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __smokeping_prober_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __smokeping_prober_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __smokeping_prober_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/smokeping_prober/tasks/preflight.yml

@@ -99,17 +99,5 @@
   when:
     - smokeping_prober_binary_local_dir | length == 0
     - not smokeping_prober_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __smokeping_prober_checksums: "{{ lookup('url', smokeping_prober_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __smokeping_prober_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __smokeping_prober_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __smokeping_prober_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/snmp_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    __snmp_exporter_checksums: "{{ lookup('url', snmp_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: __snmp_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    __snmp_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ __snmp_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/snmp_exporter/tasks/preflight.yml

@@ -37,17 +37,5 @@
   when:
     - snmp_exporter_binary_local_dir | length == 0
     - not snmp_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        __snmp_exporter_checksums: "{{ lookup('url', snmp_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: __snmp_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        __snmp_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ __snmp_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml

roles/systemd_exporter/tasks/get_checksum.yml

@@ -0,0 +1,14 @@
+---
+- name: Get checksum list from github
+  ansible.builtin.set_fact:
+    _systemd_exporter_checksums: "{{ lookup('url', systemd_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
+  run_once: true
+  until: _systemd_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
+  retries: 10
+
+- name: "Get checksum for {{ go_arch }}"
+  ansible.builtin.set_fact:
+    _systemd_exporter_checksum: "{{ item.split(' ')[0] }}"
+  with_items: "{{ _systemd_exporter_checksums }}"
+  when:
+    - "('linux-' + go_arch + '.tar.gz') in item"

roles/systemd_exporter/tasks/preflight.yml

@@ -112,17 +112,5 @@
   when:
     - systemd_exporter_binary_local_dir | length == 0
     - not systemd_exporter_skip_install
-  block:
-    - name: Get checksum list from github
-      ansible.builtin.set_fact:
-        _systemd_exporter_checksums: "{{ lookup('url', systemd_exporter_checksums_url, headers=_github_api_headers, wantlist=True) | list }}"
-      run_once: true
-      until: _systemd_exporter_checksums is search('linux-' + go_arch + '.tar.gz')
-      retries: 10
-
-    - name: "Get checksum for {{ go_arch }}"
-      ansible.builtin.set_fact:
-        _systemd_exporter_checksum: "{{ item.split(' ')[0] }}"
-      with_items: "{{ _systemd_exporter_checksums }}"
-      when:
-        - "('linux-' + go_arch + '.tar.gz') in item"
+  ansible.builtin.include_tasks:
+    file: get_checksum.yml