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

ipfs repo gc streaming errors bug #4494

Open
Stebalien opened this issue Dec 15, 2017 · 2 comments
Open

ipfs repo gc streaming errors bug #4494

Stebalien opened this issue Dec 15, 2017 · 2 comments
Labels
topic/commands Topic commands topic/test failure Topic test failure

Comments

@Stebalien
Copy link
Member

Odd test failure:

expecting success: 
    test_must_fail ipfs repo gc --stream-errors 2>&1 | tee repo_gc_out &&
    grep -q "Error: could not retrieve links for $LEAF1" repo_gc_out &&
    grep -q "Error: could not retrieve links for $LEAF2" repo_gc_out &&
    grep -q "Error: garbage collection aborted" repo_gc_out
  
Error: could not retrieve links for QmSijovevteoY63Uj1uC5b8pkpDU5Jgyk2dYBqz3sMJUPc: merkledag: not found
Error: could not retrieve links for QmTbPEyrA1JyGUHFvmtx1FNZVzdBreMv8Hc8jV9sBRWhNA: The block referred to by 'QmTbPEyrA1JyGUHFvmtx1FNZVzdBreMv8Hc8jV9sBRWhNA' was not a valid merkledag node
Error: json: cannot unmarshal number into Go value of type commands.GcResult

not ok 48 - 'ipfs repo gc --stream-errors' should abort and report each error separately

#	
#	    test_must_fail ipfs repo gc --stream-errors 2>&1 | tee repo_gc_out &&
#	    grep -q "Error: could not retrieve links for $LEAF1" repo_gc_out &&
#	    grep -q "Error: could not retrieve links for $LEAF2" repo_gc_out &&
#	    grep -q "Error: garbage collection aborted" repo_gc_out

My guess would be an HTTP streaming bug. We may be reading out the final trailing 0 length as an actual result.

@Stebalien Stebalien added topic/commands Topic commands topic/test failure Topic test failure labels Dec 15, 2017
@Stebalien
Copy link
Member Author

Found in #4453.

@Stebalien Stebalien mentioned this issue Dec 15, 2017
5 tasks
@Stebalien
Copy link
Member Author

I just ran into this again... It's new so it may be related to the commands library. @keks?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/commands Topic commands topic/test failure Topic test failure
Projects
None yet
Development

No branches or pull requests

1 participant