Sean M. Collins

Results 215 comments of Sean M. Collins

I am coming around to this idea but I would want the behavior to be opt-in where the user has to explicitly set a configuration value that _then_ marks the...

OK. Is the data that you want to mark as safe for templating, data from a NetBox config context, or is there additional places?

Can you also do a test where you do the API call without the limit, then with the limit to just make sure it's not the case where the first...

I understand. However looking at your logs it appears that you have over 31k virtual machines. Thank you for checking that we are getting the same amount of machines with...

[Have you enabled caching?](https://netbox-ansible-collection.readthedocs.io/en/latest/plugins/nb_inventory_inventory.html#parameter-cache)

The job that had won the race logged: ``` claiming lock on: environments waiting for lock ................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................... claimed! ```

I think perhaps the module defaults should always consult the existing `NETBOX_API` environment variable instead of having users have to manually define it via something like: ```yaml - hosts: all...

@squirrel289 Hi, the latest commits in devel should resolve those issues, please update your branch

I'm digging into this a bit more, I used `git blame` to look up #101 which references #95, where they have a [comment](https://github.com/netbox-community/ansible_modules/issues/95#issuecomment-585920566) where they look at how the Netbox...

Yep, seeing this in CI tests against 4.1 and 4.3. This was being masked by an issue we've had lately with version string parsing so something might have slipped in.