-
Notifications
You must be signed in to change notification settings - Fork 2
Add 1704463200_asmap.dat #6
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
Conversation
|
Only difference seems to be the final asmap.dat file. |
|
Seems like the |
|
Hm, running it twice on my machine produces the same output Can @Emzy or @fjahr, can you upload your file? (you'll have to zip it for github to accept it) Here's my asmap.dat: |
I also get the same after a rerun. Here my asmap.dat: |
|
The decoded files ( |
|
Mmm, I got Hash of Sounds like it's putting something machine specific in? |
|
The issue with The hash of my file is now |
|
Works. Now I got the same asmap.dat file. |
|
I get the same hash now, hooray! |
|
Alright, for the first run two ACKs are good enough for me :) Please join the next run in ~24h if possible, which hopefully should run a lot cleaner so we can use it as a demo: #7 |
This is a demo PR adding the
asmap.datresult for the coordinated launch from #4 from the epoch 1704463200. We had a match from 4 out of 5 participants in the coordinated launch.This particular case required an additional cleanup step to account for the current configuration/limitation of the compression algorithm. It allows ASNs up to 33521664, even though all 4 byte numbers are technically allowed (aside from bogon ranges). The result contained a single offending line:
2606:9dc0:6666::/48 AS4199999899. This line is removed as shown below. Going forward I will add some flag to kartograf to do this by default since it's not the first time we have seen this issue and each time it looked the entry was not an actual route but a leaked internal mapping/test.For the following to work you woull also need to check out: bitcoin/bitcoin#28793