next #305

Open
clawbot wants to merge 1 commits from next into main
Collaborator

On next since main, reviewed, tagged v1.0.0-rc4: the health check answers 503 with status error when every lookup in the last round of DNS checks got no usable reply from a root server, as on a network that intercepts DNS queries (#302). Before that it answered ok while dnswatcher checked nothing.

To know before merging or deploying:

  • If fsn1app1 cannot reach the root servers, or answers DNS queries itself, the container turns unhealthy when the first round of checks finishes; until then the health check answers ok. If that round ends within the 60 seconds upaas waits, upaas marks the deploy failed.
  • A network outage that fails a whole round keeps the health check at error until the next round, up to DNSWATCHER_DNS_INTERVAL (1 hour by default).
  • Merge with a merge commit, as last time: next contains main, and a squash makes them diverge.
  • #303 (main into prod) is the deploy. Merge this first and that PR carries this fix as well.

No known bugs are open.

Model: opus-5-5

On `next` since `main`, reviewed, tagged `v1.0.0-rc4`: the health check answers 503 with `status` `error` when every lookup in the last round of DNS checks got no usable reply from a root server, as on a network that intercepts DNS queries (https://git.eeqj.de/sneak/dnswatcher/issues/302). Before that it answered `ok` while dnswatcher checked nothing. To know before merging or deploying: - If fsn1app1 cannot reach the root servers, or answers DNS queries itself, the container turns unhealthy when the first round of checks finishes; until then the health check answers `ok`. If that round ends within the 60 seconds upaas waits, upaas marks the deploy failed. - A network outage that fails a whole round keeps the health check at `error` until the next round, up to `DNSWATCHER_DNS_INTERVAL` (1 hour by default). - Merge with a merge commit, as last time: `next` contains `main`, and a squash makes them diverge. - https://git.eeqj.de/sneak/dnswatcher/pulls/303 (`main` into `prod`) is the deploy. Merge this first and that PR carries this fix as well. No known bugs are open. Model: opus-5-5
clawbot added 1 commit 2026-10-18 17:28:50 +02:00
The health check answered ok while every DNS lookup failed, as on a
network that intercepts DNS queries. A lookup whose root servers give no
usable reply now fails with ErrNoRootServerAnswered. At the end of each
round of DNS checks that shutdown did not cut short, the watcher records
whether every lookup failed with it or with ErrIntercepted; the health
check then gives status error, and /.well-known/healthcheck and /health
answer 503. Until the first round finishes it stays ok.

The health check now depends on the watcher, so fx starts the watcher
before the HTTP server and stops the server first. NewWithRootServers
lets tests start a resolver's walks at real servers that are not root
servers.

Model: opus-5-5
sneak was assigned by clawbot 2026-10-18 17:29:01 +02:00
clawbot added this to the 1.0 milestone 2026-10-18 17:29:02 +02:00
clawbot added the merge-ready label 2026-10-18 17:29:02 +02:00
Some required checks failed
check / check (push) Canceled after 0s
Required
Details
Some required checks were not successful.
You are not authorized to merge this pull request.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin next:next
git checkout next
Sign in to join this conversation.