Skip to content

Fix time_server test's dead PUB5 NTP servers - #390

Merged
ddemlow merged 1 commit into
mainfrom
fix/time-server-public-ntp
Nov 2, 2026
Merged

ddemlow merged 1 commit into
mainfrom
fix/time-server-public-ntp

Conversation

@ddemlow

@ddemlow ddemlow commented Nov 2, 2026

Copy link
Copy Markdown
Member

Summary

`time_server_a`/`time_server_b` (`10.5.11.5`/`10.5.11.75`) were two dedicated chrony NTP VMs on PUB5/XLab's network (see `ci-infra/ntp-chrony/Readme.md`), dead in the same 2026-10-27 outage as the rest of that network. HyperCore validates NTP server reachability server-side, so the test fails outright with `"X is not a valid NTP server"` rather than just storing an unreachable value.

Repointed at two public NTP providers (`time.google.com`, `time.cloudflare.com`) instead of rebuilding dedicated chrony infra on Indy-lab — `smtp-relay.gmail.com` (used by the `smtp` test, same config file) already confirms these clusters have outbound internet reachability. Avoided `pool.ntp.org` domains for either value, per this file's own existing comment that they return KOD (rate-limiting rejection).

Test plan

  • Next `integ-test` run shows `time_server` passing on both `10.6.112.64` and `10.6.42.19`

🤖 Generated with Claude Code

time_server_a/b (10.5.11.5/10.5.11.75) were dedicated chrony VMs on PUB5/XLab's
network (see ci-infra/ntp-chrony/Readme.md), dead in the same 2026-10-27 outage
as the rest of that network. HyperCore validates NTP server reachability
server-side, so the test failed with "X is not a valid NTP server" rather than
just storing an unreachable value.

Repointed at two public NTP providers (time.google.com, time.cloudflare.com)
instead of rebuilding dedicated chrony infra on Indy-lab -- smtp-relay.gmail.com
(used by the smtp test, same config file) already confirms these clusters have
outbound internet reachability. Avoided pool.ntp.org domains for both values,
per this file's own existing comment that they return KOD.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@ddemlow
ddemlow merged commit 4778482 into main Nov 2, 2026
6 checks passed
@ddemlow
ddemlow deleted the fix/time-server-public-ntp branch November 2, 2026 16:59
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