V14 Builtin Actors - Githubissues

ChainSafe / fil-actor-states

State handling crates for the Filecoin Actors

Apache License 2.0

3 stars 2 forks source link

V14 Builtin Actors #311

Closed sudo-shashank closed 2 months ago

sudo-shashank commented 2 months ago

Summary of changes Changes introduced in this pull request:

Added v14 actors
Updated KNOWN_CIDS and all actor methods in fil-actor-interface.
Increase rust stack size to RUST_MIN_STACK=8388608 due to builtin::known_cids::tests::test_loading_static_value

Other information and links

https://github.com/filecoin-project/builtin-actors/releases/tag/v14.0.0-rc.1

ruseinov commented 2 months ago

@sudo-shashank are we considering rc1 stable?

LesnyRumcajs commented 2 months ago

@sudo-shashank are we considering rc1 stable?

It is considered stable once testing is done, and we are confident that it will work correctly with the upcoming calibration network upgrade.

ruseinov commented 2 months ago

@sudo-shashank are we considering rc1 stable?

It is considered stable once testing is done, and we are confident that it will work correctly with the upcoming calibration network upgrade.

What does this mean for this PR? I am assuming it'll need to hang here till it's confirmed stable.

LesnyRumcajs commented 2 months ago

@sudo-shashank are we considering rc1 stable?

It is considered stable once testing is done, and we are confident that it will work correctly with the upcoming calibration network upgrade.

What does this mean for this PR? I am assuming it'll need to hang here till it's confirmed stable.

Yeah, until we test that it works correctly for the NV23 upgrade, there's just no point in merging it. I'll give a green light once it's done.