Skip to content
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.

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

[SHIP-1523] Add X Layer #12747

Merged
merged 12 commits into from
Apr 30, 2024
Merged

[SHIP-1523] Add X Layer #12747

merged 12 commits into from
Apr 30, 2024

Conversation

friedemannf
Copy link
Member

@friedemannf friedemannf commented Apr 8, 2024

This adds default configs for X Layer, based on the current Polygon zkEVM configs.

Also, add a new ChainType parameter that is used to ignore gasless transactions, similar to what's already being implemented for Gnosis.

…1523-xlayer

# Conflicts:
#	integration-tests/go.mod
#	integration-tests/go.sum
#	integration-tests/load/go.mod
#	integration-tests/load/go.sum
Copy link
Contributor

I see you added a changeset file but it does not contain a tag. Please edit the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

@friedemannf friedemannf marked this pull request as ready for review April 29, 2024 18:47
@friedemannf friedemannf requested review from a team as code owners April 29, 2024 18:47
@@ -9,9 +9,9 @@ import (
// chainSpecificIsUsable allows for additional logic specific to a particular
// Config that determines whether a transaction should be used for gas estimation
func chainSpecificIsUsable(tx evmtypes.Transaction, baseFee *assets.Wei, chainType config.ChainType, minGasPriceWei *assets.Wei) bool {
if chainType == config.ChainGnosis || chainType == config.ChainXDai {
if chainType == config.ChainGnosis || chainType == config.ChainXDai || chainType == config.ChainXLayer {
// GasPrice 0 on most chains is great since it indicates cheap/free transactions.
Copy link
Member Author

Choose a reason for hiding this comment

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

cc @samsondav as you seem to have implemented this back then?

Copy link
Member Author

Choose a reason for hiding this comment

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

I couldn't find any resources on this for Gnosis, and there don't seem to be any gasless transactions anymore happening on Gnosis. Do you remember what info you based this on?

@friedemannf friedemannf requested a review from samsondav April 29, 2024 18:50
Copy link
Contributor

@prashantkumar1982 prashantkumar1982 left a comment

Choose a reason for hiding this comment

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

Looks good.

davidcauchi
davidcauchi previously approved these changes Apr 30, 2024
@friedemannf friedemannf requested a review from amit-momin April 30, 2024 09:41
@dimriou dimriou added this pull request to the merge queue Apr 30, 2024
Merged via the queue into develop with commit 2729ef7 Apr 30, 2024
105 checks passed
@dimriou dimriou deleted the SHIP-1523-xlayer branch April 30, 2024 15:50
davidcauchi pushed a commit that referenced this pull request May 6, 2024
* Add X Layer

* go mod tidy

* Add ChainType for XLayer to ignore transactions with 0 gasPrice

* revert switch stmt

* update changeset

* fix test

* add BlockHistoryEstimator test for XLayer
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.

5 participants