Skip to content

Update tanssi and dancelight bootnodes#1490

Merged
evgeny-s merged 1 commit into
masterfrom
f/update_tanssi_and_dancelight_bootnodes
Feb 17, 2026
Merged

Update tanssi and dancelight bootnodes#1490
evgeny-s merged 1 commit into
masterfrom
f/update_tanssi_and_dancelight_bootnodes

Conversation

@purestakeoskar
Copy link
Copy Markdown
Contributor

Update bootnodes for tanssi and dancelight

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 12, 2026

📝 Walkthrough

Walkthrough

This PR updates bootstrap node configurations in two chain specification files. The dancelight spec removes three boot node endpoints and adds two new RPC endpoints, while the starlight spec replaces four bootstrap DNS entries with four new RPC entries. Telemetry endpoints and protocol IDs remain unchanged in both files.

Changes

Cohort / File(s) Summary
Chain Specification BootNodes Updates
specs/solochain/dancelight-raw-specs.json, specs/solochain/starlight-raw-specs.json
Updated bootstrap node lists with new RPC endpoint configurations. Dancelight spec adds two DNS-over-HTTPS-tunneled RPC endpoints (deo-dancelight-rpc-1, uko-dancelight-rpc-1) while removing three old entries. Starlight spec replaces four old bootstrap DNS entries with four new rpc-1 entries (deo-tanssi-rpc-1, uko-tanssi-rpc-1, fro-tanssi-rpc-1, fro-tanssi-rpc-2).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

A8-mergeoncegreen, not-breaking

Suggested reviewers

  • evgeny-s
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title directly and clearly summarizes the main change: updating bootnodes for tanssi and dancelight networks, which matches the file modifications shown in the raw summary.
Description check ✅ Passed The pull request description is directly related to the changeset, describing the update of bootnodes for tanssi and dancelight, which corresponds to the modifications made to the specification files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into master

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch f/update_tanssi_and_dancelight_bootnodes

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
specs/solochain/dancelight-raw-specs.json (1)

5-8: Bootnode list reduced from 3 to 2 entries.

The entries look well-formed. Note that the reduction from 3 to 2 bootnodes slightly decreases bootstrap redundancy. If additional RPC nodes are available (as was done for starlight with 4 entries), consider adding them for resilience.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 OpenGrep (1.16.0)
specs/solochain/starlight-raw-specs.json

┌──────────────┐
│ Opengrep CLI │
└──────────────┘

�[32m✔�[39m �[1mOpengrep OSS�[0m
�[32m✔�[39m Basic security coverage for first-party code vulnerabilities.

�[1m Loading rules from local config...�[0m

specs/solochain/dancelight-raw-specs.json

┌──────────────┐
│ Opengrep CLI │
└──────────────┘

�[32m✔�[39m �[1mOpengrep OSS�[0m
�[32m✔�[39m Basic security coverage for first-party code vulnerabilities.

�[1m Loading rules from local config...�[0m

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link
Copy Markdown
Contributor

WASM runtime size check:

Compared to target branch

dancebox runtime: 1900 KB (no changes) ✅

flashbox runtime: 1124 KB (no changes) ✅

dancelight runtime: 2656 KB (no changes) 🚨

starlight runtime: 2580 KB (no changes) 🚨

container chain template simple runtime: 1516 KB (no changes) ✅

container chain template frontier runtime: 1848 KB (no changes) ✅

Copy link
Copy Markdown
Contributor

@augustocollado augustocollado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report

(master)

@@                             Coverage Diff                             @@
##           master   f/update_tanssi_and_dancelight_bootnodes     +/-   ##
===========================================================================
  Coverage   73.91%                                     73.91%   0.00%     
  Files         569                                        569             
  Lines       83085                                      83085             
===========================================================================
  Hits        61410                                      61410             
  Misses      21675                                      21675             
Files Changed Coverage

Coverage generated

@evgeny-s evgeny-s added B0-silent Changes should not be mentioned in any release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit A8-mergeoncegreen Pull request is reviewed well. not-breaking Does not need to be mentioned in breaking changes labels Feb 17, 2026
@evgeny-s evgeny-s merged commit d602f6b into master Feb 17, 2026
81 of 99 checks passed
@evgeny-s evgeny-s deleted the f/update_tanssi_and_dancelight_bootnodes branch February 17, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A8-mergeoncegreen Pull request is reviewed well. B0-silent Changes should not be mentioned in any release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants