Sean M. Collins

Results 145 comments of Sean M. Collins

I'm currently getting killed by this when I am using `netbox_cable` with a loop with a lot of cables.

I don't think having to fetch the netbox ID for a prefix is better than actually defining the IP prefix itself.

I believe this needs to be fixed in netbox. Specifically, `letters` should mean any valid UTF8 character, for a slug

I see netbox-community/netbox#3741 In fact, https://github.com/netbox-community/netbox/issues/3741#issuecomment-617244630 says that > the fix for https://github.com/netbox-community/netbox/issues/3721 now allows general Unicode characters in the slug. Hence the code which generates the slug could be...

Hi, Thank you for your lengthy response. Your research matches a lot of what I also researched, and I think what I am driving towards is that we need to...

I am looking for issues that we can re-open on the NetBox server side. netbox-community/netbox#3741 would be ideal to re-open but it is locked and I cannot unlock. Worst case...

``` File "/tmp/ansible-test-e4138r2s/ansible/inventory/manager.py", line 290, in parse_source plugin.parse(self._inventory, self._loader, source, cache=cache) File "/tmp/ansible-test-e4138r2s/ansible/plugins/inventory/auto.py", line 58, in parse plugin.parse(inventory, loader, path, cache=cache) File "/home/runner/.ansible/collections/ansible_collections/netbox/netbox/plugins/inventory/nb_inventory.py", line 1888, in parse self.main() File "/home/runner/.ansible/collections/ansible_collections/netbox/netbox/plugins/inventory/nb_inventory.py",...

Yeah I see that the server returned an HTTP error 406 ``` [WARNING]: * Failed to parse /home/runner/.ansible/collections/ansible_collect ions/netbox/netbox/tests/output/.tmp/integration/inventory-v2.11-vans_y50-ÅÑŚÌβ ŁÈ/tests/integration/targets/inventory-v2.11/files/test-inventory.yml with auto plugin: 406 Not Acceptable ``` https://github.com/netbox-community/ansible_modules/runs/6116117348?check_suite_focus=true#step%3A14%3A5289=

Yep. It's saved as a `DecimalField` in the API server - https://github.com/netbox-community/netbox/blob/develop/netbox/dcim/models/cables.py#L81 Let's fix it.

@rodewalder https://github.com/netbox-community/ansible_modules/blob/devel/plugins/modules/netbox_cable.py#L307 is the line that needs to change (and probably some unit tests) if you are interested in making a contribution