Skip to content

Conversation

@alexhooketh
Copy link

Description

According to the dev of zksync2-python, default priority fee on Sepolia testnet is different from Mainnet. This leads the contract deployment example from the docs to fail when used on Sepolia, but presumably work properly on Mainnet. Creator of the issue and I have personally stumbled upon this bug/ambiguity.

File "/Users/jordan/Desktop/untron-2/cli/untroncli.py", line 126, in deploy_implementation
    tx_hash = zk_web3.zksync.send_raw_transaction(msg)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/web3/eth/eth.py", line 396, in send_raw_transaction
    return self._send_raw_transaction(transaction)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/web3/module.py", line 75, in caller
    result = w3.manager.request_blocking(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/web3/manager.py", line 330, in request_blocking
    return self.formatted_response(
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/web3/manager.py", line 293, in formatted_response
    raise ValueError(error)
ValueError: {'code': 3, 'message': 'max priority fee per gas higher than max fee per gas', 'data': '0x'}

I added max_priority_fee_per_gas=0 # if testnet field on all TxCreateContract and TxCreate2Contract examples. I believe this must be mentioned in the docs, as most deployments in dev environment happen on testnet. However, let me know if I should mention this differently/in some different place of the page.

Linked Issues

Mentioned above

Additional context

@alexhooketh alexhooketh requested a review from a team as a code owner September 28, 2024 16:06
@alexhooketh alexhooketh requested a review from a team September 28, 2024 16:06
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.

1 participant