Skip to content

Support blob fee market when zeroBaseFee enabled for forks with blobs #9344

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jframe merged 3 commits into from
Oct 22, 2025
Merged

Support blob fee market when zeroBaseFee enabled for forks with blobs #9344

jframe merged 3 commits into from
Oct 22, 2025

Conversation

@jframe
Copy link
Contributor

@jframe jframe commented Oct 22, 2025

PR description

Support blob fee market for zero fee market for forks with blobs

Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests
  • hive tests: Engine or other RPCs modified?

jframe added 2 commits October 22, 2025 16:48
@jframe
Support blob fee market for zero fee market for forks with blobs
efb8f80 
Signed-off-by: Jason Frame <jason.frame@consensys.net>
@jframe
Test for zero blob fee market
4bbe637 
Signed-off-by: Jason Frame <jason.frame@consensys.net>
@jframe jframe marked this pull request as ready for review October 22, 2025 07:25
@jframe
rename test
6a81d7a 
Signed-off-by: Jason Frame <jason.frame@consensys.net>
pinges
pinges approved these changes Oct 22, 2025
View reviewed changes
Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

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

LGTM

@jframe jframe merged commit 85b8c77 into hyperledger:main Oct 22, 2025
46 checks passed
@jframe jframe deleted the zero_blob_fee_market branch October 22, 2025 10:16
@jframe jframe changed the title Support blob fee market for zero fee market for forks with blobs Support blob fee market when zeroBaseFee enabled for forks with blobs Oct 22, 2025
jflo pushed a commit to jflo/besu that referenced this pull request Nov 17, 2025
@jframe @jflo
Support blob fee market when zeroBaseFee enabled for forks with blobs (…
7e61508 
…hyperledger#9344)

Signed-off-by: Jason Frame <jason.frame@consensys.net>
pinges pushed a commit to pinges/besu that referenced this pull request Dec 15, 2025
@jframe @pinges
Support blob fee market when zeroBaseFee enabled for forks with blobs (…
82da875 
…hyperledger#9344)

Signed-off-by: Jason Frame <jason.frame@consensys.net>
Signed-off-by: stefan.pingel@consensys.net <stefan.pingel@consensys.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pinges pinges pinges approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jframe @pinges