dainok / netdoc

Automatic Network Documentation plugin for NetBox
GNU General Public License v3.0
84 stars 14 forks source link

Ignore prefix created if same vlue already existed on Netbox #107

Closed dietybright closed 8 months ago

dietybright commented 8 months ago

I have fixed all prefix duplicate issue after remove those duplicated prefix in mgmt vrf, after re-discovery, Netdoc auto re-create those prefix

dainok commented 8 months ago

Prefixes are created based on IP VRF. Can you double check if duplicated prefixes have different VRFs?

On Tue, Nov 14, 2023 at 05:13, dietybright @.***(mailto:On Tue, Nov 14, 2023 at 05:13, dietybright < wrote:

I have fixed all prefix duplicate issue after remove those duplicated prefix in mgmt vrf, after re-discovery, Netdoc auto re-create those prefix

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

dietybright commented 8 months ago

Yes it is different vrf.

When discover mgmt switch, the mgmt IP is belong to mgmt-vrf, then it keep re-create mgmt-vrf

dainok commented 8 months ago

Ok, the procedure is: Update IP address and prefix, setting the right VRF Delete useless prefixes Check in the report if everything is fine Import from ARP table

Check Wiki for additional details.

Let me know if you have issues with this procedure

Sent from Proton Mail for iOS

On Tue, Nov 14, 2023 at 09:56, dietybright @.***(mailto:On Tue, Nov 14, 2023 at 09:56, dietybright < wrote:

Yes it is different vrf.

When discover mgmt switch, the mgmt IP is belong to mgmt-vrf, then it keep re-create mgmt-vrf

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

dietybright commented 8 months ago

Hi @dainok ,

I have followed the procedure correctly, the point is when running discovery script periodically , netdoc will import the deleted VRF again

dainok commented 8 months ago

Check https://github.com/dainok/netdoc/pull/111

dainok commented 8 months ago

I also removed /32 prefixes.