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

Sharness failure from killing iptb nodes #3677

Closed
ghost opened this issue Feb 12, 2017 · 7 comments
Closed

Sharness failure from killing iptb nodes #3677

ghost opened this issue Feb 12, 2017 · 7 comments
Assignees
Labels
topic/test failure Topic test failure

Comments

@ghost
Copy link

ghost commented Feb 12, 2017

Version information: master

Type: bug

Priority: P3

Description:

https://travis-ci.org/ipfs/go-ipfs/jobs/200779018

expecting success: 
		iptb stop
	
ipfs kill err:  error killing daemon /Users/travis/gopath/src/github.com/ipfs/go-ipfs/test/sharness/trash directory.t0130-multinode.sh/.iptb/1: os: process already finished
not ok 27 - shut down nodes
@ghost ghost added the topic/test failure Topic test failure label Feb 12, 2017
@whyrusleeping
Copy link
Member

I'm assuming this is an issue with the new makefiles, @Kubuxu take a look

@Kubuxu
Copy link
Member

Kubuxu commented Feb 12, 2017

I don't think it is me. I can't see any way I could have caused it, but it is good to know.

@Kubuxu
Copy link
Member

Kubuxu commented Feb 12, 2017

We should add error log read when iptb fails.

@whyrusleeping
Copy link
Member

It looks like a daemon is dying for some reason during the iptb tests.

@Kubuxu
Copy link
Member

Kubuxu commented Feb 12, 2017

I have updated IPTB in process, maybe something got broken from version to version.

@whyrusleeping
Copy link
Member

Found the issue. iptb kill was using waitto wait on the daemons to be killed, but wait only works on child process. The daemons spawned by iptb are set to not be children of iptb so they can run after the iptb start command finishes. The wait call was erroring and we werent checking the error. I switched it to use a kill(0) loop. Will update in ipfs soon.

@Kubuxu
Copy link
Member

Kubuxu commented Mar 15, 2017

Fixed.

@Kubuxu Kubuxu closed this as completed Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/test failure Topic test failure
Projects
None yet
Development

No branches or pull requests

2 participants