Skip to content

chore: tune mainnet slasher penalties and sequencer allocation#21451

Merged
alexghr merged 2 commits intov4from
am/alpha-config
Mar 13, 2026
Merged

chore: tune mainnet slasher penalties and sequencer allocation#21451
alexghr merged 2 commits intov4from
am/alpha-config

Conversation

@aminsammara
Copy link
Contributor

Summary

  • Lower SLASH_INACTIVITY_TARGET_PERCENTAGE from 0.8 to 0.7
  • Disable SLASH_DUPLICATE_PROPOSAL_PENALTY and SLASH_DUPLICATE_ATTESTATION_PENALTY (set to 0)
  • Increase SLASH_GRACE_PERIOD_L2_SLOTS from 1200 to 8400
  • Reduce perBlockAllocationMultiplier from 2 to 1.2

Test plan

  • Verify generated config files are updated after running yarn generate in relevant packages
  • Deploy to devnet/testnet and confirm slasher behavior matches new parameters

Lower inactivity target percentage from 0.8 to 0.7, disable duplicate
proposal and attestation penalties, increase grace period from 1200 to
8400 L2 slots, and reduce per-block allocation multiplier from 2 to 1.2.
@spalladino spalladino added the ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure label Mar 12, 2026
…y_method priority fee (#21458)

## Summary
Fixes test failures in PR #21451 caused by changing
`perBlockAllocationMultiplier` from 2 to 1.2 in
`DefaultSequencerConfig`.

- **sequencer-client.test.ts**: Updated expected values in
`computeBlockLimits` tests to reflect multiplier=1.2 instead of 2.
Multi-block mode now correctly expects `maxL2BlockGas=240_000` (was
400_000), `maxTxsPerBlock=24` (was 40), and corresponding DA gas limits.
- **gossip_network.test.ts**: Added comment explaining the proven block
timeout must exceed one full epoch duration plus proving time.

ClaudeBox log: https://claudebox.work/s/95e2c727bf329f12?run=1
@aminsammara aminsammara requested a review from spalladino March 12, 2026 23:31
buildCheckpointIfEmpty: false,
publishTxsWithProposals: false,
perBlockAllocationMultiplier: 2,
perBlockAllocationMultiplier: 1.2,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch 👍

@alexghr alexghr changed the base branch from v4 to backport-to-v4-staging March 13, 2026 11:44
@alexghr alexghr requested a review from charlielye as a code owner March 13, 2026 11:44
@alexghr alexghr changed the base branch from backport-to-v4-staging to v4 March 13, 2026 11:45
@alexghr alexghr enabled auto-merge (squash) March 13, 2026 11:45
@alexghr alexghr merged commit 5bb169c into v4 Mar 13, 2026
13 checks passed
@alexghr alexghr deleted the am/alpha-config branch March 13, 2026 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants