Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

ipfs.metrics.scope error -> TODO: mock repo #800

Open
sanderpick opened this issue May 30, 2019 · 1 comment
Open

ipfs.metrics.scope error -> TODO: mock repo #800

sanderpick opened this issue May 30, 2019 · 1 comment
Labels
bug Something isn't working

Comments

@sanderpick
Copy link
Member

When adding files, I see this with the new App based IPFS setup:

08:47:14.800 ERROR       core: failure on stop:  context canceled; TODO: mock repo builder.go:36

This doesn't seem to cause any other issues... just a noisy error in the logs. The context is always related to "ipfs.metrics.scope":

Screenshot 2019-05-30 09 30 01

@sanderpick sanderpick added the bug Something isn't working label May 30, 2019
@sanderpick
Copy link
Member Author

sanderpick commented Aug 2, 2019

Actually, this is happening because I'm using the HashOnly setting when adding data at one point during the sync process. This causes this mock repo to be constructed: https://github.com/ipfs/go-ipfs/blob/master/core/coreapi/unixfs.go#L64. When it's no longer needed, I'm guessing Close is called on it, which returns this TODO err to the mock node's error channel.

Still harmless, just good to know the actual cause. We should probably PR to go-ipfs a change that either removes the error or otherwise makes it more obvious to users who just want to use the hash only option of add.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant