Skip to content

chore: use /dnsaddr to aggregate CS bootstrappers - #7375

Merged
hanabi1224 merged 1 commit into
mainfrom
use-dnsaddr-bootstrap
Jul 16, 2026
Merged

chore: use /dnsaddr to aggregate CS bootstrappers#7375
hanabi1224 merged 1 commit into
mainfrom
use-dnsaddr-bootstrap

Conversation

@LesnyRumcajs

@LesnyRumcajs LesnyRumcajs commented Jul 16, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Summary of changes

❯ dig +short TXT _dnsaddr.bootstrap-calibnet.chainsafe-fil.io
"dnsaddr=/dns/bootstrap-calibnet-2.chainsafe-fil.io/tcp/34000/p2p/12D3KooWEiBN8jBX8EBoM3M47pVRLRWV812gDRUJhMxgyVkUoR48"
"dnsaddr=/dns/bootstrap-calibnet-1.chainsafe-fil.io/tcp/34000/p2p/12D3KooWS3ZRhMYL67b4bD5XQ6fcpTyVQXnDe8H89LvwrDqaSbiT"
"dnsaddr=/dns/bootstrap-calibnet-0.chainsafe-fil.io/tcp/34000/p2p/12D3KooWABQ5gTDHPWyvhJM7jPhtNwNJruzTEo32Lo4gcS5ABAMm"

❯ dig +short TXT _dnsaddr.bootstrap-mainnet.chainsafe-fil.io
"dnsaddr=/dns/bootstrap-mainnet-0.chainsafe-fil.io/tcp/34000/p2p/12D3KooWKKkCZbcigsWTEu1cgNetNbZJqeNtysRtFpq7DTqw3eqH"
"dnsaddr=/dns/bootstrap-mainnet-2.chainsafe-fil.io/tcp/34000/p2p/12D3KooWHQRSDFv4FvAjtU32shQ7znz7oRbLBryXzZ9NMK2feyyH"
"dnsaddr=/dns/bootstrap-mainnet-1.chainsafe-fil.io/tcp/34000/p2p/12D3KooWGnkd9GQKo3apkShQDaq1d6cKJJmsVe6KiQkacUk1T8oZ"

Changes introduced in this pull request:

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • I have read and agree to the CONTRIBUTING document.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

Summary by CodeRabbit

  • Chores
    • Updated calibration and mainnet network bootstrap configuration to use consolidated DNS-based peer discovery.
    • Retained the existing calibration bootstrap entry.

@LesnyRumcajs
LesnyRumcajs requested a review from a team as a code owner July 16, 2026 15:37
@LesnyRumcajs
LesnyRumcajs requested review from hanabi1224 and sudo-shashank and removed request for a team July 16, 2026 15:37
@coderabbitai

coderabbitai Bot commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2913f536-aa44-4859-823d-a1762891f9b7

📥 Commits

Reviewing files that changed from the base of the PR and between 5c99bb0 and f56b7aa.

📒 Files selected for processing (2)
  • build/bootstrap/calibnet
  • build/bootstrap/mainnet
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • filecoin-project/lotus (manual)

Walkthrough

Bootstrap configurations for Calibnet and mainnet replace hardcoded peer multiaddresses with consolidated DNS-based multiaddresses.

Changes

Bootstrap endpoint configuration

Layer / File(s) Summary
DNS bootstrap entries
build/bootstrap/calibnet, build/bootstrap/mainnet		 Calibnet and mainnet bootstrap peer lists now use /dnsaddr/ entries instead of explicit node addresses.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: hanabi1224, sudo-shashank

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: switching ChainSafe bootstrap peers to /dnsaddr aggregation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch use-dnsaddr-bootstrap
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch use-dnsaddr-bootstrap

Comment @coderabbitai help to get the list of available commands.

@LesnyRumcajs LesnyRumcajs mentioned this pull request Jul 16, 2026
Merged
2 of 8 tasks
@hanabi1224
hanabi1224 added this pull request to the merge queue Jul 16, 2026
Merged via the queue into main with commit 516fe70 Jul 16, 2026
34 checks passed
@hanabi1224
hanabi1224 deleted the use-dnsaddr-bootstrap branch July 16, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sudo-shashank sudo-shashank Awaiting requested review from sudo-shashank sudo-shashank is a code owner automatically assigned from ChainSafe/forest
1 more reviewer
@hanabi1224 hanabi1224 hanabi1224 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LesnyRumcajs @hanabi1224