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

"insufficient funds for gas" error when doing debug_traceBlockByNumber on 0x108F49C #2002

Closed
quick-pawiromitchel opened this issue Jan 25, 2023 · 1 comment · Fixed by #2016
Closed

"insufficient funds for gas" error when doing debug_traceBlockByNumber on 0x108F49C #2002

quick-pawiromitchel opened this issue Jan 25, 2023 · 1 comment · Fixed by #2016
Assignees
@palango
Labels
type:bug Something isn't working

Comments

@quick-pawiromitchel
Copy link

Description

When running debug_traceBlockByNumber on block 0x108F49C results in a insufficient funds for gas error

https://celoscan.io/block/17364124

Current Behavior

Running the following request

{
    "method": "debug_traceBlockByNumber",
    "params": [
        "0x108F49C",
        {
            "tracer": "callTracer"
        }
    ],
    "id": 1,
    "jsonrpc": "2.0"
}

Results in this error

{
    "jsonrpc": "2.0",
    "id": 1,
    "error": {
        "code": -32000,
        "message": "insufficient funds for gas * price + value + gatewayFee: address 0xBF4D711071Cd9c0402387DeCDD53d3babDb82Bf6 have 1702475464032708 want 1710879300000000"
    }
}

Celo client version: v1.7.2
OS: Linux
@carterqw2 carterqw2 added triage Issue needs triaging and removed blockchain triage Issue needs triaging labels Feb 9, 2023
@palango
Copy link
Contributor

palango commented Feb 15, 2023

Hi @quick-pawiromitchel . Thanks for reporting this.

I successfully reproduced this and it might be related to gas calculation in the tracing module.

@palango palango added the type:bug Something isn't working label Feb 20, 2023
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Add regression test for #2002
e97a7d3
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Fix gas price calculations (#2002)
1f791de 
Fix the tracing module to correctly use the base fee when calculating
gas prices.
@palango palango mentioned this issue Feb 20, 2023
Merged
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Add regression test for #2002
05d631b
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Fix gas price calculations (#2002)
021618e 
Fix the tracing module to correctly use the base fee when calculating
gas prices.
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Add regression test for #2002
ae83027
palango added a commit that referenced this issue Feb 20, 2023
@palango
tracer: Fix gas price calculations (#2002)
62aa19d 
Fix the tracing module to correctly use the base fee when calculating
gas prices.
palango added a commit that referenced this issue Feb 23, 2023
@palango
tracer: Add regression test for #2002
2dc44f1
palango added a commit that referenced this issue Feb 23, 2023
@palango
tracer: Fix gas price calculations (#2002)
f6da48e 
Fix the tracing module to correctly use the base fee when calculating
gas prices.
palango added a commit that referenced this issue Feb 23, 2023
@palango
Fix gas price calculations in tracer (#2016)
0caacdf 
* tracer: Add regression test for #2002

* tracer: Fix gas price calculations (#2002)

Fix the tracing module to correctly use the base fee when calculating
gas prices.

* tracer: Only pass fee currency into `getBaseFee`

* tracer: PR review

* tests: Move contract code to testutils
palango added a commit that referenced this issue Mar 10, 2023
@palango
Fix gas price calculations in tracer (#2016)
1b09c3f 
* tracer: Add regression test for #2002

* tracer: Fix gas price calculations (#2002)

Fix the tracing module to correctly use the base fee when calculating
gas prices.

* tracer: Only pass fee currency into `getBaseFee`

* tracer: PR review

* tests: Move contract code to testutils
gastonponti pushed a commit that referenced this issue Mar 13, 2023
@palango @gastonponti
Fix gas price calculations in tracer (#2016)
dfec439 
* tracer: Add regression test for #2002

* tracer: Fix gas price calculations (#2002)

Fix the tracing module to correctly use the base fee when calculating
gas prices.

* tracer: Only pass fee currency into `getBaseFee`

* tracer: PR review

* tests: Move contract code to testutils
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@palango palango

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

Successfully merging a pull request may close this issue.

Fix gas price calculations in tracer celo-org/celo-blockchain
3 participants
@palango @carterqw2 @quick-pawiromitchel