chore(gatus): drop the PBS 2 check, which has no clients left #699

2026-08-28T09:29:57Z

claude-bot commented

2026-08-28 09:29:57 +00:00

pbs2.renner.uno:8007 wurde alle 60 Sekunden als „Infrastructure 1" geprüft. Beide deklarierten Clients sind weg:

der PVE-Cluster, laut pve_hosts.yml nach Bare-Metal-Talos migriert („no SSH at all")
nas-1, ausgeschaltet und mit #696 aus dem Repo entfernt

Ein Liveness-Check auf einen Backup-Server, auf den nichts sichert, berichtet über nichts. Und hide-errors: true sorgte dafür, dass sein Status auf der öffentlichen Seite ohnehin keine Aussage trug.

Das Label behält seine Nummer

Die Legende führt bereits Infrastructure 2 = retired (was Proxmox VE) — dieselbe Konvention gilt jetzt für 1. Die Labels sind öffentlich sichtbar; umzunummerieren würde still die Bedeutung jeder anderen Nummer verschieben.

Nebenbefund im selben Kommentarblock

Die Wartungs-Legende endete bei Web Service 9, die Endpunkte führten aber ein Web Service 10 — das ist TRMNL. Die Zuordnung stimmte ab 10 also nicht mehr mit der Realität überein. Ergänzt.

Geprüft

Die eingebettete config.yaml parst weiterhin, 12 Endpunkte bleiben (vorher 13), und die einzige verbliebene pbs2-Erwähnung ist die Entfernungsnotiz selbst. kubectl apply --dry-run=server sauber.

Zur Herkunft

Der Fund stammt aus deiner Korrektur, dass alles aus dem Repo beantwortbar ist. Ich hatte „was nas-1 gesichert hat" als offene Frage weitergereicht, statt in der Historie nachzusehen — git show 847664ea~1:ansible/inventory/host_vars/nas-1.renner.uno/vars.yml beantwortet sie vollständig: neun Pfade nach pbs2, Datastore pool, plus die Garage-Rolle, die Hetzner ersetzt hat. Beim Verfolgen von pbs2 durchs Repo fiel dann auf, dass der Server selbst noch überwacht wurde.

`pbs2.renner.uno:8007` wurde alle 60 Sekunden als „Infrastructure 1" geprüft. Beide deklarierten Clients sind weg: - der PVE-Cluster, laut `pve_hosts.yml` nach Bare-Metal-Talos migriert („no SSH at all") - nas-1, ausgeschaltet und mit #696 aus dem Repo entfernt Ein Liveness-Check auf einen Backup-Server, auf den nichts sichert, berichtet über nichts. Und `hide-errors: true` sorgte dafür, dass sein Status auf der öffentlichen Seite ohnehin keine Aussage trug. ## Das Label behält seine Nummer Die Legende führt bereits `Infrastructure 2 = retired (was Proxmox VE)` — dieselbe Konvention gilt jetzt für 1. Die Labels sind **öffentlich sichtbar**; umzunummerieren würde still die Bedeutung jeder anderen Nummer verschieben. ## Nebenbefund im selben Kommentarblock Die Wartungs-Legende endete bei `Web Service 9`, die Endpunkte führten aber ein **`Web Service 10`** — das ist TRMNL. Die Zuordnung stimmte ab 10 also nicht mehr mit der Realität überein. Ergänzt. ## Geprüft Die eingebettete `config.yaml` parst weiterhin, 12 Endpunkte bleiben (vorher 13), und die einzige verbliebene `pbs2`-Erwähnung ist die Entfernungsnotiz selbst. `kubectl apply --dry-run=server` sauber. ## Zur Herkunft Der Fund stammt aus deiner Korrektur, dass alles aus dem Repo beantwortbar ist. Ich hatte „was nas-1 gesichert hat" als offene Frage weitergereicht, statt in der Historie nachzusehen — `git show 847664ea~1:ansible/inventory/host_vars/nas-1.renner.uno/vars.yml` beantwortet sie vollständig: neun Pfade nach `pbs2`, Datastore `pool`, plus die Garage-Rolle, die Hetzner ersetzt hat. Beim Verfolgen von `pbs2` durchs Repo fiel dann auf, dass der Server selbst noch überwacht wurde.

claude-bot added 1 commit 2026-08-28 09:29:58 +00:00

chore(gatus): drop the PBS 2 check, which has no clients left

ci/woodpecker/pr/ci-manifests-test Pipeline was successful

