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.

Sign up for GitHub

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

Jump to bottom

fix: Trust L1 RPC #41

Merged
merged 1 commit into from
Jun 25, 2024
Merged

fix: Trust L1 RPC #41

merged 1 commit into from
Jun 25, 2024

Conversation

twoshark
Copy link
Contributor

What

This gets around the following error starting up op-node:

failed to fetch runtime config data" err="failed to fetch unsafe block signing address from system config: failed to fetch proof of storage slot 0x65a7ed542fb37fe237fdfbdd70b31598523fe5b32879e307bae27a0bd9581c08 at block 0x26af2a02dc48831d3d002bf3c064af37e769f05aa3a0dbfa70c4967ba7887ed8: eth_getProof is unimplemented for historical blocks

Testing

A local l1, the neccassary deployments and config for an l2 and the l2 nodes starts with the following:

  1. Checkout this branch and run
kurtosis run path/to/optimism-package --args-file path/to/optimism-package/network_params.yaml

@twoshark
fix: trust l1 rpc in op-node
6c00f9c 
- get around lack of support for eth_getProof on local L1
@barnabasbusa barnabasbusa enabled auto-merge (squash) June 25, 2024 08:42
@barnabasbusa barnabasbusa merged commit 05dcd2e into ethpandaops:main Jun 25, 2024
3 checks passed
@EthPandaOpsBot EthPandaOpsBot mentioned this pull request Jun 19, 2024
Merged
barnabasbusa added a commit that referenced this pull request Nov 22, 2024
@EthPandaOpsBot @barnabasbusa
chore(main): release 1.1.0 (#37)
ce0887d 
🤖 I have created a release *beep* *boop*
---


##
[1.1.0](1.0.0...1.1.0)
(2024-11-21)


### Features

* add additional configuration options for participants
([#81](#81))
([05d0fe9](05d0fe9))
* add custom configurable forks
([#59](#59))
([878906a](878906a))
* add hildr
([#42](#42))
([e1ce08e](e1ce08e))
* add k8s tests
([#45](#45))
([168f062](168f062))
* add op_contract_deployer_params/image
([#63](#63))
([9416a86](9416a86))
* add op-besu
([#58](#58))
([08e7548](08e7548))
* add op-erigon - mininny
([#38](#38))
([c69bf34](c69bf34))
* add sanity check
([#50](#50))
([ceaa9a9](ceaa9a9))
* Add support for op-deployer, fix multiple L2s
([#74](#74))
([eb59380](eb59380))
* add version.txt
([#36](#36))
([1d10311](1d10311))
* add wait for sync for non kurtosis chains
([#49](#49))
([0452e11](0452e11))
* Adding op-nethermind
([#46](#46))
([fba71dd](fba71dd))
* op-deployer upgrade, fix fork support
([#88](#88))
([f8d9089](f8d9089))
* Output Admin Private Key
([#71](#71))
([95abc10](95abc10))


### Bug Fixes

* ci jobs only notify if discord secret available
([#48](#48))
([9473f89](9473f89))
* contract string
([#82](#82))
([c062de0](c062de0))
* count for participant was not calculated with
([#89](#89))
([f511988](f511988))
* DNS-1035 label
([#44](#44))
([1bf8541](1bf8541))
* erigon perms issue
([#76](#76))
([0328314](0328314))
* explicitly set client contexts0
([30017f2](30017f2))
* force offset time to be int
([#64](#64))
([74b6417](74b6417))
* github action should fail if any jobs fail
([#78](#78))
([577ca3b](577ca3b))
* grammar in docs
([#33](#33))
([d6fb36f](d6fb36f))
* handling of config options
([#85](#85))
([5b085c8](5b085c8))
* image label name too long, truncate at 63 chars
([#90](#90))
([5bd687c](5bd687c))
* l2 contract deployment
([#35](#35))
([3f2b16d](3f2b16d))
* Provide default participants
([#77](#77))
([a7d72da](a7d72da))
* Remove wait for finalization for create2 factory deployment
([#60](#60))
([d4c37f0](d4c37f0))
* test k8s
([#47](#47))
([ac17e89](ac17e89))
* trust l1 rpc in op-node
([#41](#41))
([05dcd2e](05dcd2e))
* Update neth image; improve genesis tool
([#73](#73))
([71e61c8](71e61c8))
* update op-geth hash based storage
([#61](#61))
([f0b5b23](f0b5b23))
* Validate root params
([#79](#79))
([914a808](914a808))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: Barnabas Busa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@barnabasbusa barnabasbusa barnabasbusa approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@twoshark @barnabasbusa