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

flaky mDNS test: TestOtherDiscovery #1167

Closed
marten-seemann opened this issue Aug 21, 2021 · 2 comments · Fixed by libp2p/zeroconf#14 or #1220
Closed

flaky mDNS test: TestOtherDiscovery #1167

marten-seemann opened this issue Aug 21, 2021 · 2 comments · Fixed by libp2p/zeroconf#14 or #1220
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@marten-seemann
Copy link
Contributor

=== RUN   TestOtherDiscovery
      mdns_test.go:99: 
          	Error Trace:	mdns_test.go:99
          	Error:      	Condition never satisfied
          	Test:       	TestOtherDiscovery
          	Messages:   	expected peers to find each other
  --- FAIL: TestOtherDiscovery (26.31s)
@marten-seemann marten-seemann added the kind/bug A bug in existing code (including security flaws) label Aug 21, 2021
@cbegin
Copy link

cbegin commented Sep 13, 2021

This looks like the same issue I'm seeing on MacOS. mDNS simply does not seem to work at all. I tried reimplementing the mDNS example code, and also ran this same test. Both seem to indicate that mDNS is not functional and isn't discovering peers. The debug logs simply iterate forever with the following lines (since I'm using 0.0.0.0). If I specify 127.0.0.1, the results are the same, except that it will lack the adding lines, since the address was specified. There's no other activity. Tried on two different networks. Haven't tried on a different computer.

2021-09-13T10:02:00.595-0600    DEBUG   addrutil        go-addr-util/addr.go:109        ResolveUnspecifiedAddresses:[/p2p-circuit /ip4/0.0.0.0/tcp/54893] [/ip4/10.8.1.113 /ip4/127.0.0.1 /ip6/::1] [/p2p-circuit /ip4/10.8.1.113/tcp/54893 /ip4/127.0.0.1/tcp/54893]
2021-09-13T10:02:05.595-0600    DEBUG   addrutil        go-addr-util/addr.go:64 adding resolved addr:/ip4/0.0.0.0/tcp/54893 /ip4/10.8.1.113/tcp/54893 [/ip4/10.8.1.113/tcp/54893]
2021-09-13T10:02:05.595-0600    DEBUG   addrutil        go-addr-util/addr.go:64 adding resolved addr:/ip4/0.0.0.0/tcp/54893 /ip4/127.0.0.1/tcp/54893 [/ip4/10.8.1.113/tcp/54893 /ip4/127.0.0.1/tcp/54893]
2021-09-13T10:02:05.595-0600    DEBUG   addrutil        go-addr-util/addr.go:109        ResolveUnspecifiedAddresses:[/p2p-circuit /ip4/0.0.0.0/tcp/54893] [/ip4/10.8.1.113 /ip4/127.0.0.1 /ip6/::1] [/p2p-circuit /ip4/10.8.1.113/tcp/54893 /ip4/127.0.0.1/tcp/54893]

@Stebalien
Copy link
Member

@cbegin please file a new bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
3 participants