-
-
Notifications
You must be signed in to change notification settings - Fork 647
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
Add Optimism Bedrock #901
Add Optimism Bedrock #901
Conversation
3a7a36e
to
728739d
Compare
@martinboehm ready for review, successful bedrock upgrade on my end and blockbook still working as expected |
728739d
to
e08f203
Compare
e08f203
to
935ab06
Compare
ddc88e5
to
def39b1
Compare
@martinboehm if you have any time at some point, would we be able to upstream optimism (bedrock)? I have been running the blockbook for the past 6 months or so without any issues. I am not using the infra, so that could use some 👀 to make sure everything is working as expected. Let me know if you have time or any thoughts. Thank you! |
230c9c9
to
ac4fb9d
Compare
@martinboehm ok, pushed some updates for the backend. note I haven't ever actually run the archive setup fully, but feel free to test it and happy to help debug any issues |
@kaladinlight Blockbook runs well. Just be aware of this commit 35ab7a3. Maybe it would be a good idea to backport it to this PR. I successfully built
After I modified the redirection to
I realized that you are doing the build as a post install step. However, there is no go environment on the server. Could you please move the build step to the |
@martinboehm Hey, just got back from travel. Happy to hear blockbook is running well. I will go ahead and backport that commit and take a closer look at the backend config to sort those issues and give you a ping. |
32112fc
to
7e59bbc
Compare
@martinboehm I backported the |
@kaladinlight Thank you, the reported issues are fixed. However, run into another. The
It is probably tryinig to store the |
@martinboehm There doesn't look to be any directory flag for the op_node local files. I just updated the exec command to run from the backend dir to ensure those files are created and stored there as expected. |
@kaladinlight Thank you, unfortunately I get |
@martinboehm bleh... will take a look at a different strategy. |
@martinboehm ok, I took another look and did find some path flags for these files, they were just nested under |
@martinboehm Ok, pushed those updates. Hopefully the last thing. Let me know if everything is working as expected now, thanks! |
@kaladinlight you are amazing, thank you 👍 |
@martinboehm so close! moved the go build into the extract command. I kept the |
Hmm, build was without problem, however on start I get
It is because
|
@martinboehm can you give it another try with this update f0729e2 |
It looks good now 👍 |
@kaladinlight I have probably the last request/question. I would prefer |
The shortcut is used for the native asset symbol which is accurately |
@martinboehm just making sure you have seen the above comment since I didn't @ you directly. Let me know if that makes sense, or if you have any questions there. Looking forward to getting this in! |
qq: will this chain type be compatible with Base Layer 2? Sorry if not relevant. Thanks! |
This specific PR only supports optimism, but I do have base support working and coming next based off this. Stay tuned! |
@kaladinlight Thank you!! |
Blockbook:
Etherscan: