Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pagination limit reached #2

Open
Dome17 opened this issue Jun 14, 2019 · 1 comment
Open

Pagination limit reached #2

Dome17 opened this issue Jun 14, 2019 · 1 comment

Comments

@Dome17
Copy link

Dome17 commented Jun 14, 2019

On line 281 in the netbox-joined inventory script the pagination limit is generally set to 256:
api_url_params['limit'] = 256

When we updated Netbox to the latest version (v.2.5.13), the script didn't work anymore and hanged itself while reading the interfaces of a switch. As already mentioned, the source of the error was the too low limit of 256.

The reason was, that not all interfaces were read by the script as the limit seems to be too small after the upgrade. Missing interfaces lead to a "KEY-Error".

When we raised the limit to 1000, the script worked fine again. (Pagination Max 1000)

However, I am of the opinion that this error could also have occurred inconspicuously, which would have resulted in a misconfiguration of the interfaces. Could the limit of 1000 be exceeded at some point without immediately noticing this in order to avoid misconfiguration?
Or can I rely on this limit being sufficient?

Would be great if you could take a look at this failure. Thank you very much.

@ypid
Copy link
Contributor

ypid commented Jul 20, 2020

This issue does not exist with Netbox 2.8 anymore, at least to my understanding. This script uses the default maximum of 1000 and makes subsequent requests if more than 1000 entries exist. I tested this by setting the script limit to 4. The exact same output is generated. All good. Also note that there are two settings: MAX_PAGE_SIZE and PAGINATE_COUNT.

This can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants