Fix var-naming lint: use systemd_exporter_ prefix for role variables

This commit is contained in:
Rasmus Wejlgaard 2026-04-03 11:22:29 +00:00
parent 62c179da60
commit f00e0fcd3c

View file

@ -20,36 +20,41 @@
- name: Check if systemd_exporter binary exists - name: Check if systemd_exporter binary exists
ansible.builtin.stat: ansible.builtin.stat:
path: /usr/local/bin/systemd_exporter path: /usr/local/bin/systemd_exporter
register: _systemd_exporter_bin register: systemd_exporter_bin
- name: Get installed version - name: Get installed version
ansible.builtin.command: /usr/local/bin/systemd_exporter --version ansible.builtin.command: /usr/local/bin/systemd_exporter --version
register: _systemd_exporter_installed_version register: systemd_exporter_installed_version
changed_when: false changed_when: false
failed_when: false failed_when: false
when: _systemd_exporter_bin.stat.exists when: systemd_exporter_bin.stat.exists
- name: Set architecture and release facts - name: Set architecture fact
ansible.builtin.set_fact: ansible.builtin.set_fact:
_sde_arch: >- systemd_exporter_arch: >-
{{ ansible_architecture {{ ansible_architecture
| regex_replace('x86_64', 'amd64') | regex_replace('x86_64', 'amd64')
| regex_replace('aarch64', 'arm64') }} | regex_replace('aarch64', 'arm64') }}
- name: Set release name - name: Set release and URL facts
ansible.builtin.set_fact: ansible.builtin.set_fact:
_sde_release: "systemd_exporter-{{ systemd_exporter_version }}.linux-{{ _sde_arch }}" systemd_exporter_release: >-
_sde_base_url: "https://github.com/prometheus-community/systemd_exporter" systemd_exporter-{{ systemd_exporter_version }}.linux-{{ systemd_exporter_arch }}
systemd_exporter_base_url: >-
https://github.com/prometheus-community/systemd_exporter
- name: Download and install systemd_exporter - name: Download and install systemd_exporter
when: >- when: >-
not _systemd_exporter_bin.stat.exists or not systemd_exporter_bin.stat.exists or
systemd_exporter_version not in systemd_exporter_version not in
(_systemd_exporter_installed_version.stdout | default('')) (systemd_exporter_installed_version.stdout | default(''))
block: block:
- name: Download systemd_exporter tarball - name: Download systemd_exporter tarball
ansible.builtin.get_url: ansible.builtin.get_url:
url: "{{ _sde_base_url }}/releases/download/v{{ systemd_exporter_version }}/{{ _sde_release }}.tar.gz" url: >-
{{ systemd_exporter_base_url }}/releases/download/v{{
systemd_exporter_version }}/{{
systemd_exporter_release }}.tar.gz
dest: /tmp/systemd_exporter.tar.gz dest: /tmp/systemd_exporter.tar.gz
mode: '0644' mode: '0644'
@ -61,7 +66,7 @@
- name: Install systemd_exporter binary - name: Install systemd_exporter binary
ansible.builtin.copy: ansible.builtin.copy:
src: "/tmp/{{ _sde_release }}/systemd_exporter" src: "/tmp/{{ systemd_exporter_release }}/systemd_exporter"
dest: /usr/local/bin/systemd_exporter dest: /usr/local/bin/systemd_exporter
mode: '0755' mode: '0755'
owner: root owner: root
@ -75,7 +80,7 @@
state: absent state: absent
loop: loop:
- /tmp/systemd_exporter.tar.gz - /tmp/systemd_exporter.tar.gz
- "/tmp/{{ _sde_release }}" - "/tmp/{{ systemd_exporter_release }}"
- name: Deploy systemd_exporter service file - name: Deploy systemd_exporter service file
ansible.builtin.template: ansible.builtin.template: