-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
adds two new infoblox lookup plugins #35189
Conversation
privateip
commented
Jan 22, 2018
- nios - lookup plugin to return nios objects to the playbook
- nios_next_ip - lookup plugin to return the next avaiable ip address
* nios - lookup plugin to return nios objects to the playbook * nios_next_ip - lookup plugin to return the next avaiable ip address
The test
The test
|
@@ -0,0 +1,105 @@ | |||
# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add to .github/BOTMETA.yml
like https://github.com/ansible/ansible/pull/35179/files
Will ensure we get notified about bugs and PRs
@@ -31,35 +31,49 @@ class ModuleDocFragment(object): | |||
description: | |||
- Specifies the DNS host name or address for connecting to the remote | |||
instance of NIOS WAPI over REST | |||
- Value can also be specified using INFOBLOX_HOST environment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C(INFOBLOX_HOST)
Same for other ENVs
try: | ||
network = terms[0] | ||
except IndexError: | ||
raise AnsibleError('the network must be specified') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we say which network?
for a given network CIDR | ||
requirements: | ||
- infoblox_client | ||
extends_documentation_fragment: nios |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if extends_documentation_fragments:
works. for lookups
. I don't see it used in any other lookups
make webdocs
walk to store
buy coffee
Review docs
would confirm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
confirmed works fine
Merged, CI failures not related |
* adds two new infoblox lookup plugins * nios - lookup plugin to return nios objects to the playbook * nios_next_ip - lookup plugin to return the next avaiable ip address * adds some additional examples to nios lookup * fix up pep8 failures * updates in response to review comments