Skip to content

Conversation

@niran
Copy link
Contributor

@niran niran commented Dec 1, 2025

This task sets the minimum base fee to 200,000 wei (2e-4 gwei) and the DA footprint scalar to 312 on Base Mainnet.

200,000 wei was chosen based on the p5 base fee observed on Base Mainnet from November 7 to 14 and rounded down slightly to get a nice number. The DA footprint was calculated from da_footprint_gas_scalar = block_gas_target_throughput / (blob_target * 128,000 bytes / 12 seconds * estimation_ratio) where block_gas_target_throughput = 30,000,000 gas/sec, blob_target = 6, and estimation_ratio = 1.5. 30,000,000 / (6 * 128,000 / 12 * 1.5) = 312.5, which was rounded down to the nearest integer.

This task was adapted from sepolia/2025-11-18-set-jovian-parameters (#515), the Sepolia version of this task.

niran added 2 commits December 1, 2025 11:39
- Add SetJovianParameters.s.sol script
- Configure .env with mainnet addresses and Jovian params
- Update Makefile for mainnet multisig structure
- Add README and validation file
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Dec 1, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

Copy link
Contributor

@jackchuma jackchuma left a comment

Choose a reason for hiding this comment

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

Overall LGTM - only thing is we've been adding a FACILITATORS.md file to tasks lately - could you add one here as well? Reference: https://github.com/base/contract-deployments/blob/main/mainnet/2025-11-10-increase-gas-and-elasticity-limit/FACILITATOR.md

@niran
Copy link
Contributor Author

niran commented Dec 2, 2025

Overall LGTM - only thing is we've been adding a FACILITATORS.md file to tasks lately - could you add one here as well?

added: aa0d95d

xenoliss
xenoliss previously approved these changes Dec 3, 2025
jackchuma
jackchuma previously approved these changes Dec 3, 2025
@cb-heimdall cb-heimdall dismissed stale reviews from xenoliss and jackchuma December 3, 2025 16:45

Approved review 3535774609 from xenoliss is now dismissed due to new commit. Re-request for approval.

@niran niran requested review from jackchuma and xenoliss December 3, 2025 16:53
@cb-heimdall
Copy link
Collaborator

Review Error for roger-bai-coinbase @ 2025-12-03 16:55:37 UTC
User failed mfa authentication, see go/mfa-help

@niran niran merged commit eca2f14 into main Dec 4, 2025
5 checks passed
@niran niran deleted the mainnet-jovian-params branch December 4, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants