satisfy shellcheck

[log-quiet] / sysd-prom-fail

diff --git a/sysd-prom-fail b/sysd-prom-fail
index ea84f7490797b670d4579a724ec926382c999112..d8a2c0a0d9f97c5976b6583d5c6cd04ba753f2f4 100755 (executable)
--- a/sysd-prom-fail
+++ b/sysd-prom-fail
@@ -1,5 +1,12 @@
 #!/bin/bash
-# Copyright (C) 2022 Ian Kelling
+# I, Ian Kelling, follow the GNU license recommendations at
+# https://www.gnu.org/licenses/license-recommendations.en.html. They
+# recommend that small programs, < 300 lines, be licensed under the
+# Apache License 2.0. This file contains or is part of one or more small
+# programs. If a small program grows beyond 300 lines, I plan to switch
+# its license to GPL.
+
+# Copyright 2024 Ian Kelling
 
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -13,6 +20,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+
+
+# requires 1.4.0 to run with multiple units due to bug before that:
+# https://github.com/prometheus/node_exporter/pull/2475
+
 set -eE -o pipefail
 trap 'echo "$0:$LINENO:error: \"$BASH_COMMAND\" returned $?" >&2' ERR
 
@@ -33,13 +45,7 @@ if [[ ! -e $dir  ]]; then
   exit 0
 fi
 
-# we have to merge files due to this:
-# https://github.com/prometheus/node_exporter/issues/1885
-# or else we could put the label in the metric name, but that
-# is a bad practice.
-# Note, i found this https://github.com/hansmi/prometheus-textformat-merge
-# but it seems overkill.
-f=$dir/${ser_name}-result-fail.premerge
+f=$dir/${ser_name}-result.prom
 ftmp=$f.$$
 
 
@@ -66,17 +72,4 @@ fi
 if $write_count; then
   printf 'node_systemd_unit_result_fail_count{name="%s"} %s\n' "$ser_name" "$count" >$ftmp
   mv $ftmp $f
-  finaltmp=$dir/sysd-result-fail.prom.$$
-  # i had a minor problem that multiple of these scripts executed at the same time, and ended up with
-  # multiple instances of the same file
-  # ls -lai
-  # ...
-  # 3896242 -rw-r--r-- 1 root       root         439 Apr  4 00:02 sysd-result-fail.prom
-  # 3896242 -rw-r--r-- 1 root       root         439 Apr  4 00:02 sysd-result-fail.prom
-  # 3896242 -rw-r--r-- 1 root       root         439 Apr  4 00:02 sysd-result-fail.prom
-  # this seems harmless, https://www.spinics.net/lists/linux-btrfs/msg111245.html
-  # however, lets do a bit to avoid it: sleep a random amount of time from 0-.5 seconds.
-  sleep 0.$(( RANDOM % 500 ))
-  cat $dir/*-result-fail.premerge >$finaltmp
-  mv $finaltmp $dir/sysd-result-fail.prom
 fi