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

feat: add hildr #42

Merged
merged 8 commits into from
Jul 15, 2024
Merged

feat: add hildr #42

merged 8 commits into from
Jul 15, 2024

Conversation

barnabasbusa
Copy link
Contributor

No description provided.

@barnabasbusa barnabasbusa linked an issue Jun 27, 2024 that may be closed by this pull request
@barnabasbusa
Copy link
Contributor Author

cc: @GrapeBaBa

@barnabasbusa
Copy link
Contributor Author

optimism-java/hildr#172 is currently a blocker. op-batcher can't connect to op-cl-1-hildr, and thus unable to get the chain info. We need rpc addr to be set to 0.0.0.0 in order to make the node's ports reachable from another container.

@thinkAfCod
Copy link

The sequencer functionality of hildr is not yet completed.So users only need to start op-geth when using hildr.

@barnabasbusa
Copy link
Contributor Author

So hildr is incapable of block production without op-node?

@barnabasbusa
Copy link
Contributor Author

what does --enable-sequencer flag do then?

@thinkAfCod
Copy link

So hildr is incapable of block production without op-node?

yes, it is now.

@thinkAfCod
Copy link

what does --enable-sequencer flag do then?

This is predefined. Although it currently has no effect, it will be used to start the sequencer function once Hildr's sequencer feature is implemented.

@GrapeBaBa
Copy link

@barnabasbusa I am on vacation, @thinkAfCod would help to fix any blocking issue for this PR.

@barnabasbusa barnabasbusa marked this pull request as draft July 2, 2024 12:44
@GrapeBaBa
Copy link

@barnabasbusa please use latest version to check again.

@barnabasbusa
Copy link
Contributor Author

Looks a lot better, we seem to have hildr working with op-geth, op-reth, op-erigon

Screenshot 2024-07-15 at 14 51 41

@barnabasbusa barnabasbusa marked this pull request as ready for review July 15, 2024 12:52
@barnabasbusa barnabasbusa enabled auto-merge (squash) July 15, 2024 12:52
@barnabasbusa barnabasbusa requested a review from pk910 July 15, 2024 12:54
@barnabasbusa barnabasbusa merged commit e1ce08e into main Jul 15, 2024
5 checks passed
@barnabasbusa barnabasbusa deleted the bbusa/hildr branch July 15, 2024 15:11
barnabasbusa added a commit that referenced this pull request Nov 22, 2024
🤖 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Hildr
4 participants