Skip to content

add ability to pass a tracer into BlockSimulate#9708

Merged
daniellehrner merged 14 commits intobesu-eth:mainfrom
macfarla:9639-traces-for-simulated-tx
Jan 30, 2026
Merged

add ability to pass a tracer into BlockSimulate#9708
daniellehrner merged 14 commits intobesu-eth:mainfrom
macfarla:9639-traces-for-simulated-tx

Conversation

@macfarla
Copy link
Copy Markdown
Contributor

PR description

Add method to BlockSimulate that accepts a custom tracer.
Note there is a limitation that this cannot be used with the EthSimulateV1 option to trace eth transfers. This PR includes a short term quick fix (throw an ex if both are specified) for this, since Glamsterdam will implement tracing of eth transfers by default.

Fixed Issue(s)

Refs #9639

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?

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@daniellehrner daniellehrner enabled auto-merge (squash) January 30, 2026 16:38
@daniellehrner daniellehrner merged commit 2cfe332 into besu-eth:main Jan 30, 2026
46 checks passed
@macfarla macfarla deleted the 9639-traces-for-simulated-tx branch February 4, 2026 22:43
macfarla added a commit to CPerezz/besu that referenced this pull request Feb 6, 2026
* initial groundwork

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* block simulator

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* formatting

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* comments

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* changelog

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* plugin hash

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* changelog to unreleased

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* update plugin api

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* don't need changes to simulateAndPersistWorldState

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

* remove tracer from param

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Co-authored-by: daniellehrner <daniel.lehrner@consensys.net>
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.

2 participants