-
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
Bug Fixes for ontap_net_vlan.py #44209
Conversation
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.
This removes options from a module releases in 2.6, that may break your users
@@ -81,8 +75,6 @@ def __init__(self): | |||
parent_interface=dict(required=True, type='str'), | |||
vlanid=dict(required=True, type='str'), | |||
node=dict(required=True, type='str'), | |||
interface_name=dict(required=False, type='str'), |
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.
interface_name
has been removed which was in using in 2.6, though docs don't explain what you should use instead
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.
Where should we document this?
ONTAP required interface name to be - anything else would result in an error. Since we are already getting these 2 part of we don't need to ask the user for the interface name to create the correct name.
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.
In the DOCUMENTION
section, possibly under notes:
Also if users of this module in Ansible 2.6 will need to modify there playbooks to work with Ansible 2.7 is should also be documented in docs/docsite/rst/porting_guides/porting_guide_2.7.rst
SUMMARY
Bug Fixes for ontap_net_vlan.py
ISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION