Full update does not work about netbox-proxbox

[matthewwee]

I'm not sure whether the MTU value is important to you or Proxbox but after getting this error I just nanoed into /opt/netbox/venv/lib/python3.11/site-packages/netbox_proxbox/proxbox_api/updates/node.py and changed 'mtu' : int(iface.mtu) to 'mtu' : 1500.

For me there was also another error lower down in the file with the line

ntb_iface = list(nb.dcim.interfaces.filter(device_id=netbox_node.id, name=iface['name']))

where I got TypeError: string indices must be integers so I removed the subscript from iface and after that Proxbox seemed to work for me.

Hope this helps

[RomLecat]

It did the tricks, thanks!

[ocfmatt]

For me there was also another error lower down in the file with the line

ntb_iface = list(nb.dcim.interfaces.filter(device_id=netbox_node.id, name=iface['name']))

where I got TypeError: string indices must be integers so I removed the subscript from iface and after that Proxbox seemed to work for me.

Hope this helps

I have this issue too but I don't follow what changes need to be made. What changes did you make on which lines?

Regards,
Matt.

[matthewwee]

I have this issue too but I don't follow what changes need to be made. What changes did you make on which lines?

I can’t say which line number because I’m not sure whether our files are exactly the same but on the line (quite near the bottom)

ntb_iface = list(nb.dcim.interfaces.filter(device_id=netbox_node.id, name=iface['name']))

I changed name=iface['name'] near the end to just name=iface, since iface appears to be of type str.

To be honest I have stopped using Proxbox because I faced the same issue described in #140 and that burned me and I felt I could not trust the software anymore. The fix I described above worked and helped me with the initial import of the data. Since then I have been keeping NetBox and Proxmox in sync manually.

[ocfmatt]

Thanks. I can see the changes you make reference to have already been made in master and develop branches. I am having other undefined type errors on full update and starting to get sidetracked by this. I really wanted this to work but I think I'm at the point where switching to manually adding VMs to netbox is my quicker option and I'll revisit this when I import my larger Proxmox cluster of several hundred VMs to netbox.