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

test(coreapi): use a thread-safe datastore everywhere #6222

Merged
merged 1 commit into from
May 9, 2019
Merged

Conversation

Stebalien
Copy link
Member

@Stebalien Stebalien commented Apr 16, 2019

@ghost ghost assigned Stebalien Apr 16, 2019
@ghost ghost added the status/in-progress In progress label Apr 16, 2019
@Stebalien Stebalien requested a review from magik6k April 16, 2019 00:23
@anacrolix
Copy link
Contributor

I'll test it.

@anacrolix
Copy link
Contributor

It fixes the race condition. There's still another failure but it might be unrelated:

anacrolix@anacrolix-mbp-2018:~/go/src/github.com/ipfs/go-ipfs$ vgo get github.com/libp2p/go-libp2p-kad-dht@master
go: finding github.com/libp2p/go-libp2p-kad-dht master
anacrolix@anacrolix-mbp-2018:~/go/src/github.com/ipfs/go-ipfs$ vgo test -race ./core/coreapi/test -run TestIface/Dht/TestDhtProvide -v
=== RUN   TestIface
=== RUN   TestIface/Dht
=== RUN   TestIface/Dht/TestDhtProvide
--- FAIL: TestIface (0.21s)
    --- FAIL: TestIface/Dht (0.20s)
        --- FAIL: TestIface/Dht/TestDhtProvide (0.19s)
            dht.go:141: got wrong provider:  != QmSDKLcgEdjM8hqh1MEGrLmaZGwgeQRurzGqA9gYyJJkaL

@Stebalien
Copy link
Member Author

ipfs/interface-go-ipfs-core#28

(actually due to the go-libp2p-peer change and a faulty test)

@ghost ghost removed the status/in-progress In progress label Apr 16, 2019
@Stebalien Stebalien reopened this May 1, 2019
@ghost ghost added the status/in-progress In progress label May 1, 2019
fixes libp2p/go-libp2p-kad-dht#321

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
@Stebalien Stebalien merged commit a506625 into master May 9, 2019
@Stebalien Stebalien deleted the fix/dht-321 branch May 9, 2019 06:50
@ghost ghost removed the status/in-progress In progress label May 9, 2019
@Stebalien Stebalien restored the fix/dht-321 branch May 30, 2019 22:32
@Stebalien Stebalien deleted the fix/dht-321 branch May 30, 2019 22:34
guseggert pushed a commit to ipfs/boxo that referenced this pull request Dec 6, 2022
And fix for peer ID formatting changes.

fixes ipfs/kubo#6222 (comment)


This commit was moved from ipfs/interface-go-ipfs-core@29b26f5
guseggert pushed a commit to ipfs/boxo that referenced this pull request Mar 15, 2023
And fix for peer ID formatting changes.

fixes ipfs/kubo#6222 (comment)


This commit was moved from ipfs/interface-go-ipfs-core@29b26f5
Jorropo pushed a commit to ipfs/go-libipfs-rapide that referenced this pull request Mar 23, 2023
And fix for peer ID formatting changes.

fixes ipfs/kubo#6222 (comment)


This commit was moved from ipfs/interface-go-ipfs-core@29b26f5
hacdias pushed a commit that referenced this pull request Nov 29, 2023
And fix for peer ID formatting changes.

fixes #6222 (comment)


This commit was moved from ipfs/interface-go-ipfs-core@29b26f5

This commit was moved from ipfs/boxo@d493b70
hacdias pushed a commit that referenced this pull request Nov 29, 2023
And fix for peer ID formatting changes.

fixes #6222 (comment)


This commit was moved from ipfs/interface-go-ipfs-core@29b26f5

This commit was moved from ipfs/boxo@d493b70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#319 causes test failures in IPFS
3 participants