Skip to content

Let sf and sf2 push metrics and logs to the monitoring hub - #12

Merged
TakGN merged 2 commits into
mainfrom
monitoring-ingest-sf-sf2
Oct 22, 2026
Merged

TakGN merged 2 commits into
mainfrom
monitoring-ingest-sf-sf2

Conversation

@TakGN

@TakGN TakGN commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

Part of ipni/infra#188 (register sf and sf2 with Argo CD), under ipni/infra#166.

sf (103.232.206.165) and sf2 (103.232.206.162) aren't in this inventory, so the monitoring hub's 8443 allowlist on lith-1 doesn't cover them. Once they're registered with Argo CD, Alloy on them pushes metrics and logs to ingest.monitoring.ipni.io:8443, which goes straight to lith-1, not through Cloudflare. This sets monitoring_ingest_extra_sources to their two addresses, as the comment in the role already suggested.

What it changes on the boxes

  • lith-1 only: two new ufw rules, 8443/tcp ALLOW <ip> # monitoring ingest. ufw_allowlist keeps the rules in step with the list, so dropping an address later removes its rule.
  • Nothing on sing-1 or chic-1, and no restart.
  • Safe to apply before sf/sf2 are registered: ingest still requires the alloy basic-auth password, and nothing pushes from those addresses until Alloy runs there.

Rollout

CI runs a platform.yml dry run on this PR. After the merge it runs another dry run, then the apply, once a required reviewer approves it in production. The dry run should show only the two monitoring ingest rules on lith-1.

🤖 Generated with Claude Code

TakGN and others added 2 commits October 22, 2026 13:06
sf and sf2 are not in the inventory, so the hub's 8443 allowlist does not
cover them. They are about to run Alloy next to their old agents
(ipni/infra#166, #188).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@TakGN
TakGN merged commit 1225714 into main Oct 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant