-
Notifications
You must be signed in to change notification settings - Fork 38
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
Addresses not in proper prefix after scanning #6
Comments
The main purpose of this script is to add only IP addresses --/32--, not prefixes. That's because I assume that the network organisation --prefixes, aggregations, and VRFs-- must be done manually by NOC engineers. It's quite hard to presume a prefix from a given IP address --e.g., 10.1.2.3/32 could belong either to prefix 10.1.2.0/24 or to prefix 10.1.2.0/25 or to prefix 10.1.2.0/26... This way, IP address prefixes and aggregations are done by humans, because only they know how network should be organised. I've just reviewed you PR --thanks in advance--, but AFAIK, single IP addresses should be registered in /32 prefix, not using the mask of prefix they belong. I'll keep both the issue and PR opened, to hear more from you. BTW, sorry for the delay. I'm pretty busy these days, and, yes, I accept that beer. :) |
Hi, thanks for your reply and no worries about the delay. Maybe this can be implemented optionally like with a parameter --add-prefix for example? |
Your suggestion is good: if a certain flag is used, |
I'm trying to update my PR to implement this if a flag is set but my free time for this is limited as well. |
Please check PR #7 which I updated. |
Thanks for your help. I created issue #18 and this will be fixed there along with other improvements. |
EDIT: See PR #7
Hi,
thanks for this script, it's working great.
Only issues I have is that the Addresses which are found are not getting added into any prefix.
Looks like this is happening because there is no VRF added to the addresses.
Is there any way we can associate VRFs to the scanned subnets which will then be added to the addresses?
Also there are almost no hostnames or such added to the description. Can we maybe have an option to add vendor names to the description (from mac addresses)?
Thank you in advance. I'll try to find a way myself with my very very limited python skills, I'd buy you a beer or two If you can help out though! :)
The text was updated successfully, but these errors were encountered: