Skip to content

feat(checks): warn when a dual-stack probe only answered on its fallback family - #852

Merged
fclairamb merged 7 commits into
feat/todos-2026-11-11from
feat/todos-2026-11-12
Nov 12, 2026
Merged

fclairamb merged 7 commits into
feat/todos-2026-11-11from
feat/todos-2026-11-12

Conversation

@fclairamb

Copy link
Copy Markdown
Owner

Stacked on #848: merge it first.

What changed

  • HTTP and Prometheus checks now return warning (rolled up as degraded) when a dual-stack target only answered on its fallback family, e.g. IPv6 unreachable but IPv4 up (answers the "which family failed" part of the issue, the IPv4 fallback itself shipped in fix(egress): fall back to IPv4 when the IPv6 path hangs (Happy Eyeballs) #849). The result carries output.ip_fallback and the dashboard shows it. Pinned-family checks never warn.

Decided by autopilot

  • Spec fclairamb/solidping-business#69 follow-up, Q1: warn only when the first family failed or was still pending 1 s after its start (recommended answer).
  • Q2: no separate v4/v6 sub-probes (recommended answer).

Notes

  • With the 1 s threshold and the 300 ms race, a silently dropped v6 SYN with instant v4 will not warn; tune FamilyFallbackThreshold if wanted.
  • An HTTP check on localhost can now warn when ::1 is refused but 127.0.0.1 answers.
  • Batch gate: build, lint, Go tests, e2e typecheck/lint green; Playwright not run locally (left to CI).

Closes fclairamb/solidping-business#69

fclairamb and others added 6 commits November 12, 2026 09:12
…ck family (fclairamb/solidping-business#69)

The Happy Eyeballs race (0c5ccaf) keeps a lossy IPv6 path from making a
dual-stack target look down; this spec restores the lost signal as a warning.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ack family

Observe each connect attempt per family (httptrace); when the first family
failed, or stayed silent for 1 s, while the other connected, the HTTP and
Prometheus checks return a warning with output.ip_fallback and a message.
A failure keeps its status; a pinned family never warns. Shown in the
dashboard (result-ip-fallback) and documented.

Closes fclairamb/solidping-business#69

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…h on its fallback family should say so

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…h on its fallback family should say so

Adds checkhttp tests for the guard and stdlib dialer fallback paths and a
Prometheus scrape fallback test; documents why the message says "this region".

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@fclairamb fclairamb added autopilot frozen Batch PR under review: takes only its own fixes, new work goes to the next batch implemented Every spec of the PR is implemented and its batch gate was green automerge labels Nov 12, 2026
@fclairamb
fclairamb merged commit 45c3c21 into feat/todos-2026-11-11 Nov 12, 2026
2 checks passed
@fclairamb
fclairamb deleted the feat/todos-2026-11-12 branch November 12, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge autopilot autorelease frozen Batch PR under review: takes only its own fixes, new work goes to the next batch implemented Every spec of the PR is implemented and its batch gate was green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant