Skip to content

Support nv25 and builtin-actors v16#309

Merged
ziscky merged 22 commits into
devfrom
feat/lotus-v1.32.0-rc3
Apr 3, 2025
Merged

Support nv25 and builtin-actors v16#309
ziscky merged 22 commits into
devfrom
feat/lotus-v1.32.0-rc3

Conversation

@ziscky
Copy link
Copy Markdown
Contributor

@ziscky ziscky commented Mar 19, 2025

Support for Lotus nv25 and builtin-actors v16

Overview

This PR introduces support for builtin-actors v16 for NV25 and fixes a significant bug with calibration network version handling.

Key Changes

New Features

  • Added support for builtin-actors v16
  • Implemented version iterator to handle different latest versions for mainnet and calibration networks
  • Updated missing versions across various actors
  • Enhanced version mapping support

Bug Fixes

  • Fixed mainnet edge case issues
  • Resolved calibration network mapping issues
  • Updated various actor implementations for better compatibility

Testing Improvements

  • Increased test coverage across all actor tests
  • Updated test suite to use new LatestVersion method
  • Enhanced test utilities and improved test cases

Code Maintenance

  • Updated dependencies (go.mod and go.sum)
  • Performed code cleanup and organization
  • Added new version iterator utility (tools/version_iterator.go)

Modified Components

The changes affect multiple components including:

  • Account, Cron, Datacap, EAM, EVM actors
  • Init, Market, Miner actors
  • Multisig, Payment Channel actors
  • Power, Reward, Verified Registry actors
  • Testing utilities and version mapping tools

Testing

All tests have been updated and verified to pass with the new changes. Test coverage has been improved across the codebase.

Copy link
Copy Markdown
Member

@emmanuelm41 emmanuelm41 left a comment

Choose a reason for hiding this comment

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

It is simply amazing to have this shipped and ready to be activated the moment the network height is reached. I am in love with this. Just leave a question: Why tests have changed SO much? I see a really big change on tests now.

Comment thread actors/v2/market/market.go Outdated
Copy link
Copy Markdown
Contributor

@JulianToledano JulianToledano left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Comment thread tools/version_iterator.go Outdated
Comment thread tools/version_mapping.go Outdated
@ziscky ziscky changed the title Lotus v1.32.0-rc3 Full Lotus v1.32.0-rc3 Mar 21, 2025
@ziscky ziscky changed the title Full Lotus v1.32.0-rc3 Support NV25 Mar 21, 2025
@ziscky ziscky changed the title Support NV25 Support nv25 and builtin-actors v16 Mar 21, 2025
ziscky and others added 9 commits March 21, 2025 14:05
* feat: parse sector related events

* feat: add claimed power tracking

* fix: add missing info.

* feat: refactor sector parsing

* test: add bigint test

* test: add utils tests

* fix: lint

* fix: only report errors from succesfully executed traces

* fix: parse AwardBlockReward correctly

* feat: add supported version

* fix: calibration executiontrace format

* fix: refactor parser selection for calibration

* feat: add utility methods and missing miner methods

* fix: miner message filtering

* fix: default consts

* feat: store raw data if parsing fails

* fix: check if address is empty before parsing

* fix: fee transactions

* fix: parse initialpledge
@ziscky ziscky merged commit b0806de into dev Apr 3, 2025
5 checks passed
@ziscky ziscky deleted the feat/lotus-v1.32.0-rc3 branch April 3, 2025 18:22
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.

3 participants