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

ipfs shutdown errors out on Windows #4078

Closed
kenshyx opened this issue Jul 18, 2017 · 3 comments
Closed

ipfs shutdown errors out on Windows #4078

kenshyx opened this issue Jul 18, 2017 · 3 comments
Labels
help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) topic/test failure Topic test failure topic/windows Windows specific
Milestone
go-ipfs 0.4.14

Comments

@kenshyx
Copy link

kenshyx commented Jul 18, 2017

Version information:

go-ipfs version: 0.4.10-
Repo version: 5
System version: amd64/windows
Golang version: go1.8.3

Type: Bug

Severity: Medium

Description:

On my windows machine when I'm calling shutdown command I always get this Error: "Command failed: "go-ipfs\ipfs.exe" shutdown\nError: Post http://127.0.0.1:5001/api/v0/shutdown?encoding=json&stream-channels=true: read tcp 127.0.0.1:51873->127.0.0.1:5001: wsarecv: An existing connection was forcibly closed by the remote host.\n".
On linux is working as expected.
@Kubuxu
Copy link
Member

Kubuxu commented Jul 18, 2017

On linux it also sometimes doesn't work. What we probably have to do is shutdown with some delay, at least after the API socket is closed.

@whyrusleeping whyrusleeping added kind/bug A bug in existing code (including security flaws) help wanted Seeking public contribution on this issue labels Aug 30, 2017
@whyrusleeping whyrusleeping added this to the Ipfs 0.4.12 milestone Aug 30, 2017
@Kubuxu Kubuxu added the topic/test failure Topic test failure label Oct 19, 2017
@Kubuxu Kubuxu mentioned this issue Oct 19, 2017
Merged
@Kubuxu Kubuxu added the topic/windows Windows specific label Nov 6, 2017
@Kubuxu Kubuxu changed the title ipfs shutdown error ipfs shutdown errors out on Windows Nov 6, 2017
@Kubuxu Kubuxu modified the milestones: Ipfs 0.4.12, go-ipfs 0.4.13 Nov 6, 2017
@richardschneider
Copy link

I'm get an ECONNRESET when calling /api/v0/shutdown the HTTP route. It should return 200, then do the shutdown logic.

@richardschneider richardschneider mentioned this issue Feb 1, 2018
Merged
alanshaw added a commit to ipfs-inactive/js-ipfs-http-client that referenced this issue Jul 4, 2018
@alanshaw
fix: skip bitswap offline tests on all platforms
1781f2a 
The offline tests create and stop a node. ipfs/kubo#4078 seems to not only be restricted to windows.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
daviddias pushed a commit to ipfs-inactive/js-ipfs-http-client that referenced this issue Jul 4, 2018
@alanshaw @daviddias
feat: modular interface tests (#785)
2426072 
* feat: uses modular interface tests

    Reduces code repetition, allows test skipping and running only some tests.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: skip unimplemented files tests and add ls* tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: adds skips for #339

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: adds skips for key and miscellaneous

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: add types and util tests (skipped as currently failing)

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: add pin tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix(pubsub): adds skips for tests on windows

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix(config): adds skip for config.replace

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: re-adds bitswap tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: move detect-node back to dependencies

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add skip reasons

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add reason for pubsub in browser skips

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add bitswap skips for offline errors

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: remove skip for test that was removed

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core version

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update deps and test on CI (#802)

* chore: update interface-ipfs-core

* fix(dag): `dag.put()` allows for optional options (#801)

* fix(dag): ensure `dag.put()` allows for optional options

This is to align with API changes made in

ipfs-inactive/interface-js-ipfs-core@011c417

and

ipfs/js-ipfs#1415

License: MIT
Signed-off-by: Pascal Precht <[email protected]>

* fix(dag): fixes to allow options to be optional

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update ipfsd-ctl dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: increase timeout for addFromURL with wrap-with-directory

Sadly we can't guarantee ipfs.io will respond within 5s

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: skip bitswap offline tests on all platforms

The offline tests create and stop a node. ipfs/kubo#4078 seems to not only be restricted to windows.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
danieldaf pushed a commit to danieldaf/js-ipfs-api that referenced this issue Jul 21, 2018
@alanshaw @danieldaf
feat: modular interface tests (ipfs-inactive#785)
9c9dd25 
* feat: uses modular interface tests

    Reduces code repetition, allows test skipping and running only some tests.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: skip unimplemented files tests and add ls* tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: adds skips for ipfs-inactive#339

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: adds skips for key and miscellaneous

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: add types and util tests (skipped as currently failing)

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* feat: add pin tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix(pubsub): adds skips for tests on windows

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix(config): adds skip for config.replace

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: re-adds bitswap tests

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: move detect-node back to dependencies

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add skip reasons

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add reason for pubsub in browser skips

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: add bitswap skips for offline errors

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: remove skip for test that was removed

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core version

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update deps and test on CI (ipfs-inactive#802)

* chore: update interface-ipfs-core

* fix(dag): `dag.put()` allows for optional options (ipfs-inactive#801)

* fix(dag): ensure `dag.put()` allows for optional options

This is to align with API changes made in

ipfs-inactive/interface-js-ipfs-core@011c417

and

ipfs/js-ipfs#1415

License: MIT
Signed-off-by: Pascal Precht <[email protected]>

* fix(dag): fixes to allow options to be optional

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update interface-ipfs-core dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* chore: update ipfsd-ctl dependency

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: increase timeout for addFromURL with wrap-with-directory

Sadly we can't guarantee ipfs.io will respond within 5s

License: MIT
Signed-off-by: Alan Shaw <[email protected]>

* fix: skip bitswap offline tests on all platforms

The offline tests create and stop a node. ipfs/kubo#4078 seems to not only be restricted to windows.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
Stebalien added a commit that referenced this issue Jul 30, 2018
@Stebalien
these tests are broken
a920136 
see: #4078 (comment)

License: MIT
Signed-off-by: Steven Allen <[email protected]>
@Stebalien
Copy link
Member

Duplicate: #4055

@olizilla olizilla mentioned this issue Apr 18, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) topic/test failure Topic test failure topic/windows Windows specific
Projects
None yet
Milestone
go-ipfs 0.4.14
Development

No branches or pull requests
5 participants
@Stebalien @richardschneider @whyrusleeping @Kubuxu @kenshyx