fix(f3): set F3 initial power table on calibnet#6590
Conversation
WalkthroughSets calibnet's F3 InitialPowerTable to a specific CID, enables related runtime checks, updates test thresholds and RPC filter lists, and documents the change in the changelog. Changes
Sequence Diagram(s)sequenceDiagram
participant Deployer as Deployer (config change)
participant Sidecar as F3 Sidecar / Manifest
participant Node as Node (networks/mod.rs)
participant TestRunner as Test scripts & API tests
Deployer->>Sidecar: commit f3manifest_calibnet.json (InitialPowerTable=CID)
Deployer->>Node: commit networks/mod.rs (enable f3_initial_power_table)
Sidecar-->>Node: sidecar will serve initial power table (runtime)
TestRunner->>Node: start readiness loop (wait GPBFTInstance > 50)
Note right of TestRunner: once threshold met
TestRunner->>Sidecar: run `f3 status`, `f3 manifest`, `f3 certs list`, `f3 certs get`
TestRunner->>API: run API-compare with Filecoin.F3 included
API-->>TestRunner: certificate responses validated
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@CHANGELOG.md`:
- Line 32: Confirm whether a tracking issue exists for the changelog entry
currently using the PR reference
"[`#6590`](https://github.com/ChainSafe/forest/pull/6590): Set F3
`InitialPowerTable` on calibnet."; if an issue exists, replace the PR link with
the issue link using the format "[`#ISSUE_NO`](link-to-issue): Set F3
`InitialPowerTable` on calibnet." otherwise keep the PR reference as-is; update
the CHANGELOG.md entry accordingly and ensure the description text ("Set F3
`InitialPowerTable` on calibnet.") remains unchanged.
In `@scripts/tests/calibnet_no_discovery_check.sh`:
- Around line 30-37: The echo labels for the certs subcommands are swapped and
misleading; update the two echo statements so they match the commands that
follow — i.e., ensure the echo before "$FOREST_CLI_PATH f3 certs list" reads
"Test subcommands: f3 certs list" and the echo before "$FOREST_CLI_PATH f3 certs
get" reads "Test subcommands: f3 certs get" so the output aligns with the actual
commands.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
... and 6 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
Outside contributions
Summary by CodeRabbit
Bug Fixes
Tests
Documentation