chore(genvm): update runner hashes to v0.3.0-rc7 ⬆️ - #1744
Merged
ci-core-e2e-runner[bot] merged 1 commit intoAug 24, 2026
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
kp2pml30
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 19, 2026 13:08
aadbb52 to
e71303f
Compare
kp2pml30
marked this pull request as ready for review
August 19, 2026 13:46
MuncleUscles
approved these changes
Aug 20, 2026
kp2pml30
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 21, 2026 09:35
d84c4bd to
2da2c6b
Compare
MuncleUscles
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 21, 2026 10:25
01ad3ee to
2da2c6b
Compare
Squashed: the v0.3.0-rc7 runner-hash bump and the gl_call v1 repin with the re-vendored backend/node/genvm/origin/ mirror.
kp2pml30
force-pushed
the
chore/update-genvm-runner-hashes
branch
from
August 21, 2026 11:33
2da2c6b to
badb7b0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends-On: genlayerlabs/genvm-manager#24
What
Moves Studio onto GenVM v0.3.0-rc7 through genvm-manager #24. 52 files:
examples/contracts/*.py,tests/integration/icontracts/contracts/**,tests/direct/**,tests/load/contracts/counter.py,tests/test_linter_endpoint.py,tests/unit/test_genvm_debug_mode_gate.py,tests/integration/test_{upgrade_contract,deploy_reroute_to}.py, plus.github/workflows/genvm-lint.yml— its skip-list greps the pin, so a stale value there would silently re-enable linting for contracts the released linter cannot loadbackend/node/genvm/origin/—base_host.py,fees.py,public_abi.py, plushost_fns.pyandlog_asserts.py, which upstream changed in164d508and Studio had never picked up.host_fns.ResultCodeis now mandatory becausebase_hostreferences itaccepted→decidedat the two GenVM boundaries, andStorageType.LATEST_FINAL→LATEST_FINALIZEDResultCodemovedpublic_abi→host_fnsat all 9 call sites (public_abi's copy lostINTERNAL_ERROR)docker/scripts/download_genvm.shprobes.zipthen.tarin the shared runners tree;legacy-runnersstays.tarthird_party/genvm/versionfollows the verified current head offix/vm-fatal-errorsin genvm-manager OpenAI integration #24Why
The pre-finalization state was renamed from
acceptedtodecidedon the wire with no back-compat alias, and every v0.3-line runner hash changed. Hosts and SDKs have to move in lockstep — a contract pinning a hash absent from the installed build simply fails to run.Testing done
tests/unit— 1207 passed, 7 skipped in a venv built from the repo's requirementsruffclean;blackclean on everything touchedtests/consensus,tests/db-sqlalchemy) and integration/e2e were not runnable locallyResultCodeimport) are folded into the commitDecisions made
_emission_oninbackend/node/genvm/base.pymaps inbounddecided→accepted;_genvm_message_onand the unmetered allocation inbackend/protocol_rpc/fees.pysenddecidedoutbound. Studio's ownacceptedis a transaction status and a persistedtriggered_oncolumn value — renaming it through would mean a DB migration plus RPC and frontend changes for no behavioural gainblack+autoflake. Studio's copies were byte-identical toblack(upstream@12a0b4d), i.e. zero local patches, so no smart-merge was neededv0.121is untouched — it is still on the v0.2 line (1jb45aa8…), whose hashes did not changeReviewing tips
The 32 hash-bump files are mechanical; the review value is in
backend/node/genvm/origin/(re-vendor) and the 3 boundary sites that mapdecided↔accepted.Known issues
RUNNER_PINSindownload_genvm.shgrepspy-genlayer:[a-z0-9]+, which never matchespy-genlayer-multi:— pre-existing, but the multi hash now goes unverified at acquisition timebackend/protocol_rpc/fees.pyandtests/integration/test_upgrade_contract.pycarry a pre-existing blank-line diff under modernblackChecks
User facing release notes
Studio now targets GenVM v0.3.0-rc7. Contracts must pin the rc7 runner hashes and use
on='decided'in place ofon='accepted'.Changeset
v0.6-devv0.6-devv0.123-devv0.19-devv0.30-devmainmainNo PR for genlayer-js or genlayer-consensus — neither needed a change.
Depends-On:is declared only against genvm-manager#24 here. genlayerlabs/genlayer-node#1753 is the hub: it carriesDepends-Onfor every repo in the set, so one E2E run from there resolves the whole closure and fans checks out to each open PR. Sibling rows above are deliberately plain links — a fully-connected mesh would be a dependency cycle, which the E2E dispatcher rejects.