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

getLeavesWithProofsByNamespace:could not retrieve node #2174

Closed
Tracked by #2106
jcstein opened this issue May 8, 2023 · 4 comments · Fixed by #2310
Closed
Tracked by #2106

getLeavesWithProofsByNamespace:could not retrieve node #2174

jcstein opened this issue May 8, 2023 · 4 comments · Fixed by #2310
Labels
bug Something isn't working external Issues created by non node team members

Comments

@jcstein
Copy link
Member

jcstein commented May 8, 2023

Celestia Node version

v0.9.2

OS

mac/linux

Install tools

No response

Others

No response

Steps to reproduce it

This is from running a rollup and the full logs can be found here

Expected result

shares are retrieved, no error

Actual result

ERROR ipld ipld/namespace_data.go:216 getLeavesWithProofsByNamespace:could not retrieve node {"nID": "\ufffd\ufffd\ufffdy\ufffdq\u0016\ufffd", "pos": 0, "err": "context canceled"}

Relevant log output

No response

Notes

No response
@jcstein jcstein added the bug Something isn't working label May 8, 2023
@github-actions github-actions bot added the external Issues created by non node team members label May 8, 2023
@jcstein jcstein mentioned this issue May 8, 2023
Closed
@jcstein jcstein changed the title ERROR ipld ipld/namespace_data.go:216 getLeavesWithProofsByNamespace:could not retrieve node {"nID": "\ufffd\ufffd\ufffdy\ufffdq\u0016\ufffd", "pos": 0, "err": "context canceled"} getLeavesWithProofsByNamespace:could not retrieve node May 8, 2023
@renaynay renaynay mentioned this issue May 9, 2023
Open
@jcstein
Copy link
Member Author

jcstein commented Jun 2, 2023

Seeing this occur on v0.11.0-rc1 of node:

ERROR	ipld	ipld/namespace_data.go:216	getLeavesWithProofsByNamespace:could not retrieve node	{"nID": "\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0000\u0008\ufffd\ufffdy\ufffdq\u0016\ufffd", "pos": 0, "err": "context canceled"}

@Wondertan
Copy link
Member

@walldiss, @distractedm1nd, seems like we cancel ShrexND and move to IPLD and then it also gets canceled

@walldiss walldiss mentioned this issue Jun 2, 2023
Merged
renaynay pushed a commit that referenced this issue Jun 2, 2023
@walldiss
fix(getter/cascade): cascade getter should return fast on context can…
504ff93 
…cel (#2310)

## Overview

Also fixes incorrect errors.Is for `share.ErrNamespaceNotFound`

Resolves #2174
@Wondertan
Copy link
Member

@jcstein, this particular error should no more happen, so the issue is closed. However, if you were trying to get data and were unable to, then the linked PR does not fixed this. Would be nice to retry on the new rc.

@jcstein
Copy link
Member Author

jcstein commented Jun 2, 2023

cc @tuxcanfly

Wondertan pushed a commit that referenced this issue Jun 6, 2023
@walldiss @Wondertan
fix(getter/cascade): cascade getter should return fast on context can…
fefbf1a 
…cel (#2310)

## Overview

Also fixes incorrect errors.Is for `share.ErrNamespaceNotFound`

Resolves #2174
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working external Issues created by non node team members
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(getter/cascade): cascade getter should return fast on context cancel walldiss/celestia-node
2 participants
@Wondertan @jcstein