-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
AutoNAT discovery time is unexpectedly long #538
Comments
@ZenGround0 pointed out that we appear to need several rounds of AutoNAT to assert our NAT status, and that creates too much of a latency as it means that a NAT’ted peer just sits idle for that long before autorelay can jump into action. @vyzo – could you have a look? |
NAT status is asserted after 3 tries. Here are some concrete logs:
|
If it discovers multiple autonat peers, then it will try to use multiple data points to make the determination with confidence (3 is the current confidence level). |
there is also the |
These two PRs fix the main source of latency: Will update the timings in autorelay to reflect those changes once they both merge, and we should be under 1min in total time to relay advertisement with them. |
@ZenGround0 revisiting this topic again to clean the backlog. Could you confirm the PRs @vyzo mentioned fixed the slowness for you? Cheers! |
I'll take a look today, thanks! |
logs from first autonat peer discover to nat status detection:
Empirically that's 1 minute 48 seconds from first peer detection in the logs to status detection. It is a bit longer than the 1 minute expected. This could be related to go-filecoin's peer management interfering with this loose measurement (for example go-filecoin might be shutting down the first several logged connections). In any case this is enough of an improvement that I'm good to close on my end. |
@ZenGround0 cool, thanks for that confirmation (and the logs!). If this becomes a problem again, let's do another round of optimisation. |
AutoNAT discovery takes a long time ~5 minutes in the filecoin devnets. Talking with @raulk today this appears to be unexpected.
The text was updated successfully, but these errors were encountered: