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

Snapshot import occasionally fails with "..failed to read from underlying reader.." #10618

Closed
4 of 11 tasks
magik6k opened this issue Apr 4, 2023 · 1 comment · Fixed by #10636
Closed
4 of 11 tasks
Labels
good first issue Good for newcomers help wanted Extra attention is needed kind/bug Kind: Bug

Comments

@magik6k
Copy link
Contributor

magik6k commented Apr 4, 2023

Checklist

  • This is not a security-related bug/issue. If it is, please follow please follow the security policy.
  • I have searched on the issue tracker and the lotus forum, and there is no existing related issue or discussion.
  • I am running the Latest release, the most recent RC(release canadiate) for the upcoming release or the dev branch(master), or have an issue updating to any of these.
  • I did not make any code changes to lotus.

Lotus component

  • lotus daemon - chain sync
  • lotus fvm/fevm - Lotus FVM and FEVM interactions
  • lotus miner/worker - sealing
  • lotus miner - proving(WindowPoSt/WinningPoSt)
  • lotus JSON-RPC API
  • lotus message management (mpool)
  • Other

Lotus Version

latest master

Repro Steps

Import a snapshot with bad internet

Describe the Bug

Looks like snapshot imports occasionally drop, we should add a auto resume mechanism

Logging Information

ERROR: importing chain failed: failed to read from underlying reader: unexpected EOF
@magik6k magik6k added need/triage kind/bug Kind: Bug good first issue Good for newcomers help wanted Extra attention is needed labels Apr 4, 2023
@magik6k
Copy link
Contributor Author

magik6k commented Apr 4, 2023

Relevant code in https://github.com/filecoin-project/lotus/blob/master/cmd/lotus/daemon.go#L433

Worth noting that I mostly see this on .zst imports

@magik6k magik6k changed the title Snapshot occasionally import fails with "..failed to read from underlying reader.." Snapshot import occasionally fails with "..failed to read from underlying reader.." Apr 4, 2023
@rjan90 rjan90 removed the need/triage label Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed kind/bug Kind: Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants