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

Fuse and docker tests together #2241

Closed
wants to merge 7 commits into from
Closed

Fuse and docker tests together #2241

wants to merge 7 commits into from
+40 −12

Conversation

chriscool
Copy link
Contributor

This is trying to have both fuse and docker tests together.
Not sure it is possible though. Let's see how the tests look like.

thelinuxkid and others added 7 commits January 23, 2016 17:19
@thelinuxkid
Test FUSE on travis-ci

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
3467273 
License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@thelinuxkid
Setup offline routing for fuse mount

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
b891dfd 
License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@thelinuxkid
Add TEST_NO_IPV6 test flag

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
fcaaf9e 
The current Travis Trusty build does not have ipv6 support

License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@thelinuxkid
Add osx fuse support for travis-ci

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
4a45895 
License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@thelinuxkid
Do no exclude ipv6 travis tests is osx

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
ec19125 
License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@thelinuxkid
No error if PrivateKey is already loaded

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
6ea702c 
License: MIT
Signed-off-by: Andres Buritica <andres@thelinuxkid.com>
@chriscool
.travis: remove 'dist: trusty'

Verified

This commit was signed with the committer’s verified signature.
Eomm Manuel Spigolon
GPG key ID: 92F122BA19885FF2
Verified
Learn about vigilant mode
54b6f8c 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool chriscool mentioned this pull request Jan 24, 2016
Closed
@chriscool
Copy link
Contributor Author

There is the following go test failure:

ok      github.com/ipfs/go-ipfs/exchange/reprovide  0.021s

--- FAIL: TestFilePersistence (0.18s)

    ipns_test.go:189: Closed, opening new fs

    ipns_test.go:195: open /tmp/fusetest870241229/local/atestfile: no such file or directory

--- FAIL: TestMultipleDirs (0.15s)

    ipns_test.go:206: make a top level dir

    ipns_test.go:212: write a file in it

    ipns_test.go:217: sub directory

    ipns_test.go:222: file in that subdirectory

    ipns_test.go:228: closing mount, then restarting

    ipns_test.go:88: stat /tmp/fusetest901302151/local/test1: no such file or directory

FAIL

FAIL    github.com/ipfs/go-ipfs/fuse/ipns   7.131s

But it doesn't look related and the sharness tests all passed...

@chriscool
Copy link
Contributor Author

On OSX there is the following error on one of the two go tests:

?       github.com/ipfs/go-ipfs/p2p/net/filter  [no test files]

12:32:58.791 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.791 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.791 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.813 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.813 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.813 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.882 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.883 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.883 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.918 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.918 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.918 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.922 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.922 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.922 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.987 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.988 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:58.988 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.072 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.072 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.072 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.073 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.073 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.081 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.139 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.139 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.139 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.366 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.366 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.366 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.398 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.399 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.399 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.403 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.403 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.403 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.419 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.419 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

12:32:59.419 ERROR    mocknet: mock_streamio: read/write on closed pipe mock_stream.go:126

--- FAIL: TestLimitedStreams (2.39s)

    mock_test.go:579: Expected 2ish seconds but got  2.388548056s

FAIL

FAIL    github.com/ipfs/go-ipfs/p2p/net/mock    5.311s

@chriscool
Copy link
Contributor Author

cc @thelinuxkid @whyrusleeping @jbenet

@whyrusleeping
Copy link
Member

the test limited streams one just looks like its caused by a timing issue. That one should get disabled on CI

@whyrusleeping
Copy link
Member

the fuse test failures look like the changes arent being correctly propogated... thats unfun and something i'll need to look at soon

@chriscool
Copy link
Contributor Author

@whyrusleeping issue #2214 is about the first test failure I comment about above.
Also this PR should be closed because @thelinuxkid implemented what I test here in PR #2120.

@chriscool chriscool closed this Jan 29, 2016
@thelinuxkid
Copy link
Contributor

@whyrusleeping the fuse failures have been debugged. All that's left is for the changes described in #2214 to be implemented.

@chriscool chriscool deleted the tlk-fuse-and-docker branch April 30, 2016 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@chriscool @whyrusleeping @thelinuxkid