Skip to content

node, deploy: align DHT provider record TTL with the 15m default - #604

Merged
aojea merged 1 commit into
google:mainfrom
aojea:dht-provider-ttl-followups
Oct 8, 2026
Merged

aojea merged 1 commit into
google:mainfrom
aojea:dht-provider-ttl-followups

Conversation

@aojea

@aojea aojea commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Follow up on #602:

  • sam-node initializes its DHT in dht.ModeAuto and promotes to ModeServer when directly reachable (such as pod-to-pod on :5002 in Kubernetes). Default its DHTProviderAddrTTL to 15m (3 * ReprovideInterval) so dialable nodes expire stale provider records on the same schedule as routers instead of keeping the library's 48h default.
  • Drop the per-environment DHT_PROVIDER_ADDR_TTL override in .github/workflows/deploy.yaml and .github/k8s/sam-router-template.yaml: bananas was set to 1h only when 0s meant 48h, which after router: expire provider records 15 minutes after the last announcement #602 left bananas keeping stale provider records four times longer than hub (15m).

Follow up on google#602:

- sam-node initializes its DHT in ModeAuto and promotes to ModeServer when directly reachable (such as pod-to-pod on :5002 in Kubernetes). Default its DHTProviderAddrTTL to 15m (3 * ReprovideInterval) so dialable nodes expire stale provider records on the same schedule as routers instead of keeping the library's 48h default.
- Drop the per-environment DHT_PROVIDER_ADDR_TTL override in deploy.yaml and sam-router-template.yaml: bananas was set to 1h only when 0s meant 48h, which after google#602 left bananas keeping stale provider records four times longer than hub (15m).
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@aojea
aojea merged commit 3a7bb34 into google:main Oct 8, 2026
21 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