Details

ci/woodpecker/pr/ci-scripts-test Pipeline was successful

Details

ci/woodpecker/pr/pr-review Pipeline was successful

Details

75e8a81c56

pbs2.renner.uno was probed every 60 seconds as "Infrastructure 1". Both of its
declared clients are gone: the PVE cluster, retired to bare-metal Talos, and
nas-1, powered off and removed from the repo. A liveness check on a backup
server that nothing backs up to reports on nothing — and hide-errors: true
meant its status carried no detail on the public page either.

The opaque label keeps its number and is marked retired, the way Infrastructure
2 already records the Proxmox VE slot. Those labels are public; renumbering
would silently reassign what every other number means.

Also fills a gap in the maintainer legend: Web Service 10 (TRMNL) was serving
but undocumented, so the map stopped matching the endpoint list at 9.

Checked: the embedded config still parses, 12 endpoints remain, and the only
surviving pbs2 mention is the removal note.

review-bot reviewed

2026-08-28 09:31:56 +00:00

review-bot left a comment

🤖 Automated review — model claude-haiku-4-5-20251001 (complexity: trivial), commit 75e8a81c5661e67d8ae5d9f54b78cb07222bcac7.

Advisory review by the review-bot pipeline. It never approves; a human merges. Push new commits to get a fresh review.

Summary

This PR removes the monitoring check for PBS 2 (pbs2.renner.uno:8007), which no longer has any clients. The change updates the legend to mark Infrastructure 1 as retired (consistent with Infrastructure 2), and includes a well-reasoned explanatory comment. As a side fix, it also adds the missing "Web Service 10 = TRMNL" entry to the legend that was already referenced in the config but undocumented.

🟢 Nits

kubernetes/apps/gatus/configmap.yaml:18 — The legend update maintains the intentional gap in numbering (Infrastructure 1 and 2 are both retired while 3+ are active). The PR body explains this is by design since labels are publicly visible — renumbering would alter the meaning of existing numbers. This is reasonable, though it's worth noting the gap exists.

Tests

The PR is configuration-only (Kubernetes ConfigMap). The author verified:

Embedded config.yaml still parses correctly
Endpoint count decreased from 13 to 12 (as expected)
kubectl apply --dry-run=server passes without errors

No test changes needed; the removal is safe since it's simply deleting an unused monitoring target.

Recommendation

Looks reasonable — no blocking issues; the nit above is optional context.

🤖 **Automated review** — model `claude-haiku-4-5-20251001` (complexity: trivial), commit `75e8a81c5661e67d8ae5d9f54b78cb07222bcac7`. > Advisory review by the `review-bot` pipeline. It never approves; a human merges. Push new commits to get a fresh review. --- ## Summary This PR removes the monitoring check for PBS 2 (pbs2.renner.uno:8007), which no longer has any clients. The change updates the legend to mark Infrastructure 1 as retired (consistent with Infrastructure 2), and includes a well-reasoned explanatory comment. As a side fix, it also adds the missing "Web Service 10 = TRMNL" entry to the legend that was already referenced in the config but undocumented. ## 🟢 Nits - `kubernetes/apps/gatus/configmap.yaml:18` — The legend update maintains the intentional gap in numbering (Infrastructure 1 and 2 are both retired while 3+ are active). The PR body explains this is by design since labels are publicly visible — renumbering would alter the meaning of existing numbers. This is reasonable, though it's worth noting the gap exists. ## Tests The PR is configuration-only (Kubernetes ConfigMap). The author verified: - Embedded `config.yaml` still parses correctly - Endpoint count decreased from 13 to 12 (as expected) - `kubectl apply --dry-run=server` passes without errors No test changes needed; the removal is safe since it's simply deleting an unused monitoring target. ## Recommendation Looks reasonable — no blocking issues; the nit above is optional context.

k.renner merged commit f935776f2b into master

2026-08-28 09:33:48 +00:00

k.renner referenced this issue from a commit

2026-08-28 09:33:49 +00:00

Merge pull request 'chore(gatus): drop the PBS 2 check, which has no clients left' (#699) from drop-pbs2-gatus into master

claude-bot referenced this pull request

2026-08-28 09:44:42 +00:00

fix(gatus): reload the pod when its config changes #700

claude-bot referenced this issue from a commit

2026-08-28 09:54:33 +00:00

fix(gatus): drop the file.renner.uno check left behind by #696

claude-bot referenced this pull request