Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

bug: L1 Gas price fix #1618

Closed
ocdbytes opened this issue May 31, 2024 · 0 comments · Fixed by #1625
Closed

bug: L1 Gas price fix #1618

ocdbytes opened this issue May 31, 2024 · 0 comments · Fixed by #1625
Assignees
@ocdbytes
Labels
bug Something isn't working

Comments

@ocdbytes
Copy link
Contributor

Bug Report

Madara version:

0.7

Current behavior:

Currently when we run madara we update the gap price on madara chain with respect to the gas prices on L1. When there is a RPC call made to fetch the price from L1 sometimes it fails leading to crashing the chain.

image

Expected behavior:

It should wait for atleast few retries (logic is already there but somehow not working) and then chain should stop.

Steps to reproduce:

  • Just run the madara and wait for few minutes if prices are not fetched it will crash with above error shown.

Related code:

  • crates/client/l1-gas-price/worker.rs
@ocdbytes ocdbytes added the bug Something isn't working label May 31, 2024
This was referenced Jun 1, 2024
Closed
Merged
@github-actions github-actions bot locked and limited conversation to collaborators Jul 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ocdbytes ocdbytes

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: L1 gas price/fix ocdbytes/madara
feat: L1 gas price/fix ocdbytes/madara
1 participant
@ocdbytes