ci: fix check-unused-variables

Signed-off-by: juju4 <juju4@users.noreply.github.com>
prometheus-community · Jan 19, 2025 · 940c21e · 940c21e
1 parent 3a07f28
commit 940c21e
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 26 deletions.
diff --git a/roles/dns_exporter/defaults/main.yml b/roles/dns_exporter/defaults/main.yml
@@ -1,8 +1,6 @@
 ---
 dns_exporter_version: 1.0.0
 # not go, but python pip. placeholders for _common
-dns_exporter_binary_url: ""
-dns_exporter_checksums_url: ""
 dns_exporter_textfile_dir: ""
 dns_exporter_enabled_collectors: []
 dns_exporter_disabled_collectors: []
@@ -12,10 +10,8 @@ dns_exporter_basic_auth_users: {}
 # just for "prometheus.prometheus._common : Naive assertion of proper listen address" else not used
 dns_exporter_web_listen_address: "127.0.0.1:15353"
 
-dns_exporter_web_disable_exporter_metrics: false
 dns_exporter_web_listen_ip: "127.0.0.1"
 dns_exporter_web_listen_port: "15353"
-dns_exporter_web_telemetry_path: "/metrics"
 
 dns_exporter_debug_enable: false
 
@@ -28,5 +24,3 @@ dns_exporter_system_group: "dns-exp"
 dns_exporter_system_user: "{{ dns_exporter_system_group }}"
 
 dns_exporter_config_dir: "/etc/dns_exporter"
-# Local path to stash the archive and its extraction
-dns_exporter_local_cache_path: "/tmp/dns_exporter-{{ ansible_system | lower }}-{{ _dns_exporter_go_ansible_arch }}/{{ dns_exporter_version }}"
diff --git a/roles/dns_exporter/meta/argument_specs.yml b/roles/dns_exporter/meta/argument_specs.yml
@@ -30,22 +30,9 @@ argument_specs:
       dns_exporter_version:
         description: "DNS exporter package version. Also accepts latest as parameter."
         default: "1.0.0"
-      dns_exporter_binary_url:
-        description: "URL of the dns exporter binaries .tar.gz file"
-        default: "https://github.com/{{ _dns_exporter_repo }}/releases/download/v{{ dns_exporter_version }}/dns_exporter-{{ dns_exporter_version }}.{{ ansible_system | lower }}-{{ _dns_exporter_go_ansible_arch }}.tar.gz"
-      dns_exporter_checksums_url:
-        description: "URL of the dns exporter checksums file"
-        default: "https://github.com/{{ _dns_exporter_repo }}/releases/download/v{{ dns_exporter_version }}/sha256sums.txt"
-      dns_exporter_web_disable_exporter_metrics:
-        description: "Exclude metrics about the exporter itself (promhttp_*, process_*, go_*)."
-        type: bool
-        default: false
       dns_exporter_web_listen_address:
         description: "Address on which dns exporter will listen"
         default: "0.0.0.0:15353"
-      dns_exporter_web_telemetry_path:
-        description: "Path under which to expose metrics"
-        default: "/metrics"
       dns_exporter_enabled_collectors:
         description:
           - "List of dicts defining additionally enabled collectors and their configuration."
@@ -90,9 +77,6 @@ argument_specs:
           - "I(Advanced)"
           - "DNS exporter user"
         default: "dns-exp"
-      dns_exporter_local_cache_path:
-        description: "Local path to stash the archive and its extraction"
-        default: "/tmp/dns_exporter-{{ ansible_system | lower }}-{{ _dns_exporter_go_ansible_arch }}/{{ dns_exporter_version }}"
       dns_exporter_config_dir:
         description: "Path to directory with dns_exporter configuration"
         default: "/etc/dns_exporter"
diff --git a/roles/dns_exporter/molecule/alternative/molecule.yml b/roles/dns_exporter/molecule/alternative/molecule.yml
@@ -5,7 +5,6 @@ provisioner:
   inventory:
     group_vars:
       all:
-        dns_exporter_local_cache_path: "/tmp/dns_exporter-linux-amd64"
         dns_exporter_web_listen_address:
           - '127.0.0.1:8080'
           - '127.0.1.1:8080'
@@ -22,5 +21,3 @@ provisioner:
         dns_exporter_basic_auth_users:
           randomuser: examplepassword
         dns_exporter_version: 1.5.0
-        dns_exporter_binary_url: "https://github.com/prometheus/dns_exporter/releases/download/v{{\
-                                   \ dns_exporter_version }}/dns_exporter-{{ dns_exporter_version }}.linux-amd64.tar.gz"
diff --git a/roles/dns_exporter/vars/main.yml b/roles/dns_exporter/vars/main.yml
@@ -6,4 +6,3 @@ _dns_exporter_go_ansible_arch: "{{ {'i386': '386',
                                  'armv6l': 'armv6'}.get(ansible_architecture, ansible_architecture) }}"
 _dns_exporter_repo: "tykling/dns_exporter"
 _github_api_headers: "{{ {'GITHUB_TOKEN': lookup('ansible.builtin.env', 'GITHUB_TOKEN')} if (lookup('ansible.builtin.env', 'GITHUB_TOKEN')) else {} }}"
-_dns_exporter_binaries: ['dns_exporter']