Skip to content

batch-submitter: eip1559 logging#1424

Merged
tynes merged 1 commit intodevelopfrom
fix/batch-submitter-logging
Sep 7, 2021
Merged

batch-submitter: eip1559 logging#1424
tynes merged 1 commit intodevelopfrom
fix/batch-submitter-logging

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Sep 7, 2021

Description

Log values related to eip1559 before sending a transaction.

  • maxFeePerGas
  • maxPriorityFeePerGas

Also log the gasLimit in case transactions revert.

A clear and concise description of the features you're adding in this pull request.

Additional context
Batches are being submitted to L1 with the same tip and max priority fee, the tip should be set lower. This will help to debug

@changeset-bot
Copy link

changeset-bot bot commented Sep 7, 2021

🦋 Changeset detected

Latest commit: 78ca518

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/batch-submitter Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 2-reviewers A-op-batcher Area: op-batcher labels Sep 7, 2021
Log values related to eip1559 before sending a transaction.
- `maxFeePerGas`
- `maxPriorityFeePerGas`

Also log the `gasLimit` in case transactions revert.
@tynes tynes force-pushed the fix/batch-submitter-logging branch from 0c84185 to 78ca518 Compare September 7, 2021 19:18
@tynes tynes merged commit 087f1f4 into develop Sep 7, 2021
@tynes tynes deleted the fix/batch-submitter-logging branch September 7, 2021 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-op-batcher Area: op-batcher

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant