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

Bubble go-merkledag 1.0.2 #5336

Merged
merged 1 commit into from
Aug 5, 2018
Merged

Bubble go-merkledag 1.0.2 #5336

merged 1 commit into from
Aug 5, 2018

Conversation

hsanjuan
Copy link
Contributor

@hsanjuan hsanjuan commented Aug 3, 2018

go-merkledag, go-unixfs and go-path are affected.

License: MIT
Signed-off-by: Hector Sanjuan [email protected]

@hsanjuan hsanjuan self-assigned this Aug 3, 2018
@hsanjuan hsanjuan requested a review from Stebalien August 3, 2018 10:23
@hsanjuan hsanjuan requested a review from Kubuxu as a code owner August 3, 2018 10:23
@ghost ghost added the status/in-progress In progress label Aug 3, 2018
@hsanjuan
Copy link
Contributor Author

hsanjuan commented Aug 3, 2018

I think failures here are not mine. Just tried bubbling go-merkledag 1.0.1 (without my changes) and same failures pop-up. They are in core/coreapi:

--- FAIL: TestObjectAddLink (0.00s)
        object_test.go:231: failed to get block for QmaM3thHPzZE8eDFXnoZRVC2eM3GoKTw8hLAa9z7XtnDUD: blockservice: key not found
=== RUN   TestObjectAddLinkCreate
--- FAIL: TestObjectAddLinkCreate (0.00s)
        object_test.go:279: failed to get block for QmaM3thHPzZE8eDFXnoZRVC2eM3GoKTw8hLAa9z7XtnDUD: blockservice: key not found

go-merkledag 1.0.1 wasn't bubbled yet and it used a new blockservice, which used a new bitswap, which used a newer libp2p 6.0.6. @Stebalien maybe you have an idea?

@schomatis
Copy link
Contributor

@hsanjuan It would seem that the go-blockservice version used in the go-ipfs repo needs to be updated (as suggested in #5330 (comment)),

https://github.com/ipfs/go-ipfs/blob/d0a1cee9c4f36b8f6dd8e5ed2b23b123cdc499fa/package.json#L557-L559

to match the newer version 1.0.2 used in go-merkledag.

The check in

https://github.com/ipfs/go-merkledag/blob/d6d6e000ce1a76d51e4bb98da17d53eb755201cc/merkledag.go#L71-L77

is failing because of a version mismatch between the errors and ipld.ErrNotFound is not getting returned.

@whyrusleeping
Copy link
Member

Opened #5344, the err not found mismatching has been annoying for too long.

License: MIT
Signed-off-by: Jeromy <[email protected]>
@ghost ghost assigned whyrusleeping Aug 5, 2018
@whyrusleeping whyrusleeping merged commit 8d3a350 into master Aug 5, 2018
@whyrusleeping whyrusleeping deleted the update/merkledag branch August 5, 2018 22:40
@ghost ghost removed the status/in-progress In progress label Aug 5, 2018
hacdias pushed a commit to ipfs/boxo that referenced this pull request Jan 27, 2023
Bubble go-merkledag 1.0.2

This commit was moved from ipfs/kubo@8d3a350
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants