fix(ci): use pre-funded PRIVATE_KEY on mainnet tempo checks#14608
Draft
decofe wants to merge 3 commits into
Draft
fix(ci): use pre-funded PRIVATE_KEY on mainnet tempo checks#14608decofe wants to merge 3 commits into
decofe wants to merge 3 commits into
Conversation
On mainnet, tempo-check.sh was unconditionally creating a fresh wallet and calling tempo_fundAddress (faucet RPC). The mainnet faucet funds a different token than the default PathUSD the script polls for, causing fund_and_wait to timeout after ~50s. Use the provided PRIVATE_KEY (which is already funded) on mainnet, matching the pattern tempo-deploy.sh already uses. Faucet funding remains the default for devnet/testnet where no PRIVATE_KEY is set. Fixes #14606 Amp-Thread-ID: https://ampcode.com/threads/T-019dfbd8-8dbe-744c-905b-a4dd55880a7b
These stress tests (GasBurner 15M gas, MaxSizeContract 24KB EIP-170) are already validated through other means and account for ~68% of mainnet gas spend per nightly run. Amp-Thread-ID: https://ampcode.com/threads/T-019dfbd8-8dbe-744c-905b-a4dd55880a7b
zerosnacks
requested review from
0xrusowsky,
DaniPopes,
figtracer,
grandizzy,
mablr,
mattsse,
stevencartavia and
zerosnacks
as code owners
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
3 participants
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.
Problem
The nightly CI Tempo workflow (#14606) fails on mainnet because
tempo-check.shunconditionally creates a fresh wallet and callstempo_fundAddress. The mainnet faucet funds token0x20c0...16c6514b53947fdc, but the script pollsbalanceOfon PathUSD (0x20c0...0000) — a different token — so the balance never appears andfund_and_waittimes out after ~50s.Fix
Use the pre-funded
PRIVATE_KEYenv var when available (set for mainnet viasecrets.THROW_AWAY_MAINNET_PKEY), matching the patterntempo-deploy.shalready uses. Faucet funding remains the default for devnet/testnet where noPRIVATE_KEYis set.Fixes #14606