Repository navigation
Conversation
Zombienet starts the relay validators as `polkadot` from PATH, so the teardown pattern `target/testnet/polkadot` matched only their PVF workers and left both validators running after a run. Match the validators' command line instead; the workers exit with them. Checked on a local network: the old patterns killed 7 processes and left both validators running, the new ones leave nothing behind.
The rehearsal and the drills started the monitor at the best block plus one, with the nonce read there. The monitor ingests finalized blocks only, so until finality caught up it had no verified hash for its start and paged "no verified hash for Pendulum block N". Start it after the finalized head instead, with the nonce read at that head. Checked on a local network with the monitor build: best #3, finalized #1 gave one "no verified hash" page within 20 seconds from the old start and none from the new one.
Base Sepolia's public RPC refuses eth_getLogs over more than 200 blocks
("eth_getLogs is limited to a 200 range"), and the releaser and the
monitor default to 10,000-block ranges. The rehearsal and the drills now
pass MAX_BLOCK_RANGE and BASE_MAX_BLOCK_RANGE of 200 unless the shell
sets them.
The vault rejects perReleaseCap > dailyCap (CapsInverted), but the rehearsal found out only when the deploy reverted, after the network was up. The per-release cap now defaults to the daily cap (28,800 PEN each by default) instead of a fixed 28,800 that a smaller daily cap would fall below, and loading the rehearsal environment refuses an inverted pair with a message naming both values, before anything is spawned or deployed.
The monitor now reports an approval it cannot check against a finalized Pendulum burn as unverified and pauses as a precaution, rather than as a proven mismatch. Chopsticks' clock trails Anvil's, so phase 3 sees that verdict. Accept either one; the automatic pause stays the assertion.
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.
What changes
polkadotfromPATH. The teardown now matches their command line, so no validator is left running after a run; their PVF workers exit with them.MAX_BLOCK_RANGEandBASE_MAX_BLOCK_RANGEof 200 unless the shell sets them, matching Base Sepolia'seth_getLogslimit.Testing
node --checkon every changed script.polkadotprocess was left running.