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

fix: test: trace_filter test can't be called on head #12426

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Sep 2, 2024

Ref:

height := abi.ChainEpoch(*blkParam.BlockNumber)
if height > head.Height()-1 {
return nil, fmt.Errorf("requested a future epoch (beyond 'latest')")
}
ts, err := chain.GetTipsetByHeight(ctx, height, head, true)

Ref: https://github.com/filecoin-project/lotus/actions/runs/10665372320/job/29558625838
Ref: #12001 (comment)

@rvagg rvagg requested a review from snissn September 2, 2024 23:51
@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label Sep 2, 2024
@rvagg rvagg enabled auto-merge (rebase) September 3, 2024 00:09
@rvagg rvagg merged commit 714eed8 into master Sep 5, 2024
85 of 86 checks passed
@rvagg rvagg deleted the rvagg/eth_trace_wait branch September 5, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip/changelog This change does not require CHANGELOG.md update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants