From f02898ff9c3a80bef0e182106d083dbf60d6301a Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:03:47 +0400 Subject: [PATCH 1/9] Fix variable names that escaped renaming --- roles/ipmi_exporter/molecule/default/converge.yml | 2 +- roles/ipmi_exporter/tasks/configure.yml | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/roles/ipmi_exporter/molecule/default/converge.yml b/roles/ipmi_exporter/molecule/default/converge.yml index 40122c5..87f6d9a 100644 --- a/roles/ipmi_exporter/molecule/default/converge.yml +++ b/roles/ipmi_exporter/molecule/default/converge.yml @@ -3,4 +3,4 @@ hosts: all roles: - role: genlab.common.ipmi_exporter - ipmi_exp_source_dir: "molecule/default/" + ipmi_exporter_source_dir: "molecule/default/" diff --git a/roles/ipmi_exporter/tasks/configure.yml b/roles/ipmi_exporter/tasks/configure.yml index 04af8f5..d985808 100644 --- a/roles/ipmi_exporter/tasks/configure.yml +++ b/roles/ipmi_exporter/tasks/configure.yml @@ -1,9 +1,9 @@ --- - name: "Upload ipmi_exporter local configure file" notify: "(Re)start and enable ipmi_exporter" - when: ipmi_exp_source_dir is defined and ipmi_exp_source_dir | length > 0 + when: ipmi_exporter_source_dir is defined and ipmi_exporter_source_dir | length > 0 ansible.builtin.template: - src: "{{ ipmi_exp_source_dir }}/ipmi_local.conf" + src: "{{ ipmi_exporter_source_dir }}/ipmi_local.conf" dest: "{{ ipmi_exporter_config_dir }}/ipmi_local.conf" owner: root group: root @@ -11,9 +11,9 @@ - name: "Upload ipmi_exporter web configure file" notify: "(Re)start and enable ipmi_exporter" - when: ipmi_exp_web_source_dir is defined and ipmi_exp_web_source_dir | length > 0 + when: ipmi_exporter_web_source_dir is defined and ipmi_exporter_web_source_dir | length > 0 ansible.builtin.template: - src: "{{ ipmi_exp_web_source_dir }}/web_conf.yaml" + src: "{{ ipmi_exporter_web_source_dir }}/web_conf.yaml" dest: "{{ ipmi_exporter_config_dir }}/web_conf.yaml" owner: root group: root From bda231768bded2b723832f76823e6de36ebbe904 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:04:05 +0400 Subject: [PATCH 2/9] Fix variable names in `README.md` --- roles/ipmi_exporter/README.md | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/roles/ipmi_exporter/README.md b/roles/ipmi_exporter/README.md index b22a041..7e1a6f1 100644 --- a/roles/ipmi_exporter/README.md +++ b/roles/ipmi_exporter/README.md @@ -29,16 +29,16 @@ The following collectors are available and can be enabled or disabled in the con - sm-lan-mode: collects the "LAN mode" setting in the current BMC config. If it fails, the LAN mode metric (see below) will not be available ```yaml -ipmi_exp_version: 1.10.1 -ipmi_exp_dir: "/etc/exporters" -ipmi_exp_config_dir: "/etc/exporters/config" -ipmi_exp_args: "" # --[no-]native-ipmi Use native IPMI implementation instead of FreeIPMI (EXPERIMENTAL) +ipmi_exporter_version: 1.10.1 +ipmi_exporter_dir: "/etc/exporters" +ipmi_exporter_config_dir: "/etc/exporters/config" +ipmi_exporter_args: "" # --[no-]native-ipmi Use native IPMI implementation instead of FreeIPMI (EXPERIMENTAL) # --[no-]web.systemd-socket Use systemd socket activation listeners instead of port listeners (Linux only). -ipmi_exp_log_level: "info" # Only log messages with the given severity or above. One of: [debug, info, warn, error] -ipmi_exp_log_format: "logfmt" # Output format of log messages. One of: [logfmt, json] -ipmi_exp_web_listen_address: "localhost:9290" # Addresses on which to expose metrics and web interface. Repeatable for multiple addresses. Examples: `:9100` or `[::1]:9100` for http, vsock://:9100` for vsock -ipmi_exp_source_dir: ipmi_local.conf # Path to configuration file. See: https://github.com/prometheus-community/ipmi_exporter/blob/master/docs/configuration.md -ipmi_exp_web_source_dir: web_conf.yaml # Path to configuration file that can enable TLS or authentication. See: https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md +ipmi_exporter_log_level: "info" # Only log messages with the given severity or above. One of: [debug, info, warn, error] +ipmi_exporter_log_format: "logfmt" # Output format of log messages. One of: [logfmt, json] +ipmi_exporter_web_listen_address: "localhost:9290" # Addresses on which to expose metrics and web interface. Repeatable for multiple addresses. Examples: `:9100` or `[::1]:9100` for http, vsock://:9100` for vsock +ipmi_exporter_source_dir: ipmi_local.conf # Path to configuration file. See: https://github.com/prometheus-community/ipmi_exporter/blob/master/docs/configuration.md +ipmi_exporter_web_source_dir: web_conf.yaml # Path to configuration file that can enable TLS or authentication. See: https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md ``` Dependencies From a5642463cc2322767b182de3fb48b871f109a9a3 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:08:04 +0400 Subject: [PATCH 3/9] Restore source style consistency --- roles/ipmi_exporter/tasks/configure.yml | 4 ++-- roles/ipmi_exporter/tasks/install.yml | 22 +++++++++++----------- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/roles/ipmi_exporter/tasks/configure.yml b/roles/ipmi_exporter/tasks/configure.yml index d985808..3658656 100644 --- a/roles/ipmi_exporter/tasks/configure.yml +++ b/roles/ipmi_exporter/tasks/configure.yml @@ -1,5 +1,5 @@ --- -- name: "Upload ipmi_exporter local configure file" +- name: "Configure | Upload local configuration file" notify: "(Re)start and enable ipmi_exporter" when: ipmi_exporter_source_dir is defined and ipmi_exporter_source_dir | length > 0 ansible.builtin.template: @@ -9,7 +9,7 @@ group: root mode: '0640' -- name: "Upload ipmi_exporter web configure file" +- name: "Configure | Upload web configuration file" notify: "(Re)start and enable ipmi_exporter" when: ipmi_exporter_web_source_dir is defined and ipmi_exporter_web_source_dir | length > 0 ansible.builtin.template: diff --git a/roles/ipmi_exporter/tasks/install.yml b/roles/ipmi_exporter/tasks/install.yml index 4d9ec03..efdf45d 100644 --- a/roles/ipmi_exporter/tasks/install.yml +++ b/roles/ipmi_exporter/tasks/install.yml @@ -1,31 +1,31 @@ --- -- name: "Install ipmi_exporter from binary" +- name: "Install | Install ipmi_exporter binary" block: - - name: "Check ipmi_exporter version" + - name: "Install | Check ipmi_exporter availability" changed_when: false + register: ipmi_exporter_ver ansible.builtin.command: cmd: "ipmi_exporter --version" - register: ipmi_exporter_ver - - name: "Assert version correctness" + - name: "Install | Assert version correctness" ansible.builtin.assert: that: "ipmi_exporter_ver.stdout is regex('{{ ipmi_exporter_version }}')" success_msg: "ipmi_exporter version {{ ipmi_exporter_version }} is installed and working" fail_msg: "ipmi_exporter version {{ ipmi_exporter_version }} is not installed or not working correctly" rescue: - - name: "Create ipmi_exporter directories" + - name: "Install | Create directories" + loop: + - "{{ ipmi_exporter_dir }}" + - "{{ ipmi_exporter_config_dir }}" ansible.builtin.file: path: "{{ item }}" state: directory owner: root group: root mode: "0755" - with_items: - - "{{ ipmi_exporter_dir }}" - - "{{ ipmi_exporter_config_dir }}" - - name: "Unarchive ipmi_exporter tar file" + - name: "Install | Unarchive the tarball" notify: "(Re)start and enable ipmi_exporter" ansible.builtin.unarchive: src: "https://github.com/prometheus-community/ipmi_exporter/releases/\ @@ -33,7 +33,7 @@ dest: "{{ ipmi_exporter_dir }}" remote_src: true - - name: "Move ipmi_exporter binary" + - name: "Install | Rename and move the binary under PATH" ansible.builtin.copy: src: "{{ ipmi_exporter_dir }}/ipmi_exporter-{{ ipmi_exporter_version }}.linux-amd64/ipmi_exporter" dest: "/usr/local/bin/ipmi_exporter" @@ -42,7 +42,7 @@ group: root remote_src: true -- name: Create systemd service file +- name: "Install | Create a systemd service file" ansible.builtin.template: src: ipmi_exporter.service.j2 dest: /etc/systemd/system/ipmi_exporter.service From f85dd70eb9c369cab76d9b6bc88c90fc5de4c615 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:31:12 +0400 Subject: [PATCH 4/9] Simplify configuration file management --- roles/ipmi_exporter/defaults/main.yml | 2 +- roles/ipmi_exporter/molecule/default/converge.yml | 2 +- roles/ipmi_exporter/tasks/configure.yml | 12 ++++++------ roles/ipmi_exporter/tasks/install.yml | 1 - .../ipmi_exporter/templates/ipmi_exporter.service.j2 | 8 ++++---- 5 files changed, 12 insertions(+), 13 deletions(-) diff --git a/roles/ipmi_exporter/defaults/main.yml b/roles/ipmi_exporter/defaults/main.yml index c67b875..464ec49 100644 --- a/roles/ipmi_exporter/defaults/main.yml +++ b/roles/ipmi_exporter/defaults/main.yml @@ -1,7 +1,7 @@ --- ipmi_exporter_version: 1.10.1 ipmi_exporter_dir: "/etc/exporters" -ipmi_exporter_config_dir: "/etc/exporters/config" +ipmi_exporter_config_dir: "/etc/ipmi_exporter" ipmi_exporter_log_level: "info" ipmi_exporter_log_format: "logfmt" ipmi_exporter_web_listen_address: "localhost:9290" diff --git a/roles/ipmi_exporter/molecule/default/converge.yml b/roles/ipmi_exporter/molecule/default/converge.yml index 87f6d9a..a81a5b8 100644 --- a/roles/ipmi_exporter/molecule/default/converge.yml +++ b/roles/ipmi_exporter/molecule/default/converge.yml @@ -3,4 +3,4 @@ hosts: all roles: - role: genlab.common.ipmi_exporter - ipmi_exporter_source_dir: "molecule/default/" + ipmi_exporter_config_dir: "molecule/default/" diff --git a/roles/ipmi_exporter/tasks/configure.yml b/roles/ipmi_exporter/tasks/configure.yml index 3658656..44c2b3c 100644 --- a/roles/ipmi_exporter/tasks/configure.yml +++ b/roles/ipmi_exporter/tasks/configure.yml @@ -1,20 +1,20 @@ --- - name: "Configure | Upload local configuration file" notify: "(Re)start and enable ipmi_exporter" - when: ipmi_exporter_source_dir is defined and ipmi_exporter_source_dir | length > 0 + when: ipmi_exporter_local_conf is defined ansible.builtin.template: - src: "{{ ipmi_exporter_source_dir }}/ipmi_local.conf" - dest: "{{ ipmi_exporter_config_dir }}/ipmi_local.conf" + src: "{{ ipmi_exporter_local_conf }}" + dest: "{{ ipmi_exporter_config_dir }}/local.conf" owner: root group: root mode: '0640' - name: "Configure | Upload web configuration file" notify: "(Re)start and enable ipmi_exporter" - when: ipmi_exporter_web_source_dir is defined and ipmi_exporter_web_source_dir | length > 0 + when: ipmi_exporter_web_conf is defined ansible.builtin.template: - src: "{{ ipmi_exporter_web_source_dir }}/web_conf.yaml" - dest: "{{ ipmi_exporter_config_dir }}/web_conf.yaml" + src: "{{ ipmi_exporter_web_conf }}" + dest: "{{ ipmi_exporter_config_dir }}/web.yaml" owner: root group: root mode: '0640' diff --git a/roles/ipmi_exporter/tasks/install.yml b/roles/ipmi_exporter/tasks/install.yml index efdf45d..fcc315f 100644 --- a/roles/ipmi_exporter/tasks/install.yml +++ b/roles/ipmi_exporter/tasks/install.yml @@ -16,7 +16,6 @@ rescue: - name: "Install | Create directories" loop: - - "{{ ipmi_exporter_dir }}" - "{{ ipmi_exporter_config_dir }}" ansible.builtin.file: path: "{{ item }}" diff --git a/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 b/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 index 6b41d18..dc44bca 100644 --- a/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 +++ b/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 @@ -14,11 +14,11 @@ ExecStart=/usr/local/bin/ipmi_exporter \ {% if ipmi_exporter_args is defined and ipmi_exporter_args | length > 0 %} {{ ipmi_exporter_args }} \ {% endif %} -{% if ipmi_exporter_source_dir is defined and ipmi_exporter_source_dir | length > 0 %} - --config.file={{ ipmi_exporter_config_dir }}/ipmi_local.conf \ +{% if ipmi_exporter_local_conf is defined %} + --config.file={{ ipmi_exporter_config_dir }}/local.conf \ {% endif %} -{% if ipmi_exporter_web_source_dir is defined and ipmi_exporter_web_source_dir | length > 0 %} - --web.config.file={{ ipmi_exporter_config_dir }}/web_conf.yaml +{% if ipmi_exporter_web_conf is defined %} + --web.config.file={{ ipmi_exporter_config_dir }}/web.yaml {% endif %} SyslogIdentifier=ipmi_exporter From ddcbb980f677dc0927789ae2adec8b6f75d21ea6 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:33:21 +0400 Subject: [PATCH 5/9] Put the binary under a more universal path for consistency with RHEL distros --- roles/ipmi_exporter/defaults/main.yml | 1 - roles/ipmi_exporter/tasks/install.yml | 6 +++--- roles/ipmi_exporter/templates/ipmi_exporter.service.j2 | 2 +- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/roles/ipmi_exporter/defaults/main.yml b/roles/ipmi_exporter/defaults/main.yml index 464ec49..195b0c5 100644 --- a/roles/ipmi_exporter/defaults/main.yml +++ b/roles/ipmi_exporter/defaults/main.yml @@ -1,6 +1,5 @@ --- ipmi_exporter_version: 1.10.1 -ipmi_exporter_dir: "/etc/exporters" ipmi_exporter_config_dir: "/etc/ipmi_exporter" ipmi_exporter_log_level: "info" ipmi_exporter_log_format: "logfmt" diff --git a/roles/ipmi_exporter/tasks/install.yml b/roles/ipmi_exporter/tasks/install.yml index fcc315f..d841975 100644 --- a/roles/ipmi_exporter/tasks/install.yml +++ b/roles/ipmi_exporter/tasks/install.yml @@ -29,13 +29,13 @@ ansible.builtin.unarchive: src: "https://github.com/prometheus-community/ipmi_exporter/releases/\ download/v{{ ipmi_exporter_version }}/ipmi_exporter-{{ ipmi_exporter_version }}.linux-amd64.tar.gz" - dest: "{{ ipmi_exporter_dir }}" + dest: "/tmp/" remote_src: true - name: "Install | Rename and move the binary under PATH" ansible.builtin.copy: - src: "{{ ipmi_exporter_dir }}/ipmi_exporter-{{ ipmi_exporter_version }}.linux-amd64/ipmi_exporter" - dest: "/usr/local/bin/ipmi_exporter" + src: "/tmp/ipmi_exporter-{{ ipmi_exporter_version }}.linux-amd64/ipmi_exporter" + dest: "/usr/sbin/ipmi_exporter" mode: "0755" owner: root group: root diff --git a/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 b/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 index dc44bca..a07276e 100644 --- a/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 +++ b/roles/ipmi_exporter/templates/ipmi_exporter.service.j2 @@ -7,7 +7,7 @@ StartLimitBurst=5 [Service] Type=simple -ExecStart=/usr/local/bin/ipmi_exporter \ +ExecStart=/usr/sbin/ipmi_exporter \ --web.listen-address={{ ipmi_exporter_web_listen_address }} \ --log.level={{ ipmi_exporter_log_level }} \ --log.format={{ ipmi_exporter_log_format }} \ From 55ef7c1d9d2aa6aaa10fb9cda6fbaca2e4743e71 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:34:01 +0400 Subject: [PATCH 6/9] Configure to listen on `0.0.0.0` be default --- roles/ipmi_exporter/defaults/main.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/ipmi_exporter/defaults/main.yml b/roles/ipmi_exporter/defaults/main.yml index 195b0c5..3b803fa 100644 --- a/roles/ipmi_exporter/defaults/main.yml +++ b/roles/ipmi_exporter/defaults/main.yml @@ -3,5 +3,5 @@ ipmi_exporter_version: 1.10.1 ipmi_exporter_config_dir: "/etc/ipmi_exporter" ipmi_exporter_log_level: "info" ipmi_exporter_log_format: "logfmt" -ipmi_exporter_web_listen_address: "localhost:9290" +ipmi_exporter_web_listen_address: "0.0.0.0:9290" ipmi_exporter_args: "" From dded21469f0dd1708700598f53ec67d71bc35458 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:34:15 +0400 Subject: [PATCH 7/9] Update the `README.md` --- roles/ipmi_exporter/README.md | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/roles/ipmi_exporter/README.md b/roles/ipmi_exporter/README.md index 7e1a6f1..99774c7 100644 --- a/roles/ipmi_exporter/README.md +++ b/roles/ipmi_exporter/README.md @@ -14,7 +14,7 @@ Target node: Role Variables -------------- -Configuration files must have names `web_conf.yaml` and `ipmi_local.conf`. If ipmi_exp_source_dir is specified, the role searches for `web_conf.yaml` and `ipmi_local.conf` in that directory and copy to target host in `ipmi_exp_config_dir`. If the source directory is not specified, the role skips this step. In ipmi_local.conf user can describe what modules to use for metric collection. +Configuration files must have names `web_conf.yaml` and `ipmi_local.conf`. If `ipmi_exporter_local_conf` is specified, the role searches for `web_conf.yaml` and `ipmi_local.conf` in that directory and copy to target host in `ipmi_exporter_config_dir`. If the source directory is not specified, the role skips this step. In ipmi_local.conf user can describe what modules to use for metric collection. ipmi_up{collector=""} is 1 if the data for this collector could successfully be retrieved from the remote host, 0 otherwise. @@ -28,17 +28,18 @@ The following collectors are available and can be enabled or disabled in the con - sel-events: collects metrics for user-defined events in system event log (SEL). If it fails, SEL entries metrics (see below) will not be available - sm-lan-mode: collects the "LAN mode" setting in the current BMC config. If it fails, the LAN mode metric (see below) will not be available -```yaml -ipmi_exporter_version: 1.10.1 -ipmi_exporter_dir: "/etc/exporters" -ipmi_exporter_config_dir: "/etc/exporters/config" -ipmi_exporter_args: "" # --[no-]native-ipmi Use native IPMI implementation instead of FreeIPMI (EXPERIMENTAL) - # --[no-]web.systemd-socket Use systemd socket activation listeners instead of port listeners (Linux only). -ipmi_exporter_log_level: "info" # Only log messages with the given severity or above. One of: [debug, info, warn, error] -ipmi_exporter_log_format: "logfmt" # Output format of log messages. One of: [logfmt, json] -ipmi_exporter_web_listen_address: "localhost:9290" # Addresses on which to expose metrics and web interface. Repeatable for multiple addresses. Examples: `:9100` or `[::1]:9100` for http, vsock://:9100` for vsock -ipmi_exporter_source_dir: ipmi_local.conf # Path to configuration file. See: https://github.com/prometheus-community/ipmi_exporter/blob/master/docs/configuration.md -ipmi_exporter_web_source_dir: web_conf.yaml # Path to configuration file that can enable TLS or authentication. See: https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md +- `ipmi_exporter_version`: release version of the exporter +- `ipmi_exporter_config_dir`: where to put configuration files +- `ipmi_exporter_args`: + - `--[no-]native-ipmi`: Use native IPMI implementation instead of FreeIPMI (EXPERIMENTAL) + - `--[no-]web.systemd-socket`: Use systemd socket activation listeners instead of port listeners (Linux only). +- `ipmi_exporter_log_level`: "info" # Only log messages with the given severity or above. One of: [debug, info, warn, error] +- `ipmi_exporter_log_format`: "logfmt" # Output format of log messages. One of: [logfmt, json] +- `ipmi_exporter_web_listen_address`: Addresses on which to expose metrics and web interface. Repeatable for multiple addresses. + - `:9100` or `[::1]:9100` for HTTP + - `vsock://:9100` for vsock +- `ipmi_exporter_local_conf`: (optional) Path to local configuration file. See: https://github.com/prometheus-community/ipmi_exporter/blob/master/docs/configuration.md +- `ipmi_exporter_web_conf`: (optional) Path to web configuration file that can enable TLS or authentication. See: https://github.com/prometheus/exporter-toolkit/blob/master/docs/web-configuration.md ``` Dependencies From 1881e64b12b72168b491dc5f74fb0f6c4a4a4138 Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:48:25 +0400 Subject: [PATCH 8/9] Update Molecule converge to use new variable name --- .../molecule/default/{ipmi_local.conf => configs/local.conf} | 0 roles/ipmi_exporter/molecule/default/converge.yml | 2 +- 2 files changed, 1 insertion(+), 1 deletion(-) rename roles/ipmi_exporter/molecule/default/{ipmi_local.conf => configs/local.conf} (100%) diff --git a/roles/ipmi_exporter/molecule/default/ipmi_local.conf b/roles/ipmi_exporter/molecule/default/configs/local.conf similarity index 100% rename from roles/ipmi_exporter/molecule/default/ipmi_local.conf rename to roles/ipmi_exporter/molecule/default/configs/local.conf diff --git a/roles/ipmi_exporter/molecule/default/converge.yml b/roles/ipmi_exporter/molecule/default/converge.yml index a81a5b8..1782564 100644 --- a/roles/ipmi_exporter/molecule/default/converge.yml +++ b/roles/ipmi_exporter/molecule/default/converge.yml @@ -3,4 +3,4 @@ hosts: all roles: - role: genlab.common.ipmi_exporter - ipmi_exporter_config_dir: "molecule/default/" + ipmi_exporter_local_conf: "configs/local.conf" From b4fa5170d4f54cf77f7a8cacc0e831ebc5d1513a Mon Sep 17 00:00:00 2001 From: Alexander Gorelyshev Date: Fri, 19 Dec 2025 16:48:51 +0400 Subject: [PATCH 9/9] Add a `delay/retry` protection for fetching the release tarball --- roles/ipmi_exporter/tasks/install.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/roles/ipmi_exporter/tasks/install.yml b/roles/ipmi_exporter/tasks/install.yml index d841975..abb2f43 100644 --- a/roles/ipmi_exporter/tasks/install.yml +++ b/roles/ipmi_exporter/tasks/install.yml @@ -24,8 +24,10 @@ group: root mode: "0755" - - name: "Install | Unarchive the tarball" + - name: "Install | Fetch and unarchive the release tarball" notify: "(Re)start and enable ipmi_exporter" + delay: 3 + retries: 3 ansible.builtin.unarchive: src: "https://github.com/prometheus-community/ipmi_exporter/releases/\ download/v{{ ipmi_exporter_version }}/ipmi_exporter-{{ ipmi_exporter_version }}.linux-amd64.tar.gz"