Remove newline from all node files

Was actually added by manually editing the files, but Chef writes them
without newline, so our sanitization script also does it that way

.githooks/pre-commit

@@ -0,0 +1,40 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+SCRIPT="$(git rev-parse --show-toplevel)/scripts/util/sanitize_node.sh"
+
+FILES=$(git diff --cached --name-only --diff-filter=ACM -- 'nodes/*.json' || true)
+
+[[ -z "$FILES" ]] && exit 0
+
+DIRTY=0
+
+while IFS= read -r file; do
+  staged_tmp=$(mktemp)
+  sanitized_tmp=$(mktemp)
+
+  # 1. get staged version
+  git show ":$file" > "$staged_tmp"
+
+  # 2. sanitize IN PLACE (on temp copy)
+  cp "$staged_tmp" "$sanitized_tmp"
+  "$SCRIPT" "$sanitized_tmp"
+
+  # 3. if sanitizer changed file, update working tree
+  if ! diff -q "$staged_tmp" "$sanitized_tmp" >/dev/null; then
+    cp "$sanitized_tmp" "$file"
+    echo "Sanitized: $file"
+    DIRTY=1
+  fi
+
+  rm -f "$staged_tmp" "$sanitized_tmp"
+
+done <<< "$FILES"
+
+if [[ "$DIRTY" -ne 0 ]]; then
+  echo ""
+  echo "Commit paused, sanitized files must be re-staged."
+  exit 1
+fi
+
+exit 0

README.md

@@ -28,6 +28,10 @@ Clone this repository, `cd` into it, and run:
 
     bundle install
 
+### Enable Git hooks
+
+    git config core.hooksPath .githooks
+
 ## Common tasks
 
 ### Bootstrap a new host server

nodes/ejabberd-4.json

@@ -72,4 +72,4 @@
     "role[ldap_client]",
     "role[ejabberd]"
   ]
-}
+}

nodes/ejabberd-8.json

@@ -72,4 +72,4 @@
     "role[ldap_client]",
     "role[ejabberd]"
   ]
-}
+}

nodes/garage-11.json

@@ -63,4 +63,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/garage-14.json

@@ -63,4 +63,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/garage-15.json

@@ -63,4 +63,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/garage-16.json

@@ -61,4 +61,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/garage-7.json

@@ -63,4 +63,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/garage-8.json

@@ -63,4 +63,4 @@
     "role[kvm_guest]",
     "role[garage_node]"
   ]
-}
+}

nodes/gitea-2.json

@@ -76,4 +76,4 @@
     "role[gitea]",
     "role[gitea_actions_runner]"
   ]
-}
+}

nodes/her.json

@@ -55,4 +55,4 @@
     "role[base]",
     "role[kvm_host]"
   ]
-}
+}

nodes/ldap-3.kosmos.org.json

@@ -61,4 +61,4 @@
     "role[kvm_guest]",
     "role[dirsrv_supplier]"
   ]
-}
+}

nodes/ldap-4.kosmos.org.json

@@ -55,4 +55,4 @@
     "role[kvm_guest]",
     "role[dirsrv_supplier]"
   ]
-}
+}

nodes/leo.kosmos.org.json

@@ -56,4 +56,4 @@
     "role[base]",
     "recipe[kosmos_kvm::host]"
   ]
-}
+}

nodes/mail.kosmos.org.json

@@ -68,4 +68,4 @@
     "role[kvm_guest]",
     "role[email_server]"
   ]
-}
+}

nodes/postgres-11.json

@@ -66,4 +66,4 @@
     "role[kvm_guest]",
     "role[postgresql_primary]"
   ]
-}
+}

nodes/postgres-12.json

@@ -61,4 +61,4 @@
     "role[kvm_guest]",
     "role[postgresql_replica]"
   ]
-}
+}

nodes/prometheus-1.json

@@ -59,4 +59,4 @@
     "role[kvm_guest]",
     "role[prometheus_server]"
   ]
-}
+}

nodes/rs-discourse-1.json

@@ -59,4 +59,4 @@
     "role[kvm_guest]",
     "role[remotestorage_discourse]"
   ]
-}
+}

nodes/rsk-mainnet-3.json

@@ -58,4 +58,4 @@
     "role[kvm_guest]",
     "role[rskj_mainnet]"
   ]
-}
+}

nodes/rsk-testnet-4.json

@@ -57,4 +57,4 @@
     "role[base]",
     "role[kvm_guest]"
   ]
-}
+}

nodes/rsk-testnet-5.json

@@ -58,4 +58,4 @@
     "role[kvm_guest]",
     "role[rskj_testnet]"
   ]
-}
+}

nodes/rsk-testnet-6.json

@@ -57,4 +57,4 @@
     "role[kvm_guest]",
     "role[rskj_testnet]"
   ]
-}
+}

nodes/strfry-1.json

@@ -70,4 +70,4 @@
     "role[strfry]",
     "role[blossom]"
   ]
-}
+}

scripts/util/sanitize_node.sh

@@ -0,0 +1,69 @@
+#!/usr/bin/env bash
+# Strip the top-level "override" and "default" attributes from Chef node JSON files.
+#
+# Usage:
+#   ./scripts/util/strip_node_attrs.sh <node> [<node> ...]
+#   ./scripts/util/strip_node_attrs.sh --all
+#
+# A <node> may be a bare node name (e.g. "prometheus-1"), a path relative to the
+# repository root (e.g. "nodes/prometheus-1.json"), or an absolute path.
+#
+# Exit codes:
+#   0 - all target files processed successfully
+#   1 - one or more targets could not be processed (missing args, missing jq,
+#       file not found, invalid JSON, or write failure)
+
+set -uo pipefail
+
+command -v jq >/dev/null 2>&1 || exit 1
+
+root="$(git rev-parse --show-toplevel 2>/dev/null)" || root="$(pwd)"
+
+targets=()
+if [ "$#" -eq 0 ]; then
+  exit 1
+fi
+
+for arg in "$@"; do
+  if [ "$arg" = "--all" ]; then
+    shopt -s nullglob
+    for f in "$root"/nodes/*.json; do
+      targets+=("$f")
+    done
+    shopt -u nullglob
+    continue
+  fi
+
+  if [[ "$arg" == */* ]]; then
+    if [[ "$arg" = /* ]]; then
+      targets+=("$arg")
+    else
+      targets+=("$root/$arg")
+    fi
+  else
+    targets+=("$root/nodes/$arg.json")
+  fi
+done
+
+if [ "${#targets[@]}" -eq 0 ]; then
+  exit 1
+fi
+
+rc=0
+for file in "${targets[@]}"; do
+  [ -f "$file" ] || { rc=1; continue; }
+
+  jq -e . "$file" >/dev/null 2>&1 || { rc=1; continue; }
+
+  tmp="$(mktemp "${file}.XXXXXX")" || { rc=1; continue; }
+  if jq --indent 2 'del(.override, .default)' "$file" > "$tmp" 2>/dev/null; then
+    truncate -s -1 "$tmp" 2>/dev/null || true
+    mv "$tmp" "$file" || { rm -f "$tmp"; rc=1; continue; }
+  else
+    rm -f "$tmp"
+    rc=1
+    continue
+  fi
+done
+
+exit "$rc"