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

dht: Error correcting DHT entry: EOF asm_amd64.s:2361 #5099

Closed
schomatis opened this issue Jun 8, 2018 · 1 comment · Fixed by #5125
Closed

dht: Error correcting DHT entry: EOF asm_amd64.s:2361 #5099

schomatis opened this issue Jun 8, 2018 · 1 comment · Fixed by #5125
Assignees
@Stebalien
Labels
topic/test failure Topic test failure

Comments

@schomatis
Copy link
Contributor

While working on an UnixFS PR (#5098) I stumbled upon some (what I'm assuming are) unrelated DHT spurious errors on the Travis job:

15:54:17.294 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.295 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
15:54:17.317 ERROR        dht: Error correcting DHT entry: EOF asm_amd64.s:2361
--- FAIL: TestRepublish (2.63s)
	repub_test.go:69: could not resolve name

https://travis-ci.org/ipfs/go-ipfs/jobs/389786949#L824
(Line 824, the anchor might not be working.)
@whyrusleeping whyrusleeping added the topic/test failure Topic test failure label Jun 11, 2018
Stebalien added a commit that referenced this issue Jun 16, 2018
@Stebalien
make republisher test robust against timing issues
a9cb26c 
retry publishing with a longer EOL if the first attempt fails due to a timeout.

fixes #5099

License: MIT
Signed-off-by: Steven Allen <[email protected]>
@Stebalien Stebalien mentioned this issue Jun 16, 2018
Merged
@ghost ghost assigned Stebalien Jun 16, 2018
@ghost ghost added the status/in-progress In progress label Jun 16, 2018
@Stebalien
Copy link
Member

So, the errors and the failure are independent.

The error is just a noisy log line. Fix: https://github.com/libp2p/go-libp2p-kad-dht/pull/166/files#diff-01edc34c001f7e76bbe6d5023612f005L179

The test failure looks like a timing issue. Given that the test fails after 2 seconds and the record expires after 1 second, I'm guessing the record has simply expired.

@ghost ghost removed the status/in-progress In progress label Jun 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Stebalien Stebalien

Labels
topic/test failure Topic test failure
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

make republisher test robust against timing issues ipfs/kubo
3 participants
@Stebalien @whyrusleeping @schomatis