Skip to content

fix: update testnet compatibility test and bust cache on pinned artifact changes#22429

Merged
Thunkar merged 3 commits into
backport-to-v4-next-stagingfrom
claudebox/525093c87914310c-3
Apr 9, 2026
Merged

fix: update testnet compatibility test and bust cache on pinned artifact changes#22429
Thunkar merged 3 commits into
backport-to-v4-next-stagingfrom
claudebox/525093c87914310c-3

Conversation

@AztecBot

@AztecBot AztecBot commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Two changes to fix the testnet compatibility test breakage on v4-next:

1. Fix test to match actual testnet deployment

  • Testnet was deployed without SponsoredFPC — remove it from genesis computation
  • Update expected genesis archive root to 0x15684c8c... (verified against on-chain testnet L1 rollup at 0xf6d0...c1 on Sepolia)
  • All 3 values verified on-chain: VK tree root, protocol contracts hash, genesis archive root

2. Bust test cache when pinned artifacts change

  • Mix hashes of pinned-protocol-contracts.tar.gz, pinned-build.tar.gz (VKs), and mock-protocol-circuits/pinned-build.tar.gz into the compatibility test cache prefix
  • This ensures the test re-runs whenever pinned artifacts are updated, preventing the CI cache from masking mismatches

On-chain verification

$ cast call 0xf6d0...c1 "archiveAt(uint256)(bytes32)" 0 --rpc-url sepolia
0x15684c8c3d2106918d3860f777e50555b7166adff47df13cc652e2e5a50bf5c7  # genesis archive root

$ cast storage 0xf6d0...c1 <config.vkTreeRoot slot> --rpc-url sepolia
0x1dd2644a17d1ddd8831287a78c5a1033b7ae35cdf2a3db833608856c062fc2ba  # VK tree root

$ cast storage 0xf6d0...c1 <config.protocolContractsHash slot> --rpc-url sepolia
0x2672340d9a0107a7b81e6d10d25b854debe613f3272e8738e8df0ca2ff297141  # protocol contracts hash

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 9, 2026
@AztecBot AztecBot changed the title fix: restore SponsoredFPC in pinned protocol contracts tarball fix: update testnet compatibility test — testnet deployed without SponsoredFPC Apr 9, 2026
@AztecBot AztecBot force-pushed the claudebox/525093c87914310c-3 branch from 33bacec to efd1c2c Compare April 9, 2026 08:39
…act changes

- Remove SponsoredFPC from genesis computation (testnet deployed without it)
- Update expected genesis archive root (verified against on-chain testnet L1)
- Mix pinned artifact archive hashes into compatibility test cache prefix
  so the test cache busts when pinned-protocol-contracts.tar.gz or
  pinned-build.tar.gz change
@AztecBot AztecBot changed the title fix: update testnet compatibility test — testnet deployed without SponsoredFPC fix: update testnet compatibility test and bust cache on pinned artifact changes Apr 9, 2026
@AztecBot AztecBot force-pushed the claudebox/525093c87914310c-3 branch from efd1c2c to 9c7015a Compare April 9, 2026 09:04
AztecBot added 2 commits April 9, 2026 10:00
Replace cache_content_hash (which requires full CI environment) with
git hash-object for hashing pinned archive files directly.
Previous CI failure was unrelated (docs/example/bootstrap.sh), not
the cache_content_hash call. Restore the correct approach.
@Thunkar Thunkar marked this pull request as ready for review April 9, 2026 10:46
@Thunkar Thunkar merged commit 8630b41 into backport-to-v4-next-staging Apr 9, 2026
10 checks passed
@Thunkar Thunkar deleted the claudebox/525093c87914310c-3 branch April 9, 2026 10:46
Thunkar added a commit that referenced this pull request Apr 9, 2026
BEGIN_COMMIT_OVERRIDE
cherry-pick: fix(pxe): support custom PrivateKernelProver and unify
EmbeddedWalletOptions (#22348)
fix: update testnet compatibility test and bust cache on pinned artifact
changes (#22429)
fix(pxe): support custom PrivateKernelProver and unify
EmbeddedWalletOptions (backport #22348) (#22391)
refactor!: ephemeral arrays (#22162)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants