-
Notifications
You must be signed in to change notification settings - Fork 23
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
CAIDA's IXP info and membership #113
CAIDA's IXP info and membership #113
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added some comment :)
We should also update the README since there are dependencies. The ix_asns
crawler depends on ixs
, and ixs
depends on the PeeringDB crawlers.
Thanks for the comments. I have completely forgot about the README! 😨 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me now.
This adds CAIDA IXP information and IXP membership. It may adds some redundant links with PeeringDB but as it also adds new IXPs (found via HE) I wanted to make sure the links are consistent across all IXP.
How Has This Been Tested?
Tried locally, the data seems properly imported to me.
Types of changes
Checklist: