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

Evaluate Nethermind for Gnosis Chain #868

Closed
Uxio0 opened this issue May 26, 2022 · 6 comments
Closed

Evaluate Nethermind for Gnosis Chain #868

Uxio0 opened this issue May 26, 2022 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@Uxio0
Copy link
Member

Uxio0 commented May 26, 2022

What is needed?

As OpenEthereum was deprecated yesterday (more info on safe-global/safe-pm#87) the only node supported on Gnosis Chain until the merge is Nethermind

Background

  • Tx service for Gnosis Chain is using tracing for indexing
  • Nethermind should support tracing and must be fast enough to keep up with sync at least until The Merge

Related issues

@Uxio0 Uxio0 added the enhancement New feature or request label May 26, 2022
@Uxio0 Uxio0 self-assigned this May 26, 2022
@Uxio0
Copy link
Member Author

Uxio0 commented May 27, 2022

I tried the node fast-synced yesterday. tracing was working for the recent blocks but not for blocks like 3 days in the past. @luarx is resyncing the node without pruning

@Uxio0
Copy link
Member Author

Uxio0 commented Jun 14, 2022

We are having issues with trace_filter: NethermindEth/nethermind#4157 (comment)

@svanegmond
Copy link
Contributor

svanegmond commented Jul 20, 2022

I would love to collaborate with you on improving support for Nethermind as this is what Shyft operates on, being a POA network.

Present issues I have.

  1. While changing safe owners, the gas estimation functions incorrectly, in that the estimated gas is not sufficient. The web interface produces a helpful warning that the transaction may fail, and indeed it does, but doubling the gasLimit causes the transaction to succeed. The gas used was in fact below the estimated gas, so I don't know why I had to raise the limit.

  2. Just recently, new transactions (send funds) are not seen by the tx gateway. The user interface presents the transaction with 0 confirmations. I am still working this out.

DETAIL:  Key (ethereum_tx_id, trace_address)=(\x9e04a5feaddb4eac2a6a0524f993ad3e48705822926158f9a19c4d086f246d8a, 0) already exists.

postgres=# select * from history_internaltx where ethereum_tx_id = '\x9e04a5feaddb4eac2a6a0524f993ad3e48705822926158f9a19c4d086f246d8a';
-[ RECORD 1 ]----+-------------------------------------------------------------------
id               | 182
_from            | \xeb7dc17c09df5a7afa6142c25ad5974117a4d27f
gas              | 50000
data             | \x
to               | \x0000000000000000000000000000000000000000
value            | 0
gas_used         | 50000
contract_address | 
code             | 
output           | 
refund_address   | 
tx_type          | 0
call_type        | 1
trace_address    | 0
error            | 
ethereum_tx_id   | \x9e04a5feaddb4eac2a6a0524f993ad3e48705822926158f9a19c4d086f246d8a
block_number     | 6177141
timestamp        | 2022-07-20 16:50:20+00

postgres=# select * from history_internaltxdecoded where internal_tx_id =182;
-[ RECORD 1 ]--+---------------------------------------------------------------------------------------------------------------------------------------------
internal_tx_id | 182
function_name  | approveHash
arguments      | {"owner": "0x190deF14dAd5f856506C0Af7d7Ffdde78723f74e", "hashToApprove": "9f472d76ab742e632f788391e5bd6d8d052ee7a8c72b75a1364136b32c48667d"}
processed      | t

@Uxio0
Copy link
Member Author

Uxio0 commented Jul 21, 2022

Hi @svanegmond , currently we are dealing with issues preventing us from migrating to Nethermind:

  1. Regarding estimation, that's not something critical to us as we changed the way we estimate (from > 1.3.0 regular eth_estimateGas with gasPrice=0, safeTxGas=0 and baseGas=0 will work.
  2. Regarding the not indexing stuff, that could be many things not related to Nethermind.

Please keep this issue only for discussing low level Ethereum RPC not working (like the issue we have with trace_filter) and not stuff like we don't see transfers in the indexer when Nethermind is configured. But for example "we see an error from the RPC when calling eth_getLogs" is very useful for us.

@Uxio0
Copy link
Member Author

Uxio0 commented Aug 30, 2022

trace_filter is working now on an archive node using v1.14.0. I will proceed to test to sync from scratch a transaction service

@Uxio0
Copy link
Member Author

Uxio0 commented Nov 24, 2022

We are using Nethermind since last month with no issues

@Uxio0 Uxio0 closed this as completed Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants