Sean M. Collins

Results 215 comments of Sean M. Collins

@StoffelCPR So, at least in CI, our tests against NetBox API 4.1 are not creating the new service correctly since it appears the API changed, and we are not associating...

I believe the responsible code is here https://github.com/netbox-community/ansible_modules/blob/devel/plugins/module_utils/netbox_ipam.py#L219

So, looks like #1421 got us closer to resolving this issue but it's not complete. It appears to be two situations: * For API 4.1 - we are not sending...

@StoffelCPR Hi, the maintainers are volunteers so we give what time we have. You caught us during a period of time where we were not as active. Thank you for...

Yes there appears to be an issue with what was released #1465

The sanity failures are unrelated to your patch. I think a new version of ansible's sanity test checker was released and they've been increasing the strictness. Let me investigate and...

Please rebase this on the latest commits to `devel` to pick up the fix, and this should be good to go. I'm :+1: on this being merged as soon as...

@dhoffend If you have PR to `pynetbox` that solves this at a lower layer, that's great. If not, let's try and figure out a way forward. It _may_ have to...

OK, I defer to everyone else's judgement here since I haven't researched this nearly as deeply. Let me know what I can do to assist!

I do feel like we have some inconsistencies in it, because I've tried assigning `primary_ip4` and `oob_ip` as part of a `netbox_device` and it silently ignored it, or when I...