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

Support "force_subnet_association" parameter on na_ontap_interface #52691

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@angystardust
Copy link
Contributor

angystardust commented Feb 21, 2019

SUMMARY

This small patch adds a new parameter force_subnet_association to the na_ontap_interface module.
In our storage provisioning workflow, we're creating a subnet with a fixed network range so we need to force the ip association of a LIF to the provisioned subnet.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

na_ontap_interface.py

ADDITIONAL INFORMATION
ansible 2.7.6
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.5 (default, Oct 30 2018, 23:45:53) [GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]

@angystardust

This comment has been minimized.

Copy link
Contributor Author

angystardust commented Feb 21, 2019

+label netapp
+label storage

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Feb 21, 2019

The test ansible-test sanity --test validate-modules [explain] failed with 1 error:

lib/ansible/modules/storage/netapp/na_ontap_interface.py:0:0: E325 Argument 'force_subnet_association' in argument_spec defines type as <class 'bool'> but documentation defines type as 'bool'

click here for bot help

@ansibot

This comment has been minimized.

@lonico

lonico approved these changes Feb 21, 2019

Copy link
Contributor

lonico left a comment

shipit

@ansibot ansibot removed the needs_triage label Feb 21, 2019

@carchi8py
Copy link
Contributor

carchi8py left a comment

Shipit

@ansibot ansibot added the stale_ci label Mar 1, 2019

@carchi8py
Copy link
Contributor

carchi8py left a comment

Shipit

@lonico

lonico approved these changes Mar 13, 2019

Copy link
Contributor

lonico left a comment

shipit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.