Sean M. Collins comments

Results 215 comments of


                                            Sean M. Collins

[Docs]: Remove extra parenthesis in documentation for netbox.netbox.nb_lookup

Yep. Push a new PR fixing it and i'll 👍

Enhancement: Add feature to manually retry failed task

> @sc68cal how can i get this merged? I am not the maintainer, the maintainer needs to approve and merge

[Feature]: primary_ip4 field returned by nb_inventory plugin should include netmask length

I'm 👎 on this since it's really the prefix that the IP is associated with, where that information resides. Also changing this to add the prefix length to IP addresses...

[Feature]: NetBox 3.7 VPN

Sorry, referenced issues were for support in the lookup plugin after pynetbox & netbox moved the api endpoint

Allow unracking devices by setting position = 0

#330

[Feature]: Allow custom status with Inventory plugin

I am not sure what you are bumping this for. I am fine with this feature, do you plan on contributing it?

[Bug]: Unsupported parameter asn in site module

Can confirm. https://github.com/netbox-community/netbox/issues/6732 changed it from a singular ASN to multiple ASNs and no corresponding change was implemented here. Most likely because we don't have any tests covering creating a...

[Docs]: Mistake in comments for query_filters

Agree, this needs more documentation. Thanks for the research and summary!

Add prefix string to groups and hostvars output by nb_inventory plugin

I tried browsing around for a way to re-run the checks, but couldn't find it. Since this PR is outdated, please merge from master and re-push, and that will re-trigger

Fixes based on Rocky Linux 8 installed from package

What is strange to me, is that the package is supposed to create the `/etc/redis` directory when it installs the package, based on the file listing from `pkgs.org` but I...