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.

Sign up for GitHub

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

Jump to bottom

max_results uses the INFOBLOX_MAX_RETRIES environment variable #198

Closed
dthomson-triumf opened this issue Oct 20, 2023 · 2 comments
Closed

max_results uses the INFOBLOX_MAX_RETRIES environment variable #198

dthomson-triumf opened this issue Oct 20, 2023 · 2 comments

Comments

@dthomson-triumf
Copy link

See:

https://github.com/infobloxopen/infoblox-ansible/blob/b0401ab535e56f14ec8329a0fdae3c54369245d6/plugins/module_utils/api.py#L88C4-L88C4

This should be INFOBLOX_MAX_RESULTS or something like that.
@dthomson-triumf
Copy link
Author

While I'm at it: https://github.com/infobloxopen/infoblox-ansible/blob/master/plugins/module_utils/api.py#L87C92-L87C92 looks like it should probably be using INFOBLOX_WAPI_VERSION not INFOBLOX_WAP_VERSION.

JkhatriInfobox added a commit to JkhatriInfobox/infoblox-ansible that referenced this issue Dec 12, 2023
@JkhatriInfobox
Fixed BUGS for ENV Variables wapi_version and max_results
f4a1c8a 
BUG infobloxopen#198
BUG infobloxopen#185
@JkhatriInfobox JkhatriInfobox mentioned this issue Dec 12, 2023
Merged
JkhatriInfobox added a commit that referenced this issue Dec 12, 2023
@JkhatriInfobox
Fix/dtc objects (#209)
eca5194 
* Fixed LBDN and Pool Object for transform fields
[FIX] LBDN: auth_zone and pool field
[FIX] POOL: servers and monitors field

* Fixed BUGS for ENV Variables wapi_version and max_results

BUG #198
BUG #185

* Resolve Issue #116
[ADD] Version to infoblox-client in requirement.txt
matthewdennett pushed a commit to matthewdennett/infoblox-ansible that referenced this issue May 28, 2024
@JkhatriInfobox @matthewdennett
Fix/dtc objects (infobloxopen#209)
63621b1 
* Fixed LBDN and Pool Object for transform fields
[FIX] LBDN: auth_zone and pool field
[FIX] POOL: servers and monitors field

* Fixed BUGS for ENV Variables wapi_version and max_results

BUG infobloxopen#198
BUG infobloxopen#185

* Resolve Issue infobloxopen#116
[ADD] Version to infoblox-client in requirement.txt
@JkhatriInfobox
Copy link
Collaborator

As env variable for max_result is fixed in PR #209 has been merged and released.

Therefore, we are closing this issue. If the problem persists, please feel free to reopen it or create a new issue."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dthomson-triumf @JkhatriInfobox